Re: [PATCH 6/6] sched: disabled rt-bandwidth by default

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Nick Piggin
Date: Thursday, August 28, 2008 - 5:03 am

On Thursday 28 August 2008 20:54, Ingo Molnar wrote:

No, it is right. With caveats. Because you can pretty well isolate a
CPU from running kernel threads or work. At any rate, I don't think it
is your decision to just mandate this.



When I write rt apps, I run a watchdog thread which detects a hang
task and kills it.



Of course I have and of course I know what it does if you run a
for (;;) rt thread on an ordinary Linux desktop system. Trying to
"fix" that for people is not a good reason to break the API.



You're continually ignoring all of my arguments and instead raising
irrelvant things like this.

You ignored others in this thread who replied with real uses of the
rt scheduling that is being prevented by this API breakage, and
you're ignoring my examples of how it could be used and just keep
asserting that "anybody who does that is broken anyway".

You also ignored when I told you how you can fix this correctly by
introducing new SCHED_xxx scheduling policies that won't break
backwards compatibility and will be defined from the outset to be
throttled as such.

There is no customer issue and there is no handwaving about compliance;
it is a black and white issue: this behaviour breaks all documentation,
previous Linux behaviour, other systems.



How am I supposed to respond to that? My car doesn't have an air bag
but it's breaks don't stop working every 10 seconds.

Can we stop with the car and gun analogies now?



OK, if you don't wish to have further discussion then I will submit a
patch to Linus and I'll see what he says.



So what?
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 6/6] sched: disabled rt-bandwidth by default, Peter Zijlstra, (Tue Aug 19, 3:33 am)
[PATCH] sched: extract walk_tg_tree(), fix, Ingo Molnar, (Tue Aug 19, 4:42 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Max Krasnyansky, (Tue Aug 19, 11:15 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Thomas Gleixner, (Tue Aug 26, 4:09 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Theodore Tso, (Tue Aug 26, 5:50 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Stefani Seibold, (Tue Aug 26, 6:31 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Mark Hounschell, (Tue Aug 26, 6:47 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Theodore Tso, (Tue Aug 26, 10:55 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Thomas Gleixner, (Tue Aug 26, 2:37 pm)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Steven Rostedt, (Tue Aug 26, 4:00 pm)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Chris Friesen, (Wed Aug 27, 11:55 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Peter Zijlstra, (Thu Aug 28, 4:19 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Peter Zijlstra, (Thu Aug 28, 5:00 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Nick Piggin, (Thu Aug 28, 5:03 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Steven Rostedt, (Thu Aug 28, 7:15 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Steven Rostedt, (Thu Aug 28, 8:12 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Steven Rostedt, (Thu Aug 28, 8:50 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Peter Zijlstra, (Thu Aug 28, 9:05 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Steven Rostedt, (Thu Aug 28, 9:15 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Max Krasnyansky, (Thu Aug 28, 9:19 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Peter Zijlstra, (Thu Aug 28, 9:29 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Max Krasnyansky, (Thu Aug 28, 9:33 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Linus Torvalds, (Thu Aug 28, 10:26 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Steven Rostedt, (Thu Aug 28, 11:04 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Darren Hart, (Thu Aug 28, 11:10 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Mark Hounschell, (Thu Aug 28, 11:16 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Linus Torvalds, (Thu Aug 28, 11:42 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Steven Rostedt, (Thu Aug 28, 11:53 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Stefani Seibold, (Thu Aug 28, 12:39 pm)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Mike Galbraith, (Fri Aug 29, 12:56 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Peter Zijlstra, (Fri Aug 29, 1:06 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Mike Galbraith, (Fri Aug 29, 1:47 am)
Re: [PATCH 6/6] sched: disabled rt-bandwidth by default, Nick Piggin, (Fri Aug 29, 11:33 pm)