cpupower: Fix AMD Family 0x17 msr_pstate size
authorPrarit Bhargava <prarit@redhat.com>
Mon, 8 Oct 2018 15:06:18 +0000 (11:06 -0400)
committerShuah Khan (Samsung OSG) <shuah@kernel.org>
Mon, 8 Oct 2018 15:19:05 +0000 (09:19 -0600)
The msr_pstate data is only 63 bits long and should be 64 bits.

Add in the missing bit from res1 for AMD Family 0x17.

Reference: https://www.amd.com/system/files/TechDocs/54945_PPR_Family_17h_Models_00h-0Fh.pdf, page 138.

Signed-off-by: Prarit Bhargava <prarit@redhat.com>
Cc: Shuah Khan <shuah@kernel.org>
Cc: Stafford Horne <shorne@gmail.com>
Signed-off-by: Shuah Khan (Samsung OSG) <shuah@kernel.org>
tools/power/cpupower/utils/helpers/amd.c

index bb41cdd0df6bfe527cb956c531250cf5866d5ea9..eed31b27909b9aab333d3371a775f55bab22caa4 100644 (file)
@@ -33,7 +33,7 @@ union msr_pstate {
                unsigned vid:8;
                unsigned iddval:8;
                unsigned idddiv:2;
-               unsigned res1:30;
+               unsigned res1:31;
                unsigned en:1;
        } fam17h_bits;
        unsigned long long val;