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

From: Thomas Gleixner
Date: Mon Feb 22 2016 - 07:00:14 EST


On Mon, 22 Feb 2016, Peter Zijlstra wrote:

> 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?

Duh, yes.