Re: [PATCH] powercap: intel_rapl: Expose package CPUs in associated_cpus

[Dennis Chen]

Hi Kuppuswamy,

On 11/6/2026 11:41 AM, Kuppuswamy Sathyanarayanan wrote:
> Is there a user of associated_cpus? Neither perf nor turbostat reads it
> today. The "any CPU in the package" support that turbostat needed comes
> from commit 7537bae8b6eb ("powercap: intel_rapl: Remove incorrect CPU
> check in PMU context") and from is_rp_pmu_cpu(). Your patch keeps both.
> 
> If there is no consumer, I would prefer a plain revert of 525e0064f3d8.
> It fixes the issue with a smaller diff and does not add new sysfs ABI.

There is no userspace consumer. My idea was to keep the flexibility for 
userspace tools mentioned in the original commit by exposing the full
set in associated_cpus.

In that case, I will send out a v2 as a revert of 525e0064f3d8.

> This has been in since v7.0, so please add Cc: stable@xxxxxxxxxxxxxxx.

Will do.

Thanks for the review.

Best,
Dennis Chen