Re: [PATCH] intel_idle: Add ICL support

Artem Bityutskiy
2020-08-26T16:19:42Z

Indeed, when I compare them:

acpi_idle (without the patch):

CPU%c1 CPU%c6 CPU%c7 CoreTmp PkgTmp GFX%rc6 Pkg%pc2 Pkg%pc3 Pkg%pc6 Pkg%pc7 Pkg%pc8 Pkg%pc9 Pk%pc10 PkgWatt
29.48 0.00 60.71 58 58 97.96 16.96 0.00 0.00 0.00 0.00 0.00 0.00 6.08

intel_idle (with the patch):

CPU%c1 CPU%c6 CPU%c7 CoreTmp PkgTmp GFX%rc6 Pkg%pc2 Pkg%pc3 Pkg%pc6 Pkg%pc7 Pkg%pc8 Pkg%pc9 Pk%pc10 PkgWatt
56 56 96.64 300 68.29 48.58 0.00 0.00 0.00 0.00 0.00 0.00 7.38 0.00

With intel_idle we reach PC10, without it we only go as deep as PC2 - huge difference.

I really wonder why the BIOS does not expose deeper C-states... And if
it does not, is this for a reason? And how windows works then?

May be there is a BIOS update that fixes this problem? May be Windows
user get it quickly because stuff like this is often well-integrated in
Windows? Would you please check if there is newer BIOS?

Artem.