> +/* > + * For Intel processors, the meaning is the architectural PMU version > + * number. > + * > + * For AMD processors: 1 corresponds to the prior versions, and 2 > + * corresponds to AMD PerfMonV2. > + */ > +static uint32_t has_pmu_version;
The "has_" prefix sounds like a boolean type. So what about "pmu_version"? Others look good to me, Reviewed-by: Zhao Liu <zhao1....@intel.com>