Messages in this thread | | | Date | Thu, 16 Nov 2023 15:34:50 +0100 | From | Peter Zijlstra <> | Subject | Re: [PATCH v3 0/2] Rework interface between scheduler and schedutil governor |
| |
On Mon, Nov 06, 2023 at 04:05:40PM +0100, Rafael J. Wysocki wrote: > 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.
Sure, I'll go queue it. Thanks!
| |