mbox series

[v3,0/4] powercap: Enable RAPL for AMD Fam17h and Fam19h

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

Message

Victor Ding Oct. 27, 2020, 7:23 a.m. UTC
This patch series adds support for AMD Fam17h RAPL counters. As per
AMD PPR, Fam17h and Fam19h 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 or Fam19h:
  /sys/class/powercap/intel-rapl/intel-rapl:0/energy_uj
  /sys/class/powercap/intel-rapl/intel-rapl:0/intel-rapl:0:0/energy_uj

Changes in v3:
By Victor Ding <victording@google.com>
 - Rebased to the latest code.
 - Created a new rapl_defaults for AMD CPUs.
 - Removed redundant setting to zeros.
 - Stopped using the fake power limit domain 1.

Changes in v2:
By Kim Phillips <kim.phillips@amd.com>
- Added the Fam19h patch to the end of the series
- Added my Acked-by
- Added Daniel Lezcano to Cc
- (linux-pm was already on Cc)
- (No code changes)

Kim Phillips (1):
  powercap: Add AMD Fam19h RAPL support

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 |  7 ++++
 drivers/powercap/intel_rapl_msr.c    | 51 ++++++++++++++++++++--------
 3 files changed, 45 insertions(+), 16 deletions(-)

Comments

Rafael J. Wysocki Nov. 10, 2020, 7:24 p.m. UTC | #1
On Tue, Oct 27, 2020 at 8:24 AM Victor Ding <victording@google.com> wrote:
>
> This patch series adds support for AMD Fam17h RAPL counters. As per
> AMD PPR, Fam17h and Fam19h 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 or Fam19h:
>   /sys/class/powercap/intel-rapl/intel-rapl:0/energy_uj
>   /sys/class/powercap/intel-rapl/intel-rapl:0/intel-rapl:0:0/energy_uj
>
> Changes in v3:
> By Victor Ding <victording@google.com>
>  - Rebased to the latest code.
>  - Created a new rapl_defaults for AMD CPUs.
>  - Removed redundant setting to zeros.
>  - Stopped using the fake power limit domain 1.
>
> Changes in v2:
> By Kim Phillips <kim.phillips@amd.com>
> - Added the Fam19h patch to the end of the series
> - Added my Acked-by
> - Added Daniel Lezcano to Cc
> - (linux-pm was already on Cc)
> - (No code changes)
>
> Kim Phillips (1):
>   powercap: Add AMD Fam19h RAPL support
>
> 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 |  7 ++++
>  drivers/powercap/intel_rapl_msr.c    | 51 ++++++++++++++++++++--------
>  3 files changed, 45 insertions(+), 16 deletions(-)
>
> --

All patches applied as 5.11 material (which some minor edits in the
changelogs), thanks!