Hi Sandipan, On 11/8/24 5:09 AM, Sandipan Das wrote: > On 11/4/2024 3:10 PM, Dongli Zhang wrote: [snip] >> + * separate set of addresses for the selector and counter >> + * registers. Additionally, the address of the next selector or >> + * counter register is determined by incrementing the address >> + * of the current register by two. >> */ >> - if (num_pmu_gp_counters == AMD64_NUM_COUNTERS_CORE) { >> + if (num_pmu_gp_counters == AMD64_NUM_COUNTERS_CORE || >> + has_pmu_version == 2) { > > Future PMU versions are expected to be backwards compatible. So it may be > better to look for has_pmu_version > 1. > Sure. I will change that in v2. Thank you very much! Dongli Zhang