All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Zijlstra <peterz@infradead.org>
To: Fabio Checconi <fchecconi@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>, Thomas Gleixner <tglx@linutronix.de>,
	Paul Turner <pjt@google.com>,
	Dario Faggioli <faggioli@gandalf.sssup.it>,
	Michael Trimarchi <michael@evidence.eu.com>,
	Dhaval Giani <dhaval@retis.sssup.it>,
	Tommaso Cucinotta <t.cucinotta@sssup.it>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/3] sched: use EDF to throttle RT task groups v2
Date: Tue, 23 Mar 2010 21:30:07 +0100	[thread overview]
Message-ID: <1269376207.5283.5.camel@laptop> (raw)
In-Reply-To: <20100303170110.GS2490@gandalf.sssup.it>

On Wed, 2010-03-03 at 18:01 +0100, Fabio Checconi wrote: 
> > From: Peter Zijlstra <peterz@infradead.org>
> > Date: Sat, Feb 27, 2010 01:33:11PM +0100
> >
> > On Tue, 2010-02-23 at 19:56 +0100, Fabio Checconi wrote:
> > >   - Since it is not easy to mix tasks and groups on the same scheduler
> > >     queue (tasks have no deadlines), the bandwidth reserved to the tasks
> > >     in a group is controlled with two additional cgroup attributes:
> > >     rt_task_runtime_us and rt_task_period_us.  These attributes control,
> > >     within a cgroup, how much bandwidth is reserved to the tasks it
> > >     contains.  The old attributes, rt_runtime_us and rt_period_us, are
> > >     still there, and control the bandwidth assigned to the cgroup.  They
> > >     are used only for admission control. 
> > 
> > We could do this implicitly by simply setting the task_bandwidth to the
> > cgroup bandwidth - \Sum_children bandwidth.
> > 
> > Having it explicit gives the administrator slightly more control at the
> > cost of a more complex interface.
> > 
> > Just wondering if you thought about this trade-off?
> 
> I started with it, because I didn't like changing the interface and
> because four files to manage the bandwidth *are* a nightmare from the
> user perspective...  The problem with using only two files and assign
> the residual bandwidth to the tasks in the group is that it makes really
> hard to find out how much bw is allocated to the tasks, esp. when
> the child groups have different periods.  In practice we would be
> guaranteeing something but the user would have a hard time finding
> out what...

Right, we could do both, by allowing the task files to have -1 values,
effectively disabling their reservation and thus ending up with whatever
remains or so.

> The biggest problem with the four files used in the current implementation
> is that bandwidth assignments should be atomic (because setting all the
> parameters independently can force the system to go through non-feasible
> assignments, and the order to use when assigning runtime and period
> changes depending on the direction of the change).  I know this is a
> dangerous question, but I'll try it anyway: are we ready for multi-valued
> cgroup parameters?

Right, I don't know about multi-valued files, that sounds like its going
to confuse people too.

Another option might be some transactional interface, but not sure how
that would work out either.. maybe by toggling the system wide bandwidth
control off and on, not sure who's in charge of cgroupfs anyway. 


  reply	other threads:[~2010-03-23 20:30 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-23 18:56 [PATCH 0/3] sched: use EDF to throttle RT task groups v2 Fabio Checconi
2010-02-23 18:56 ` [PATCH 1/3] sched: use EDF to schedule groups Fabio Checconi
2010-02-25 20:28   ` Peter Zijlstra
2010-03-03 16:59     ` Fabio Checconi
2010-02-23 18:56 ` [PATCH 2/3] sched: enforce per-cpu utilization limits on runtime balancing Fabio Checconi
2010-02-25 20:28   ` Peter Zijlstra
2010-03-03 16:59     ` Fabio Checconi
2010-02-25 20:28   ` Peter Zijlstra
2010-03-03 17:00     ` Fabio Checconi
2010-03-23 20:32       ` Peter Zijlstra
2010-02-25 20:28   ` Peter Zijlstra
2010-03-03 16:59     ` Fabio Checconi
2010-02-25 20:28   ` Peter Zijlstra
2010-03-03 17:00     ` Fabio Checconi
2010-03-23 20:33       ` Peter Zijlstra
2010-02-25 20:28   ` Peter Zijlstra
2010-03-03 17:00     ` Fabio Checconi
2010-02-23 18:56 ` [PATCH 3/3] sched: make runtime balancing code more EDF-friendly Fabio Checconi
2010-02-25 20:28   ` Peter Zijlstra
2010-03-03 17:01     ` Fabio Checconi
2010-02-25 20:28 ` [PATCH 0/3] sched: use EDF to throttle RT task groups v2 Peter Zijlstra
2010-02-27 12:33 ` Peter Zijlstra
2010-03-03 17:01   ` Fabio Checconi
2010-03-23 20:30     ` Peter Zijlstra [this message]
2010-03-23 20:56       ` Dhaval Giani
2010-03-23 21:51         ` Tommaso Cucinotta

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=1269376207.5283.5.camel@laptop \
    --to=peterz@infradead.org \
    --cc=dhaval@retis.sssup.it \
    --cc=faggioli@gandalf.sssup.it \
    --cc=fchecconi@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=michael@evidence.eu.com \
    --cc=mingo@elte.hu \
    --cc=pjt@google.com \
    --cc=t.cucinotta@sssup.it \
    --cc=tglx@linutronix.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.