On Mon, 2009-09-21 at 23:22 +1000, Peter Williams wrote:Or is the line:
p->prio = effective_prio(p);
in wake_up_new_task() an expensive no op.
As far as I can tell from reading the code, it will always be the case
that EITHER rt_prio(p->prio) is true OR p->prio == p->normal_prio when
this call is made and, in either case, the value of p->prio will be
unchanged. In addition, when this call is made p->normal_prio is
already equal to to normal_prio(p), so the side effects of the function
(setting p->normal_prio) are also unnecessary.
Am I correct or have I missed something?
Yuck @ all that prio code..
I think you're right, sched_fork() resets the prio, so poking at it in
wake_up_new_task() seems superfluous.
I've been meaning to re-write most of the PI code one of these days, but
so far I've not had time to.
My initial goal is to replace plist with a rb-tree and fix some of the
boost paths to be inside the scheduler. That is, we currently have the
fun situation that we boost a lock owner, which becomes runnable, gets
pushed to another cpu, then current blocks and reschedules, leaving this
cpu to again sort out work.
It would be much easier if we'd first dequeue current, then boost and
then select the owner. Saves a bit of bouncing around.
The rb-tree is needed for things like PI on CFS (yes, you can do a form
of PI on proportional schedulers), and we're going to look at doing a
full sporadic task model deadline scheduler, which needs both deadline
inheritance and bandwidth inheritance.