Re: [PATCH][v2] sched: cpufreq: Fix long idle judgement logic in load calculation

From: Rafael J. Wysocki
Date: Tue Jun 12 2018 - 11:07:07 EST


On Friday, June 8, 2018 6:04:13 AM CEST Viresh Kumar wrote:
> On 08-06-18, 09:07, Chen Yu wrote:
> > According to current code implementation, detecting the long
> > idle period is done by checking if the interval between two
> > adjacent utilization update handers is long enough. Although
> > this mechanism can detect if the idle period is long enough
> > (no utilization hooks invoked during idle period), it might
> > not contain a corner case: if the task has occupied the cpu
> > for too long which causes no context switch during that
> > period, then no utilization handler will be launched until this
> > high prio task is switched out. As a result, the idle_periods
> > field might be calculated incorrectly because it regards the
> > 100% load as 0% and makes the conservative governor who uses
> > this field confusing.
> >
> > Change the judgement to compare the idle_time with sampling_rate
> > directly.
> >
> > Reported-by: Artem S. Tashkinov <t.artem@xxxxxxxxxxxx>
> > Cc: Artem S Tashkinov <t.artem@xxxxxxxxxxxx>
> > Cc: "Rafael J. Wysocki" <rjw@xxxxxxxxxxxxx>
> > Cc: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
> > Cc: linux-pm@xxxxxxxxxxxxxxx
> > Signed-off-by: Chen Yu <yu.c.chen@xxxxxxxxx>
> > ---
> > v2: Per Viresh's suggestion, ignore idle_time longer than 30mins and
> > simplify the code.
> > ---
> > drivers/cpufreq/cpufreq_governor.c | 12 +++++-------
> > 1 file changed, 5 insertions(+), 7 deletions(-)
>
> Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

Patch applied, thanks!