Messages in this thread | | | From | "Rafael J. Wysocki" <> | Date | Mon, 6 Nov 2023 16:05:40 +0100 | Subject | Re: [PATCH v3 0/2] Rework interface between scheduler and schedutil governor |
| |
On Fri, Nov 3, 2023 at 2:18 PM Vincent Guittot <vincent.guittot@linaro.org> wrote: > > Following the discussion with Qais [1] about how to handle uclamp > requirements and after syncing with him, we agreed that I should move > forward on the patchset to rework the interface between scheduler and > schedutil governor to provide more information to the latter. Scheduler > (and EAS in particular) doesn't need anymore to guess estimate which > headroom the governor wants to apply and will directly ask for the target > freq. Then the governor directly gets the actual utilization and new > minimum and maximum boundaries to select this target frequency and > doesn't have to deal anymore with scheduler internals like uclamp when > including iowait boost. > > [1] https://lore.kernel.org/lkml/CAKfTPtA5JqNCauG-rP3wGfq+p8EEVx9Tvwj6ksM3SYCwRmfCTg@mail.gmail.com/ > > Changes since v2: > - remove useless target variable > > Changes since v1: > - fix a bug (always set max even when returning early) > - fix typos > > Vincent Guittot (2): > sched/schedutil: Rework performance estimation > sched/schedutil: Rework iowait boost > > include/linux/energy_model.h | 1 - > kernel/sched/core.c | 82 ++++++++++++------------------- > kernel/sched/cpufreq_schedutil.c | 69 ++++++++++++++++---------- > kernel/sched/fair.c | 22 +++++++-- > kernel/sched/sched.h | 84 +++----------------------------- > 5 files changed, 100 insertions(+), 158 deletions(-) > > --
For the schedutil changes in the series:
Acked-by: Rafael J. Wysocki <rafael@kernel.org>
and I'm assuming this series to be targeted at sched.
| |