[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aEfwxUyu54Y1EXJI@jlelli-thinkpadt14gen4.remote.csb>
Date: Tue, 10 Jun 2025 10:45:57 +0200
From: Juri Lelli <juri.lelli@...hat.com>
To: Yuri Andriaccio <yurand2000@...il.com>
Cc: Ingo Molnar <mingo@...hat.com>, Peter Zijlstra <peterz@...radead.org>,
Vincent Guittot <vincent.guittot@...aro.org>,
Dietmar Eggemann <dietmar.eggemann@....com>,
Steven Rostedt <rostedt@...dmis.org>,
Ben Segall <bsegall@...gle.com>, Mel Gorman <mgorman@...e.de>,
Valentin Schneider <vschneid@...hat.com>,
linux-kernel@...r.kernel.org,
Luca Abeni <luca.abeni@...tannapisa.it>,
Yuri Andriaccio <yuri.andriaccio@...tannapisa.it>
Subject: Re: [RFC PATCH 5/9] sched/deadline: Hierarchical scheduling with DL
on top of RT
Hello,
On 05/06/25 09:14, Yuri Andriaccio wrote:
> From: luca abeni <luca.abeni@...tannapisa.it>
>
> Implement the hierarchical scheduling mechanism:
> - Enforce the runtime of RT tasks controlled by cgroups using the dl_server
> mechanism, based on the runtime and period (the deadline is set equal
> to the period) parameters.
> - Make sched_dl_entity store the cgroups' RT local runqueue, and provide a
> rt_rq for this runqueue.
> - Allow zeroing the runtime of a rt cgroups.
>
> Update dl_server code:
> - Make check for the return value of dl_server_apply_params on initialization
> of the fair server.
> - Initialize the dl_server in dl_server_start only if it is a fair server, but
> do not initialize other types of servers, regardless of its period value.
> - Make inc_dl_task and dec_dl_task increase/decrease by one the value of active
> tasks if a fair_server starts/stops, while ignore rt-cgroups dl_servers in
> the accounting.
>
> Co-developed-by: Alessio Balsini <a.balsini@...up.it>
> Signed-off-by: Alessio Balsini <a.balsini@...up.it>
> Co-developed-by: Andrea Parri <parri.andrea@...il.com>
> Signed-off-by: Andrea Parri <parri.andrea@...il.com>
> Co-developed-by: Yuri Andriaccio <yurand2000@...il.com>
> Signed-off-by: Yuri Andriaccio <yurand2000@...il.com>
> Signed-off-by: luca abeni <luca.abeni@...tannapisa.it>
> ---
> kernel/sched/autogroup.c | 4 +-
> kernel/sched/core.c | 15 +-
> kernel/sched/deadline.c | 145 ++++++++++++--
> kernel/sched/rt.c | 415 ++++++++++++++++++++++++++-------------
> kernel/sched/sched.h | 59 +++++-
> kernel/sched/syscalls.c | 4 +-
> 6 files changed, 471 insertions(+), 171 deletions(-)
>From changelog (and confirmed by changed lines), this patch might be
doing too many things at once and so it might be harder to review. Do
you think it can be split in smaller chunks?
Thanks,
Juri
Powered by blists - more mailing lists