LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* RT scheduler config, suggestions and questions
@ 2008-02-05 23:37 Max Krasnyanskiy
  2008-02-06  6:36 ` Peter Zijlstra
  0 siblings, 1 reply; 6+ messages in thread
From: Max Krasnyanskiy @ 2008-02-05 23:37 UTC (permalink / raw)
  To: Peter Zijlstra, Steven Rostedt, Ingo Molnar; +Cc: LKML

Folks,

I just realized that in latest Linus' tree following sysctls are under SCHED_DEBUG:
	sched_rt_period
	sched_rt_ratio

I do not believe that is correct. I know that we do not want to expose scheduler knobs
in general but theses are not the heuristic kind of knobs. There is no way the scheduler 
can magically figure out what the correct setting should be here.

Also shouldn't those new RT features that recently went be configurable and _disabled_ 
by default ? For example "RT watchdog" and "RT throttling" actually seem very questionable. 
SCHED_FIFO is clearly defined as
"
  A SCHED_FIFO process runs until either it is blocked by an I/O request, it is preempted 
  by a higher priority process, or it calls sched_yield(2).
"

Both the watchdog and the throttling are clearly braking that rule. I think it's good to have 
those features but not enabled by default and certainly not with sysctls that disable them 
hidden under debugging.
How about this:
- We introduce Kconfig options for them ?
- Expose all rt sysctls outside of #ifdef DEBUG

I can kook up some patches if that sounds ok.

----

btw I can see "watchdog" being very useful to catch hard-RT tasks that exceed the deadline.
But's it gotta be per thread. Single setting per user is not enough. Unless a use has a single 
RT task.

Max

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2008-02-06 21:51 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-02-05 23:37 RT scheduler config, suggestions and questions Max Krasnyanskiy
2008-02-06  6:36 ` Peter Zijlstra
2008-02-06 11:29   ` Peter Zijlstra
2008-02-06 18:04     ` Max Krasnyanskiy
2008-02-06 21:50       ` Peter Zijlstra
2008-02-06 17:52   ` Max Krasnyanskiy

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).