Re: [Resend][PATCH] cpufreq: conservative: Decrease frequency faster when the timer deferred
From: Viresh Kumar
Date: Tue Nov 08 2016 - 03:32:34 EST
On 8 November 2016 at 12:49, Stratos Karafotis <stratosk@xxxxxxxxxxxx> wrote:
> I think we shouldn't. That's why the patch first decreases the frequency
> by n freq steps (where n the number of deferred periods).
> Then the normal processing takes place.
The problem that I see is that the new algorithm will reduce the
frequency even if we are
on a ramp up phase.
For example consider this case:
- We have a special load running, that runs in bursts. i.e. runs for
some time, lets the CPU idle
then and then again runs.
- To run the load properly, we need to ramp up the frequency
- But the new algorithm can make the frequency stagnant in this case.
i.e. because of the idle
period you may want to decrease the frequency by delta A and then the
regular algorithm may
want to increase it by same delta A.
That's why I was asking to adopt this only in the ramp down path.
--
viresh