Re: [PATCH v2] cpufreq: schedutil: rate limits for SCHED_DEADLINE

[Rafael J. Wysocki]

On Wed, Feb 28, 2018 at 12:06 PM, Claudio Scordino
<claudio@xxxxxxxxxxxxxxx> wrote:
> When the SCHED_DEADLINE scheduling class increases the CPU utilization,
> we should not wait for the rate limit, otherwise we may miss some
> deadline.
>
> Tests using rt-app on Exynos5422 with up to 10 SCHED_DEADLINE tasks have
> shown reductions of even 10% of deadline misses with a negligible
> increase of energy consumption (measured through Baylibre Cape).
>
> Signed-off-by: Claudio Scordino <claudio@xxxxxxxxxxxxxxx>
> CC: Ingo Molnar <mingo@xxxxxxxxxx>
> CC: Patrick Bellasi <patrick.bellasi@xxxxxxx>
> CC: Dietmar Eggemann <dietmar.eggemann@xxxxxxx>
> CC: Morten Rasmussen <morten.rasmussen@xxxxxxx>
> CC: Juri Lelli <juri.lelli@xxxxxxxxxx>
> CC: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
> CC: Vincent Guittot <vincent.guittot@xxxxxxxxxx>
> CC: Todd Kjos <tkjos@xxxxxxxxxxx>
> CC: Joel Fernandes <joelaf@xxxxxxxxxx>
> CC: linux-pm@xxxxxxxxxxxxxxx
> CC: linux-kernel@xxxxxxxxxxxxxxx
> ---
> Changes from v1:
>  - Logic moved from sugov_should_update_freq() to
>    sugov_update_single()/_shared() to not duplicate data structures
>  - Rate limit not ignored in case of "fast switch"

I'm not sure about this last bit.

IMO you can set sg_policy->need_freq_update even in the "fast switch"
case to start with and special case it in the future if that turns out
to be problematic.  That is, unless you have data indicating that it
already is problematic, of course. :-)

> ---
>  kernel/sched/cpufreq_schedutil.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
>
> diff --git a/kernel/sched/cpufreq_schedutil.c b/kernel/sched/cpufreq_schedutil.c
> index 7936f54..ca6ce72 100644
> --- a/kernel/sched/cpufreq_schedutil.c
> +++ b/kernel/sched/cpufreq_schedutil.c
> @@ -273,6 +273,14 @@ static void sugov_update_single(struct update_util_data *hook, u64 time,
>         sugov_set_iowait_boost(sg_cpu, time);
>         sg_cpu->last_update = time;
>
> +       /*
> +        * Make sugov_should_update_freq() ignore the rate limit when DL
> +        * has increased the utilization.
> +        */
> +       if ((cpu_util_dl(cpu_rq(sg_cpu->cpu)) > sg_cpu->util_dl) &&
> +                       !(sg_policy->policy->fast_switch_enabled))
> +               sg_policy->need_freq_update = true;
> +
>         if (!sugov_should_update_freq(sg_policy, time))
>                 return;
>
> @@ -354,6 +362,14 @@ static void sugov_update_shared(struct update_util_data *hook, u64 time,
>
>         raw_spin_lock(&sg_policy->update_lock);
>
> +       /*
> +        * Make sugov_should_update_freq() ignore the rate limit when DL
> +        * has increased the utilization.
> +        */
> +       if ((cpu_util_dl(cpu_rq(sg_cpu->cpu)) > sg_cpu->util_dl) &&
> +                       !(sg_policy->policy->fast_switch_enabled))
> +               sg_policy->need_freq_update = true;
> +
>         sugov_get_util(sg_cpu);
>         sg_cpu->flags = flags;
>
> --
> 2.7.4
>