Re: [PATCH] intel_idle: Add ICL support

From: Artem Bityutskiy
Date: Wed Aug 26 2020 - 09:16:26 EST


On Wed, 2020-08-26 at 15:03 +0200, Guilhem Lettron wrote:
> On Wed, 26 Aug 2020 at 14:43, Rafael J. Wysocki <rafael@xxxxxxxxxx> wrote:
> > On Wed, Aug 26, 2020 at 2:05 PM Guilhem Lettron <guilhem@xxxxxxxxxxx> wrote:
> > > Use the same C-states as SKL
> >
> > Why is this change needed?
>
> On my laptop, a Dell XPS 13 7390 2-in-1 with i7-1065G7, ACPI only
> report "C1_ACPI", "C2_ACPI" and "C3_ACPI".

Also, if you could runt turbostat - we could see which _actual_ HW C-
states are used on your system, which Package C-states are reached.

Just get a reasonably new turbostat (it is part of the kernel tree, you
can compile it yourself) and run it for few seconds (like 'turbostat
sleep 10'), get the output (will be a lot of it), and we can check what
is actually going on with regards to C-states.

Artem.