@@ -291,7 +291,7 @@ struct msr_sum_array {
/* The percpu MSR sum array.*/
struct msr_sum_array *per_cpu_msr_sum;
-int idx_to_offset(int idx)
+int idx_to_offset_intel(int idx)
{
int offset;
@@ -320,7 +320,7 @@ int idx_to_offset(int idx)
return offset;
}
-int offset_to_idx(int offset)
+int offset_to_idx_intel(int offset)
{
int idx;
@@ -349,7 +349,7 @@ int offset_to_idx(int offset)
return idx;
}
-int idx_valid(int idx)
+int idx_valid_intel(int idx)
{
switch (idx) {
case IDX_PKG_ENERGY:
@@ -368,6 +368,51 @@ int idx_valid(int idx)
return 0;
}
}
+
+int (*idx_to_offset)(int idx) = idx_to_offset_intel;
+int (*offset_to_idx)(int offset) = offset_to_idx_intel;
+int (*idx_valid)(int idx) = idx_valid_intel;
+
+int idx_to_offset_amd(int idx)
+{
+ int offset;
+
+ switch (idx) {
+ case IDX_PKG_ENERGY:
+ offset = MSR_PKG_ENERGY_STAT;
+ break;
+ default:
+ offset = -1;
+ }
+
+ return offset;
+}
+
+int offset_to_idx_amd(int offset)
+{
+ int idx;
+
+ switch (offset) {
+ case MSR_PKG_ENERGY_STAT:
+ idx = IDX_PKG_ENERGY;
+ break;
+ default:
+ idx = -1;
+ }
+
+ return idx;
+}
+
+int idx_valid_amd(int idx)
+{
+ switch (idx) {
+ case IDX_PKG_ENERGY:
+ return do_rapl & MSR_PKG_ENERGY_STAT;
+ default:
+ return 0;
+ }
+}
+
struct sys_counters {
unsigned int added_thread_counters;
unsigned int added_core_counters;
@@ -3249,7 +3294,7 @@ int get_msr_sum(int cpu, off_t offset, unsigned long long *msr)
return 1;
idx = offset_to_idx(offset);
- if (idx < 0)
+ if (idx == -1)
return idx;
/* get_msr_sum() = sum + (get_msr() - last) */
ret = get_msr(cpu, offset, &msr_cur);
@@ -3277,7 +3322,7 @@ static int update_msr_sum(struct thread_data *t, struct core_data *c, struct pkg
if (!idx_valid(i))
continue;
offset = idx_to_offset(i);
- if (offset < 0)
+ if (offset == -1)
continue;
ret = get_msr(cpu, offset, &msr_cur);
if (ret) {
@@ -5348,6 +5393,12 @@ void process_cpuid()
if (!quiet)
decode_misc_feature_control();
+ if (authentic_amd || hygon_genuine) {
+ idx_to_offset = idx_to_offset_amd;
+ offset_to_idx = offset_to_idx_amd;
+ idx_valid = idx_valid_amd;
+ }
+
return;
}
Turbostat fails to correctly collect and display RAPL summary information on Family 17h and 19h AMD processors. Running turbostat on these processors returns immediately. If turbostat is working correctly then RAPL summary data is displayed until the user provided command completes. If a command is not provided by the user then turbostat is designed to continuously display RAPL information until interrupted. The issue is due to offset_to_idx() and idx_to_offset() missing support for AMD MSR addresses/offsets. offset_to_idx()'s switch statement is missing cases for AMD MSRs and idx_to_offset() does not include a path to return AMD MSR(s) for any idx. The solution is add AMD MSR support to offset_to_idx() and idx_to_offset(). These functions are split-out and renamed along architecture vendor lines for supporting both AMD and Intel MSRs. Fixes: 9972d5d84d76 ("tools/power turbostat: Enable accumulate RAPL display") Signed-off-by: Terry Bowman <terry.bowman@amd.com> Cc: Len Brown <lenb@kernel.org> Cc: linux-pm@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- tools/power/x86/turbostat/turbostat.c | 61 ++++++++++++++++++++++++--- 1 file changed, 56 insertions(+), 5 deletions(-)