Re: [PATCH 4/9] perf/x86/intel: Support hardware TopDown metrics

From: Peter Zijlstra
Date: Tue May 28 2019 - 08:56:57 EST


On Tue, May 21, 2019 at 02:40:50PM -0700, kan.liang@xxxxxxxxxxxxxxx wrote:
> + x86_pmu.has_metric = x86_pmu.intel_cap.perf_metrics;

It makes sense to duplicate that state because?

> @@ -717,6 +729,8 @@ struct x86_pmu {
> struct extra_reg *extra_regs;
> unsigned int flags;
>
> + bool has_metric;
> +
> /*
> * Intel host/guest support (KVM)
> */

You forgot how I feel about _Bool in composite types _again_ ?