[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <525C55E7.7030607@gmail.com>
Date: Mon, 14 Oct 2013 22:36:55 +0200
From: Juri Lelli <juri.lelli@...il.com>
To: Peter Zijlstra <peterz@...radead.org>
CC: tglx@...utronix.de, mingo@...hat.com, rostedt@...dmis.org,
oleg@...hat.com, fweisbec@...il.com, darren@...art.com,
johan.eker@...csson.com, p.faure@...tech.ch,
linux-kernel@...r.kernel.org, claudio@...dence.eu.com,
michael@...rulasolutions.com, fchecconi@...il.com,
tommaso.cucinotta@...up.it, nicola.manica@...i.unitn.it,
luca.abeni@...tn.it, dhaval.giani@...il.com, hgu1972@...il.com,
paulmck@...ux.vnet.ibm.com, raistlin@...ux.it,
insop.song@...il.com, liming.wang@...driver.com, jkacur@...hat.com,
harald.gustafsson@...csson.com, vincent.guittot@...aro.org
Subject: Re: [PATCH 03/14] sched: SCHED_DEADLINE structures & implementation.
On 10/14/2013 01:49 PM, Peter Zijlstra wrote:
> On Mon, Oct 14, 2013 at 12:43:35PM +0200, Juri Lelli wrote:
>> +/*
>> + * Yield task semantic for -deadline tasks is:
>> + *
>> + * get off from the CPU until our next instance, with
>> + * a new runtime.
>> + */
>
> Could you amend that comment with a reason for why this is so? I have
> vague recollections of a discussion on this subject but can't recall. It
> seems like a useful thing to have.
>
I think discussion happened before I started maintaining the patchset, but I'm
quite sure this would be helpful for bandwidth reclaiming mechanisms.
Basically, if I'm able to report that I didn't use all the budget of my current
instance, I could donate that remaining budget to other task instances.
Bandwidth reclaiming is another nice thing to have, and we actually have some
ideas on how to implement it (TODO list always grows :)).
I'll amend the comment saying that this function is of little use now, but will
be helpful in the future.
Thanks,
- Juri
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists