Re: [patch V2 10/28] x86/perf/intel_uncore: Store box in event->pmu_private

From: Peter Zijlstra
Date: Mon Feb 22 2016 - 06:45:34 EST


On Mon, Feb 22, 2016 at 11:06:50AM -0000, Thomas Gleixner wrote:
> Index: b/arch/x86/kernel/cpu/perf_event_intel_uncore_snb.c
> ===================================================================
> --- a/arch/x86/kernel/cpu/perf_event_intel_uncore_snb.c
> +++ b/arch/x86/kernel/cpu/perf_event_intel_uncore_snb.c
> @@ -313,6 +313,7 @@ static int snb_uncore_imc_event_init(str
> return -EINVAL;
>
> event->cpu = box->cpu;
> + event->pmu_private = pmu;

This seems rather pointless, did you want that to be '= box' instead?

>
> event->hw.idx = -1;
> event->hw.last_tag = ~0ULL;
>
>