On 3/7/25 1:19 AM, Zhao Liu wrote: >> +/* >> + * 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"? Sure. I will change to pmu_version. Thank you very much! Dongli Zhang > > Others look good to me, > > Reviewed-by: Zhao Liu <zhao1....@intel.com> > >
