intel / S0ixSelftestTool

S0ix Selftest Tool is designed to do the initial debugging for the S2idle path CPU Package C-state and S0ix failures in Linux OS for Intel® Client platforms, it also supports the basic runtime PC10 status check.
GNU General Public License v2.0
135 stars 18 forks source link

turbostat misses some fields #27

Open paulatz opened 2 months ago

paulatz commented 2 months ago

Hello on my Ubuntu 22.04 system the "turbostat" program does not output all the requested columns, which breaks the parsing of its output at line 1154. You can see in this sample that `GFX%rc6 and Pkg%pc7 are missing:

$ TURBO_COLUMNS="CPU%c1,CPU%c6,CPU%c7,GFX%rc6,Pkg%pc2,Pkg%pc3,Pkg%pc6,Pkg%pc7,Pkg%pc8,Pkg%pc9,Pk%pc10,SYS%LPI"
$ sudo ./turbostat --quiet --show "$TURBO_COLUMNS"
 CPU%c1 CPU%c6  CPU%c7  Pkg%pc2 Pkg%pc3 Pkg%pc6 Pkg%pc8 Pk%pc10 SYS%LPI
3.70    55.25   20.25   0.07    0.02    0.00    0.00    0.00    0.00
7.31    29.69   34.46   0.07    0.02    0.00    0.00    0.00    0.00
7.31
3.18    19.23   53.53
3.18
4.18    15.46   48.20
4.18
3.12    19.95   44.34
3.12
2.87    16.67   56.68
2.87
4.83    20.75   46.25
4.83
2.55    80.50   0.00
2.39    82.21   0.00
1.13    85.01   0.00
1.07    86.04   0.00
5.00    78.17   0.00
3.24    81.14   0.00
3.49    80.29   0.00
4.08    78.42   0.00

I'm not sure which is the best way to work around this issue

driesbenoit commented 2 months ago

My pull request " fix incorrect extraction of turbostat output" #26 fixes this issue. Note that this issue is also mentioned in issue #23.

qwang59 commented 2 months ago

GFX%rc6 is not displayed because the graphics driver is not loaded. GFX%rc6 becomes available when the graphics sysfs is created.

qwang59 commented 2 months ago

Thanks for your patch, I merged. And I also need to modify it later because we cannot ignore GFX%rc6, which is very important.