Re: [PATCH V4 04/17] perf stat: Clear HEADER_CPU_PMU_CAPS

From: Arnaldo Carvalho de Melo
Date: Fri Apr 17 2020 - 10:42:52 EST


Em Thu, Mar 19, 2020 at 01:25:04PM -0700, kan.liang@xxxxxxxxxxxxxxx escreveu:
> From: Kan Liang <kan.liang@xxxxxxxxxxxxxxx>
>
> The CPU PMU capabilities information is only useful for perf record with
> LBR call stack.
>
> Clear the header for perf stat.

Ditto.

> Reviewed-by: Andi Kleen <ak@xxxxxxxxxxxxxxx>
> Signed-off-by: Kan Liang <kan.liang@xxxxxxxxxxxxxxx>
> ---
> tools/perf/builtin-stat.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tools/perf/builtin-stat.c b/tools/perf/builtin-stat.c
> index ec053dc1e35c..b5c8a5ab5e75 100644
> --- a/tools/perf/builtin-stat.c
> +++ b/tools/perf/builtin-stat.c
> @@ -1595,6 +1595,7 @@ static void init_features(struct perf_session *session)
> perf_header__clear_feat(&session->header, HEADER_TRACING_DATA);
> perf_header__clear_feat(&session->header, HEADER_BRANCH_STACK);
> perf_header__clear_feat(&session->header, HEADER_AUXTRACE);
> + perf_header__clear_feat(&session->header, HEADER_CPU_PMU_CAPS);
> }
>
> static int __cmd_record(int argc, const char **argv)
> --
> 2.17.1
>

--

- Arnaldo