Re: [PATCH v1 0/4] cpuidle: governors: teo: Assorted improvements

From: Rafael J. Wysocki

Date: Thu Nov 13 2025 - 10:25:15 EST


On Thu, Nov 13, 2025 at 4:21 PM Christian Loehle
<christian.loehle@xxxxxxx> wrote:
>
> On 11/12/25 16:21, Rafael J. Wysocki wrote:
> > Hi,
> >
> > This is a bunch of teo cpuidle governor improvements, some of which are related
> > to a bug report discussed recently:
> >
> > https://lore.kernel.org/linux-pm/CAEmPcwsNMNnNXuxgvHTQ93Mx-q3Oz9U57THQsU_qdcCx1m4w5g@xxxxxxxxxxxxxx/
> >
> > The first patch fixes a bug that may cause an overly deep idle state
> > to be selected when the scheduler tick has been already stopped.
> >
> > Patch [2/4] removes an unnecessary function argument.
> >
> > Patch [3/4] makes teo_update() to use s64 as the data type for its local
> > variables more consistently.
> >
> > The last patch reworks the governor's decay implementation to also decay
> > metric values lower than 8.
> >
>
> Tested-by: Christian Loehle <christian.loehle@xxxxxxx>

Thank you!

> Test results below, although there really isn't anything interesting in there.
> teo-1 to teo-4 (patches 1 to 4 respectively are essentially indistinguishable from
> teo-m = mainline)

Good, that's how it should be.