Skip to content
Commit 9d6c0e58 authored by He Rongguang's avatar He Rongguang Committed by Shuah Khan
Browse files

cpupower: fix TSC MHz calculation



Commit 'cpupower: Make TSC read per CPU for Mperf monitor' (c2adb187)
changes TSC counter reads per cpu, but left time diff global (from start
of all cpus to end of all cpus), thus diff(time) is too large for a
cpu's tsc counting, resulting in far less than acutal TSC_Mhz and thus
`cpupower monitor` showing far less than actual cpu realtime frequency.

/proc/cpuinfo shows frequency:
cat /proc/cpuinfo | egrep -e 'processor' -e 'MHz'
...
processor : 171
cpu MHz   : 4108.498
...

before fix (System 100% busy):
    | Mperf              || Idle_Stats
 CPU| C0   | Cx   | Freq  || POLL | C1   | C2
 171|  0.77| 99.23|  2279||  0.00|  0.00|  0.00

after fix (System 100% busy):
    | Mperf              || Idle_Stats
 CPU| C0   | Cx   | Freq  || POLL | C1   | C2
 171|  0.46| 99.54|  4095||  0.00|  0.00|  0.00

Fixes: c2adb187 ("cpupower: Make TSC read per CPU for Mperf monitor")
Signed-off-by: default avatarHe Rongguang <herongguang@linux.alibaba.com>
Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
parent 46fd8c70
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment