* [PATCH] The x86 cpu RAPL id was added in to support BXT core
@ 2015-10-13 15:59 amy.l.wiles
2015-10-13 23:07 ` Pandruvada, Srinivas
0 siblings, 1 reply; 4+ messages in thread
From: amy.l.wiles @ 2015-10-13 15:59 UTC (permalink / raw)
To: rui.zhang, srinivas.pandruvada, linux-pm; +Cc: Amy Wiles
From: Amy Wiles <amy.l.wiles@intel.com>
BUG=None
TEST=Navigate to /sys/class/powercap and check that the directory intel-rapl:0 is present
Signed-off-by: Amy Wiles <amy.l.wiles@intel.com>
---
drivers/powercap/intel_rapl.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/powercap/intel_rapl.c b/drivers/powercap/intel_rapl.c
index 5efacd0..cc97f08 100644
--- a/drivers/powercap/intel_rapl.c
+++ b/drivers/powercap/intel_rapl.c
@@ -1102,6 +1102,7 @@ static const struct x86_cpu_id rapl_ids[] __initconst = {
RAPL_CPU(0x4A, rapl_defaults_tng),/* Tangier */
RAPL_CPU(0x56, rapl_defaults_core),/* Future Xeon */
RAPL_CPU(0x5A, rapl_defaults_ann),/* Annidale */
+ RAPL_CPU(0X5C, rapl_defaults_core),/* Broxton */
RAPL_CPU(0x5E, rapl_defaults_core),/* Skylake-H/S */
RAPL_CPU(0x57, rapl_defaults_hsw_server),/* Knights Landing */
{}
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] The x86 cpu RAPL id was added in to support BXT core
2015-10-13 15:59 [PATCH] The x86 cpu RAPL id was added in to support BXT core amy.l.wiles
@ 2015-10-13 23:07 ` Pandruvada, Srinivas
2015-10-14 4:53 ` Jacob Pan
0 siblings, 1 reply; 4+ messages in thread
From: Pandruvada, Srinivas @ 2015-10-13 23:07 UTC (permalink / raw)
To: Zhang, Rui, Wiles, Amy L, linux-pm@vger.kernel.org
Cc: jacob.jun.pan@linux.intel.com
On Tue, 2015-10-13 at 08:59 -0700, amy.l.wiles@intel.com wrote:
> From: Amy Wiles <amy.l.wiles@intel.com>
>
> BUG=None
> TEST=Navigate to /sys/class/powercap and check that the directory
> intel-rapl:0 is present
Remove above lines and just say "Enabled Broxton RAPL support".
Copied to Jacob.
Thanks,
Srinivas
> Signed-off-by: Amy Wiles <amy.l.wiles@intel.com>
> ---
> drivers/powercap/intel_rapl.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/powercap/intel_rapl.c
> b/drivers/powercap/intel_rapl.c
> index 5efacd0..cc97f08 100644
> --- a/drivers/powercap/intel_rapl.c
> +++ b/drivers/powercap/intel_rapl.c
> @@ -1102,6 +1102,7 @@ static const struct x86_cpu_id rapl_ids[]
> __initconst = {
> RAPL_CPU(0x4A, rapl_defaults_tng),/* Tangier */
> RAPL_CPU(0x56, rapl_defaults_core),/* Future Xeon */
> RAPL_CPU(0x5A, rapl_defaults_ann),/* Annidale */
> + RAPL_CPU(0X5C, rapl_defaults_core),/* Broxton */
> RAPL_CPU(0x5E, rapl_defaults_core),/* Skylake-H/S */
> RAPL_CPU(0x57, rapl_defaults_hsw_server),/* Knights Landing
> */
> {}
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] The x86 cpu RAPL id was added in to support BXT core
2015-10-13 23:07 ` Pandruvada, Srinivas
@ 2015-10-14 4:53 ` Jacob Pan
0 siblings, 0 replies; 4+ messages in thread
From: Jacob Pan @ 2015-10-14 4:53 UTC (permalink / raw)
To: Pandruvada, Srinivas; +Cc: Zhang, Rui, Wiles, Amy L, linux-pm@vger.kernel.org
On Tue, 13 Oct 2015 23:07:21 +0000
"Pandruvada, Srinivas" <srinivas.pandruvada@intel.com> wrote:
> On Tue, 2015-10-13 at 08:59 -0700, amy.l.wiles@intel.com wrote:
> > From: Amy Wiles <amy.l.wiles@intel.com>
> >
> > BUG=None
> > TEST=Navigate to /sys/class/powercap and check that the directory
> > intel-rapl:0 is present
> Remove above lines and just say "Enabled Broxton RAPL support".
> Copied to Jacob.
>
agreed, other than that looks good to me.
> Thanks,
> Srinivas
>
>
> > Signed-off-by: Amy Wiles <amy.l.wiles@intel.com>
> > ---
> > drivers/powercap/intel_rapl.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/powercap/intel_rapl.c
> > b/drivers/powercap/intel_rapl.c
> > index 5efacd0..cc97f08 100644
> > --- a/drivers/powercap/intel_rapl.c
> > +++ b/drivers/powercap/intel_rapl.c
> > @@ -1102,6 +1102,7 @@ static const struct x86_cpu_id rapl_ids[]
> > __initconst = {
> > RAPL_CPU(0x4A, rapl_defaults_tng),/* Tangier */
> > RAPL_CPU(0x56, rapl_defaults_core),/* Future Xeon */
> > RAPL_CPU(0x5A, rapl_defaults_ann),/* Annidale */
> > + RAPL_CPU(0X5C, rapl_defaults_core),/* Broxton */
> > RAPL_CPU(0x5E, rapl_defaults_core),/* Skylake-H/S */
> > RAPL_CPU(0x57, rapl_defaults_hsw_server),/* Knights Landing
> > */
> > {}
[Jacob Pan]
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] The x86 cpu RAPL id was added in to support BXT core
@ 2015-10-13 16:13 amy.l.wiles
0 siblings, 0 replies; 4+ messages in thread
From: amy.l.wiles @ 2015-10-13 16:13 UTC (permalink / raw)
To: rui.zhang, srinivas.pandruvada, linux-pm; +Cc: Amy Wiles
From: Amy Wiles <amy.l.wiles@intel.com>
BUG=None
TEST=Navigate to /sys/class/powercap and check that the directory intel-rapl:0 is present
Signed-off-by: Amy Wiles <amy.l.wiles@intel.com>
---
drivers/powercap/intel_rapl.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/powercap/intel_rapl.c b/drivers/powercap/intel_rapl.c
index 5efacd0..cc97f08 100644
--- a/drivers/powercap/intel_rapl.c
+++ b/drivers/powercap/intel_rapl.c
@@ -1102,6 +1102,7 @@ static const struct x86_cpu_id rapl_ids[] __initconst = {
RAPL_CPU(0x4A, rapl_defaults_tng),/* Tangier */
RAPL_CPU(0x56, rapl_defaults_core),/* Future Xeon */
RAPL_CPU(0x5A, rapl_defaults_ann),/* Annidale */
+ RAPL_CPU(0X5C, rapl_defaults_core),/* Broxton */
RAPL_CPU(0x5E, rapl_defaults_core),/* Skylake-H/S */
RAPL_CPU(0x57, rapl_defaults_hsw_server),/* Knights Landing */
{}
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-10-14 4:53 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-13 15:59 [PATCH] The x86 cpu RAPL id was added in to support BXT core amy.l.wiles
2015-10-13 23:07 ` Pandruvada, Srinivas
2015-10-14 4:53 ` Jacob Pan
-- strict thread matches above, loose matches on Subject: below --
2015-10-13 16:13 amy.l.wiles
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox