On Thu, 17 Mar 2016, Jacob Pan wrote:

> On Thu, 17 Mar 2016 15:56:03 -0700
> Srinivas Pandruvada <[email protected]> wrote:
> 
> > > > +#define MSR_PLATFORM_ENERGY_STATUS     0x0000064d
> > > > +#define MSR_PLATFORM_POWER_LIMIT       0x0000065c  
> > > Please add those defines to intel_rapl.c as they're being used only
> > > there, apparently.  
> > OK. I will move them.
> 
> I think it could be used by perf rapl pmu also, but later.

Right. And they should be added to perf _BEFORE_ that powercap stuff utilizes
them.

Thanks,

        tglx

Reply via email to