Re: [patch V2 26/28] x86/perf/intel/rapl: Convert it to a per package facility

[Peter Zijlstra]

On Mon, Feb 22, 2016 at 11:07:02AM -0000, Thomas Gleixner wrote:
> +static inline struct rapl_pmu *cpu_to_rapl_pmu(unsigned int cpu)
> +{
> +     return rapl_pmus->pmus[topology_logical_package_id(cpu)];
> +}

> @@ -370,6 +378,7 @@ static int rapl_pmu_event_init(struct pe
>               return -EINVAL;
>  
>       /* must be done before validate_group */
> +     pmu = cpu_to_rapl_pmu(event->cpu);
>       event->cpu = pmu->cpu;
>       event->pmu_private = pmu;

This again looks like pmu_private is 'trivially' replacable with
something like:

static inline struct rapl_pmu *event_to_rapl_pmu(struct perf_event *event)
{
        return cpu_to_rapl_pmu(event->cpu);
}