* [PATCH] powercap/rapl: add support for cpu model 0x3f
@ 2014-08-14 14:26 Jason Baron
2014-09-01 23:49 ` Rafael J. Wysocki
0 siblings, 1 reply; 4+ messages in thread
From: Jason Baron @ 2014-08-14 14:26 UTC (permalink / raw)
To: jacob.jun.pan, linux-pm
Cc: rafael.j.wysocki, durgadoss.r, srinivas.pandruvada
I've confirmed that monitoring the package power usage as well as setting power
limits appear to be working as expected. Supports the package and dram domains.
Tested aginst cpu:
Intel(R) Xeon(R) CPU E5-2650 v3 @ 2.30GHz
Signed-off-by: Jason Baron <jbaron@akamai.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 b1cda6f..a362dcc 100644
--- a/drivers/powercap/intel_rapl.c
+++ b/drivers/powercap/intel_rapl.c
@@ -953,6 +953,7 @@ static const struct x86_cpu_id rapl_ids[] = {
{ X86_VENDOR_INTEL, 6, 0x3a},/* Ivy Bridge */
{ X86_VENDOR_INTEL, 6, 0x3c},/* Haswell */
{ X86_VENDOR_INTEL, 6, 0x3d},/* Broadwell */
+ { X86_VENDOR_INTEL, 6, 0x3f},/* Haswell */
{ X86_VENDOR_INTEL, 6, 0x45},/* Haswell ULT */
/* TODO: Add more CPU IDs after testing */
{}
--
1.8.2.rc2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] powercap/rapl: add support for cpu model 0x3f
2014-08-14 14:26 [PATCH] powercap/rapl: add support for cpu model 0x3f Jason Baron
@ 2014-09-01 23:49 ` Rafael J. Wysocki
2014-09-02 14:31 ` Jason Baron
0 siblings, 1 reply; 4+ messages in thread
From: Rafael J. Wysocki @ 2014-09-01 23:49 UTC (permalink / raw)
To: Jason Baron
Cc: jacob.jun.pan, linux-pm, rafael.j.wysocki, durgadoss.r,
srinivas.pandruvada
On Thursday, August 14, 2014 02:26:08 PM Jason Baron wrote:
> I've confirmed that monitoring the package power usage as well as setting power
> limits appear to be working as expected. Supports the package and dram domains.
>
> Tested aginst cpu:
>
> Intel(R) Xeon(R) CPU E5-2650 v3 @ 2.30GHz
>
> Signed-off-by: Jason Baron <jbaron@akamai.com>
Has this been ACKed by Jacob?
> ---
> 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 b1cda6f..a362dcc 100644
> --- a/drivers/powercap/intel_rapl.c
> +++ b/drivers/powercap/intel_rapl.c
> @@ -953,6 +953,7 @@ static const struct x86_cpu_id rapl_ids[] = {
> { X86_VENDOR_INTEL, 6, 0x3a},/* Ivy Bridge */
> { X86_VENDOR_INTEL, 6, 0x3c},/* Haswell */
> { X86_VENDOR_INTEL, 6, 0x3d},/* Broadwell */
> + { X86_VENDOR_INTEL, 6, 0x3f},/* Haswell */
> { X86_VENDOR_INTEL, 6, 0x45},/* Haswell ULT */
> /* TODO: Add more CPU IDs after testing */
> {}
>
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] powercap/rapl: add support for cpu model 0x3f
2014-09-01 23:49 ` Rafael J. Wysocki
@ 2014-09-02 14:31 ` Jason Baron
2014-09-02 17:41 ` Jacob Pan
0 siblings, 1 reply; 4+ messages in thread
From: Jason Baron @ 2014-09-02 14:31 UTC (permalink / raw)
To: Rafael J. Wysocki, jacob.jun.pan@linux.intel.com
Cc: linux-pm@vger.kernel.org, rafael.j.wysocki@intel.com,
durgadoss.r@intel.com, srinivas.pandruvada@linux.intel.com
Hi Rafael,
I think Jacob is ok with it:
https://lkml.org/lkml/2014/8/13/702
but he didn't give it a formal ack.
Thanks,
-Jason
On 09/01/2014 07:49 PM, Rafael J. Wysocki wrote:
> On Thursday, August 14, 2014 02:26:08 PM Jason Baron wrote:
>> I've confirmed that monitoring the package power usage as well as setting power
>> limits appear to be working as expected. Supports the package and dram domains.
>>
>> Tested aginst cpu:
>>
>> Intel(R) Xeon(R) CPU E5-2650 v3 @ 2.30GHz
>>
>> Signed-off-by: Jason Baron <jbaron@akamai.com>
>
> Has this been ACKed by Jacob?
>
>> ---
>> 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 b1cda6f..a362dcc 100644
>> --- a/drivers/powercap/intel_rapl.c
>> +++ b/drivers/powercap/intel_rapl.c
>> @@ -953,6 +953,7 @@ static const struct x86_cpu_id rapl_ids[] = {
>> { X86_VENDOR_INTEL, 6, 0x3a},/* Ivy Bridge */
>> { X86_VENDOR_INTEL, 6, 0x3c},/* Haswell */
>> { X86_VENDOR_INTEL, 6, 0x3d},/* Broadwell */
>> + { X86_VENDOR_INTEL, 6, 0x3f},/* Haswell */
>> { X86_VENDOR_INTEL, 6, 0x45},/* Haswell ULT */
>> /* TODO: Add more CPU IDs after testing */
>> {}
>>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] powercap/rapl: add support for cpu model 0x3f
2014-09-02 14:31 ` Jason Baron
@ 2014-09-02 17:41 ` Jacob Pan
0 siblings, 0 replies; 4+ messages in thread
From: Jacob Pan @ 2014-09-02 17:41 UTC (permalink / raw)
To: Jason Baron
Cc: Rafael J. Wysocki, linux-pm@vger.kernel.org,
rafael.j.wysocki@intel.com, durgadoss.r@intel.com,
srinivas.pandruvada@linux.intel.com
On Tue, 02 Sep 2014 10:31:53 -0400
Jason Baron <jbaron@akamai.com> wrote:
> Hi Rafael,
>
> I think Jacob is ok with it:
>
> https://lkml.org/lkml/2014/8/13/702
>
> but he didn't give it a formal ack.
>
Acked-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
> Thanks,
>
> -Jason
>
> On 09/01/2014 07:49 PM, Rafael J. Wysocki wrote:
> > On Thursday, August 14, 2014 02:26:08 PM Jason Baron wrote:
> >> I've confirmed that monitoring the package power usage as well as
> >> setting power limits appear to be working as expected. Supports
> >> the package and dram domains.
> >>
> >> Tested aginst cpu:
> >>
> >> Intel(R) Xeon(R) CPU E5-2650 v3 @ 2.30GHz
> >>
> >> Signed-off-by: Jason Baron <jbaron@akamai.com>
> >
> > Has this been ACKed by Jacob?
> >
> >> ---
> >> 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 b1cda6f..a362dcc 100644
> >> --- a/drivers/powercap/intel_rapl.c
> >> +++ b/drivers/powercap/intel_rapl.c
> >> @@ -953,6 +953,7 @@ static const struct x86_cpu_id rapl_ids[] = {
> >> { X86_VENDOR_INTEL, 6, 0x3a},/* Ivy Bridge */
> >> { X86_VENDOR_INTEL, 6, 0x3c},/* Haswell */
> >> { X86_VENDOR_INTEL, 6, 0x3d},/* Broadwell */
> >> + { X86_VENDOR_INTEL, 6, 0x3f},/* Haswell */
> >> { X86_VENDOR_INTEL, 6, 0x45},/* Haswell ULT */
> >> /* TODO: Add more CPU IDs after testing */
> >> {}
> >>
> >
>
[Jacob Pan]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-09-02 17:45 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-08-14 14:26 [PATCH] powercap/rapl: add support for cpu model 0x3f Jason Baron
2014-09-01 23:49 ` Rafael J. Wysocki
2014-09-02 14:31 ` Jason Baron
2014-09-02 17:41 ` Jacob Pan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).