Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1591254

Re: [PATCH 0/6] cpufreq: schedutil: fixes for flags updates

Show all headers | View raw

On 2 March 2017 at 16:45, Patrick Bellasi <patrick.bellasi@arm.com> wrote:
> The current version of schedutil has some issues related to the management
> of update flags used by systems with frequency domains spawning multiple CPUs.
>
> Each time a CPU utilisation update is issued by the scheduler a set of flags
> are configured to define (mainly) which class is asking for a utilisation
> update. These flags are then used by the frequency selection policy to
> identify the OPP to choose.
>
> In the current implementation, CPU flags are overridden each time the
> scheduler calls schedutil for an update. Such a behaviour produces issues
> in these scenarios, where we assume CPU1 and CPU2 share the same frequency
> domain:
> a) a RT task which executed on CPU1 can keep the domain at an high frequency
>    for a long period of time, even if there are no longer RT tasks on
>    CPUs in that domain

Normally this is dropped after a tick.
Nevertheless, there is an issue if the freq_update_delay_ns is shorter
than a tick because of sugov RT thread

> b) a FAIR task co-scheduled in the same CPU of a RT task can override the
>    flags configured by the RT task and potentially this can cause an
>    unwanted frequency drop
>
> These misbehaviours have been verified using a set of simple rt-app based
> synthetic workloads, running on a ARM's Juno board, and results are
> available in this Notebook [1].
>
> This series proposes a set of fixes for the aforementioned issues as well
> as a small improvement to speedup the selection of the maximum frequency
> when RT tasks enter a CPU.
>
> This series is based on top of today's tip/sched/core and is public available
> from this repository:
>
>   git://www.linux-arm.com/linux-pb eas/schedutil/flags_fixes
>
> Cheers Patrick
>
> [1] https://gist.github.com/d6a21b459a18091b2b058668a550010d
>
> Patrick Bellasi (6):
>   cpufreq: schedutil: reset sg_cpus's flags at IDLE enter
>   cpufreq: schedutil: ignore the sugov kthread for frequencies
>     selections
>   cpufreq: schedutil: ensure max frequency while running RT/DL tasks
>   cpufreq: schedutil: relax rate-limiting while running RT/DL tasks
>   cpufreq: schedutil: avoid utilisation update when not necessary
>   sched/rt: fast switch to maximum frequency when RT tasks are scheduled
>
>  include/linux/sched.h            |  1 +
>  kernel/sched/cpufreq_schedutil.c | 59 ++++++++++++++++++++++++++++++++++------
>  kernel/sched/idle_task.c         |  4 +++
>  kernel/sched/rt.c                | 15 ++++++++--
>  4 files changed, 68 insertions(+), 11 deletions(-)
>
> --
> 2.7.4
>

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/6] cpufreq: schedutil: fixes for flags updates Patrick Bellasi <patrick.bellasi@arm.com> - 2017-03-02 16:50 +0100
  [PATCH 3/6] cpufreq: schedutil: ensure max frequency while running RT/DL tasks Patrick Bellasi <patrick.bellasi@arm.com> - 2017-03-02 16:50 +0100
    Re: [PATCH 3/6] cpufreq: schedutil: ensure max frequency while  running RT/DL tasks Viresh Kumar <viresh.kumar@linaro.org> - 2017-03-03 10:00 +0100
      Re: [PATCH 3/6] cpufreq: schedutil: ensure max frequency while  running RT/DL tasks Patrick Bellasi <patrick.bellasi@arm.com> - 2017-03-03 13:50 +0100
  [PATCH 5/6] cpufreq: schedutil: avoid utilisation update when not necessary Patrick Bellasi <patrick.bellasi@arm.com> - 2017-03-02 17:00 +0100
  [PATCH 2/6] cpufreq: schedutil: ignore the sugov kthread for frequencies selections Patrick Bellasi <patrick.bellasi@arm.com> - 2017-03-02 17:00 +0100
    Re: [PATCH 2/6] cpufreq: schedutil: ignore the sugov kthread for  frequencies selections Viresh Kumar <viresh.kumar@linaro.org> - 2017-03-03 07:30 +0100
      Re: [PATCH 2/6] cpufreq: schedutil: ignore the sugov kthread for  frequencies selections Patrick Bellasi <patrick.bellasi@arm.com> - 2017-03-03 14:20 +0100
        Re: [PATCH 2/6] cpufreq: schedutil: ignore the sugov kthread for  frequencies selections Viresh Kumar <viresh.kumar@linaro.org> - 2017-03-06 06:20 +0100
    Re: [PATCH 2/6] cpufreq: schedutil: ignore the sugov kthread for  frequencies selections Steven Rostedt <rostedt@goodmis.org> - 2017-03-06 15:40 +0100
  [PATCH 6/6] sched/rt: fast switch to maximum frequency when RT tasks are scheduled Patrick Bellasi <patrick.bellasi@arm.com> - 2017-03-02 17:00 +0100
  Re: [PATCH 0/6] cpufreq: schedutil: fixes for flags updates Vincent Guittot <vincent.guittot@linaro.org> - 2017-03-02 17:20 +0100
    Re: [PATCH 0/6] cpufreq: schedutil: fixes for flags updates Patrick Bellasi <patrick.bellasi@arm.com> - 2017-03-02 18:40 +0100

csiph-web