Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1360245
| From | Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH] powercap: intel_rapl: PSys support |
| Date | 2016-03-18 00:00 +0100 |
| Message-ID | <rdP3A-5d9-11@gated-at.bofh.it> (permalink) |
| References | <rdOAy-50E-5@gated-at.bofh.it> <rdP3A-5d9-7@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Thu, 2016-03-17 at 23:51 +0100, Borislav Petkov wrote: > On Thu, Mar 17, 2016 at 03:23:29PM -0700, Srinivas Pandruvada wrote: > > > > Skylake processor supports a new set of RAPL registers for > > controlling > > entire platform instead of just CPU package. This is useful for > > thermal > > and power control when source of power/thermal is not just CPU/GPU. > > This change adds a new platform domain (AKA PSys) to the current > > power capping Intel RAPL driver. > > PSys also supports PL1 (long term) and PL2 (short term) control > > like > > package domain. This also follows same MSRs for energy and time > > units as package domain. > > Unlike package domain, PSys support requires more than just > > processor > > level implementation. The other parts in the system need additional > > implementation, which OEMs needs to support. So not all Skylake > > systems will support PSys. > > > > Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel > > .com> > > --- > > arch/x86/include/asm/msr-index.h | 3 ++ > > drivers/powercap/intel_rapl.c | 63 > > ++++++++++++++++++++++++++++++++++++++++ > > 2 files changed, 66 insertions(+) > > > > diff --git a/arch/x86/include/asm/msr-index.h > > b/arch/x86/include/asm/msr-index.h > > index 2da46ac..bd383f5 100644 > > --- a/arch/x86/include/asm/msr-index.h > > +++ b/arch/x86/include/asm/msr-index.h > > @@ -190,6 +190,9 @@ > > #define MSR_PP1_ENERGY_STATUS 0x00000641 > > #define MSR_PP1_POLICY 0x00000642 > > > > +#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. Thanks, Srinivas > > -- > Regards/Gruss, > Boris. > > SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, > HRB 21284 (AG Nürnberg)
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH] powercap: intel_rapl: PSys support Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> - 2016-03-17 23:30 +0100
Re: [PATCH] powercap: intel_rapl: PSys support Borislav Petkov <bp@suse.de> - 2016-03-18 00:00 +0100
Re: [PATCH] powercap: intel_rapl: PSys support Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> - 2016-03-18 00:00 +0100
Re: [PATCH] powercap: intel_rapl: PSys support Jacob Pan <jacob.jun.pan@linux.intel.com> - 2016-03-18 03:00 +0100
Re: [PATCH] powercap: intel_rapl: PSys support Borislav Petkov <bp@suse.de> - 2016-03-18 10:10 +0100
Re: [PATCH] powercap: intel_rapl: PSys support Thomas Gleixner <tglx@linutronix.de> - 2016-03-18 11:00 +0100
csiph-web