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

From: "Rafael J. Wysocki" <rafael@kernel.org>
To: Victor Ding <victording@google.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Daniel Lezcano <daniel.lezcano@linaro.org>,
	Kim Phillips <kim.phillips@amd.com>,
	Zhang Rui <rui.zhang@intel.com>,
	Linux PM <linux-pm@vger.kernel.org>,
	Borislav Petkov <bp@alien8.de>, "H. Peter Anvin" <hpa@zytor.com>,
	Ingo Molnar <mingo@redhat.com>, Joerg Roedel <jroedel@suse.de>,
	Kan Liang <kan.liang@linux.intel.com>,
	Pawan Gupta <pawan.kumar.gupta@linux.intel.com>,
	"Peter Zijlstra (Intel)" <peterz@infradead.org>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	Sean Christopherson <sean.j.christopherson@intel.com>,
	Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	Tony Luck <tony.luck@intel.com>,
	Vineela Tummalapalli <vineela.tummalapalli@intel.com>,
	"the arch/x86 maintainers" <x86@kernel.org>
Subject: Re: [PATCH v3 0/4] powercap: Enable RAPL for AMD Fam17h and Fam19h
Date: Tue, 10 Nov 2020 20:24:34 +0100	[thread overview]
Message-ID: <CAJZ5v0h8m-rUxqo5hVGDrm+Rm_0nEwN1yQCeKQVE5AbbR2GXFA@mail.gmail.com> (raw)
In-Reply-To: <20201027072358.13725-1-victording@google.com>

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!