From: Tejun Heo <tj@kernel.org>
To: luca abeni <luca.abeni@santannapisa.it>
Cc: Yuri Andriaccio <yuri.andriaccio@santannapisa.it>,
Peter Zijlstra <peterz@infradead.org>,
Yuri Andriaccio <yurand2000@gmail.com>,
Ingo Molnar <mingo@redhat.com>,
Juri Lelli <juri.lelli@redhat.com>,
Vincent Guittot <vincent.guittot@linaro.org>,
Dietmar Eggemann <dietmar.eggemann@arm.com>,
Steven Rostedt <rostedt@goodmis.org>,
Ben Segall <bsegall@google.com>, Mel Gorman <mgorman@suse.de>,
Valentin Schneider <vschneid@redhat.com>,
linux-kernel@vger.kernel.org, hannes@cmpxchg.org,
mkoutny@suse.com, cgroups@vger.kernel.org
Subject: Re: [BULK] Re: [RFC PATCH v5 20/29] sched/deadline: Allow deeper hierarchies of RT cgroups
Date: Wed, 20 May 2026 08:49:01 -1000 [thread overview]
Message-ID: <ag4CHYXcwOXfRCBZ@slm.duckdns.org> (raw)
In-Reply-To: <20260519230258.0342358c@nowhere>
On Tue, May 19, 2026 at 11:02:58PM +0200, luca abeni wrote:
> On Mon, 18 May 2026 08:47:37 -1000
> Tejun Heo <tj@kernel.org> wrote:
> [...]
> > I wonder whether it can be generalized more. Would something like the
> > following work? I'm going to ignore period for the sake of simplicity
> > as it doesn't seem to affect admission decisions.
> >
> > - There is no root cgroup.rt.max in line with other control knobs.
>
> Well, the reason we had "rt.{runtime,period}_us" (now "rt.max") in the
> root cgroup is that RT cgroups are scheduled by dl entities (one dl
> entity per cpu), and these dl entities must be accounted for in the
> SCHED_DEADLINE admission test... The easiest way to do this is to
> reserve a fixed fraction of the CPU time to RT cgroups, leaving the
> remaining fraction to SCHED_DEADLINE tasks. And we used rt.max to
> configure the fraction of CPU time reserved for RT cgroups; do you have
> suggestions about alternative interfaces for this configuration?
I see. That makes sense. I think rt.max in the root fits the bill for that
purpose. This is different from other resources in that there's sub-division
at the root level but I think that's an inherent property here.
> > - Setting a budget is subject to admission control in both directions
> > - the budget source (the nearest budgeted ancestor, or the root pool
> > if none) should have enough to give out and the target budget should
> > be big enough to contain the actual usages and !max descendants in
> > the subtree. Going to max is always fine - the source previously gave
> > the budget out, so it has room to take everything back.
>
> OK... Just to understand: if we consider this situation
> root cgroup -> G1 (50, 100) -> G2 (10, 100)
> and G1 switches to "max", what happens to G2? Does it stay (10, 100),
> or is it forced to switch to "max", too?
Stay (10, 100).
> I was thinking about enforcing that a cgroup can have runtime > 0 only
> if it is a direct child of the root cgroup, or if its parent has
> runtime > 0 and is not "max" (so, in the previous example G1 can switch
> to "max" only if G2 sets its runtime to 0). Could this be acceptable?
Interfaces which change their config values without being written to tend to
cause problems and confusions for users. So, if possible, and here it seems
the resulting behavior seems pretty consistent, better to stick with the
convention, I think.
Thanks.
--
tejun
prev parent reply other threads:[~2026-05-20 18:49 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20260430213835.62217-1-yurand2000@gmail.com>
[not found] ` <20260430213835.62217-21-yurand2000@gmail.com>
2026-05-05 15:15 ` [RFC PATCH v5 20/29] sched/deadline: Allow deeper hierarchies of RT cgroups Peter Zijlstra
2026-05-05 19:56 ` Tejun Heo
2026-05-07 10:53 ` Peter Zijlstra
2026-05-07 15:03 ` Juri Lelli
2026-05-07 15:05 ` Peter Zijlstra
2026-05-07 16:39 ` luca abeni
2026-05-11 9:29 ` Juri Lelli
2026-05-11 17:52 ` Tejun Heo
2026-05-07 16:44 ` luca abeni
2026-05-11 9:40 ` luca abeni
2026-05-11 18:15 ` Tejun Heo
2026-05-11 17:37 ` Tejun Heo
2026-05-07 14:30 ` luca abeni
2026-05-11 18:28 ` Tejun Heo
2026-05-12 17:38 ` Yuri Andriaccio
2026-05-12 18:19 ` Tejun Heo
2026-05-12 18:20 ` Tejun Heo
2026-05-13 12:08 ` Yuri Andriaccio
2026-05-13 19:10 ` Tejun Heo
2026-05-14 7:25 ` luca abeni
2026-05-14 22:20 ` Tejun Heo
2026-05-18 15:27 ` Yuri Andriaccio
2026-05-18 18:47 ` Tejun Heo
2026-05-19 21:02 ` [BULK] " luca abeni
2026-05-20 18:49 ` Tejun Heo [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ag4CHYXcwOXfRCBZ@slm.duckdns.org \
--to=tj@kernel.org \
--cc=bsegall@google.com \
--cc=cgroups@vger.kernel.org \
--cc=dietmar.eggemann@arm.com \
--cc=hannes@cmpxchg.org \
--cc=juri.lelli@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=luca.abeni@santannapisa.it \
--cc=mgorman@suse.de \
--cc=mingo@redhat.com \
--cc=mkoutny@suse.com \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
--cc=vincent.guittot@linaro.org \
--cc=vschneid@redhat.com \
--cc=yurand2000@gmail.com \
--cc=yuri.andriaccio@santannapisa.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox