mbox series

[0/3] powercap: Enable RAPL for AMD Fam17h

Message ID 20200729105206.2991064-1-victording@google.com (mailing list archive)
Headers show
Series powercap: Enable RAPL for AMD Fam17h | expand

Message

Victor Ding July 29, 2020, 10:52 a.m. UTC 
This patch series adds support for AMD Fam17h RAPL counters. As per
AMD PPR, Fam17h support RAPL counters to monitor power usage. The RAPL
counter operates as with Intel RAPL. Therefore, it is beneficial to
re-use existing framework for Intel, especially to allow existing tools
to seamlessly run on AMD.

From the user's point view, this series enables the following two sysfs
entry on AMD Fam17h:
  /sys/class/powercap/intel-rapl/intel-rapl:0/energy_uj
  /sys/class/powercap/intel-rapl/intel-rapl:0/intel-rapl:0:0/energy_uj


Victor Ding (3):
  x86/msr-index: sort AMD RAPL MSRs by address
  powercap/intel_rapl_msr: Convert rapl_msr_priv into pointer
  powercap: Add AMD Fam17h RAPL support

 arch/x86/include/asm/msr-index.h     |  3 +-
 drivers/powercap/intel_rapl_common.c |  2 +
 drivers/powercap/intel_rapl_msr.c    | 58 +++++++++++++++++++++-------
 3 files changed, 47 insertions(+), 16 deletions(-)

Comments

Rafael J. Wysocki Sept. 21, 2020, 3:51 p.m. UTC | #1 
On Wed, Jul 29, 2020 at 12:52 PM Victor Ding <victording@google.com> wrote:
>
> This patch series adds support for AMD Fam17h RAPL counters. As per
> AMD PPR, Fam17h support RAPL counters to monitor power usage. The RAPL
> counter operates as with Intel RAPL. Therefore, it is beneficial to
> re-use existing framework for Intel, especially to allow existing tools
> to seamlessly run on AMD.
>
> From the user's point view, this series enables the following two sysfs
> entry on AMD Fam17h:
>   /sys/class/powercap/intel-rapl/intel-rapl:0/energy_uj
>   /sys/class/powercap/intel-rapl/intel-rapl:0/intel-rapl:0:0/energy_uj
>
>
> Victor Ding (3):
>   x86/msr-index: sort AMD RAPL MSRs by address

I haven't received this patch.

Any chance to resend the entire series with a CC to linux-pm?

>   powercap/intel_rapl_msr: Convert rapl_msr_priv into pointer
>   powercap: Add AMD Fam17h RAPL support
>
>  arch/x86/include/asm/msr-index.h     |  3 +-
>  drivers/powercap/intel_rapl_common.c |  2 +
>  drivers/powercap/intel_rapl_msr.c    | 58 +++++++++++++++++++++-------
>  3 files changed, 47 insertions(+), 16 deletions(-)
>
> --
> 2.28.0.rc0.142.g3c755180ce-goog
>
Daniel Lezcano Sept. 21, 2020, 3:53 p.m. UTC | #2 
On 21/09/2020 17:51, Rafael J. Wysocki wrote:
> On Wed, Jul 29, 2020 at 12:52 PM Victor Ding <victording@google.com> wrote:
>>
>> This patch series adds support for AMD Fam17h RAPL counters. As per
>> AMD PPR, Fam17h support RAPL counters to monitor power usage. The RAPL
>> counter operates as with Intel RAPL. Therefore, it is beneficial to
>> re-use existing framework for Intel, especially to allow existing tools
>> to seamlessly run on AMD.
>>
>> From the user's point view, this series enables the following two sysfs
>> entry on AMD Fam17h:
>>   /sys/class/powercap/intel-rapl/intel-rapl:0/energy_uj
>>   /sys/class/powercap/intel-rapl/intel-rapl:0/intel-rapl:0:0/energy_uj
>>
>>
>> Victor Ding (3):
>>   x86/msr-index: sort AMD RAPL MSRs by address
> 
> I haven't received this patch.
> 
> Any chance to resend the entire series with a CC to linux-pm?

Is it possible to add me in Cc too ?

Thanks