summaryrefslogtreecommitdiffstats
path: root/tools/power/x86/turbostat
diff options
context:
space:
mode:
authorCalvin Walton <calvin.walton@kepstin.ca>2018-07-27 07:50:53 -0400
committerLen Brown <len.brown@intel.com>2018-07-27 12:54:19 -0400
commit5aa3d1a20a233d4a5f1ec3d62da3f19d9afea682 (patch)
tree1d88b4ecc77fc26eadf6966c901b03b995c6bff0 /tools/power/x86/turbostat
parent2ffbb22406079fec2c3a6ad6ee1dc99fede740ac (diff)
downloadtalos-op-linux-5aa3d1a20a233d4a5f1ec3d62da3f19d9afea682.tar.gz
talos-op-linux-5aa3d1a20a233d4a5f1ec3d62da3f19d9afea682.zip
tools/power turbostat: Read extended processor family from CPUID
This fixes the reported family on modern AMD processors (e.g. Ryzen, which is family 0x17). Previously these processors all showed up as family 0xf. See the document https://support.amd.com/TechDocs/56255_OSRR.pdf section CPUID_Fn00000001_EAX for how to calculate the family from the BaseFamily and ExtFamily values. This matches the code in arch/x86/lib/cpu.c Signed-off-by: Calvin Walton <calvin.walton@kepstin.ca> Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'tools/power/x86/turbostat')
-rw-r--r--tools/power/x86/turbostat/turbostat.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c
index 2b0135599f37..6c8effebf7c5 100644
--- a/tools/power/x86/turbostat/turbostat.c
+++ b/tools/power/x86/turbostat/turbostat.c
@@ -4457,7 +4457,9 @@ void process_cpuid()
family = (fms >> 8) & 0xf;
model = (fms >> 4) & 0xf;
stepping = fms & 0xf;
- if (family == 6 || family == 0xf)
+ if (family == 0xf)
+ family += (fms >> 20) & 0xff;
+ if (family >= 6)
model += ((fms >> 16) & 0xf) << 4;
if (!quiet) {
OpenPOWER on IntegriCloud