Re: [PATCH] sched: Don't allow priority switch to realtime whenthe task doesn't belong to init_task_group and when CONFIG_RT_GROUP_SCHEDisn't set

[Peter Zijlstra]

On Thu, 2008-11-20 at 11:48 +0530, Bharata B Rao wrote:
> Applies on 2.6.28-rc5.
> 
> With CONFIG_RT_GROUP_SCHED not set, don't allow a task's priority switch
> to realtime if the task isn't part of init_task_group.
> 
> A task belonging to a fair group could use sched_setscheduler/sched_setparam
> to become a realtime task. If such a task belongs to one of the
> child groups of init_task_group and if CONFIG_RT_GROUP_SCHED is not set,
> then it ends up getting queued in init_task_group's runqueue.
> So we have a situation where, a task belongs to one group (child)
> but ends in the runqueue of another group (init_task_group).
> This does not look correct.
> 
> Fix this by failing such priority change requests in sched_setscheduler()
> and sched_setparam().

NAK

!RT_GROUP means the RT tasks should be fully invariant to any grouping
configuration. This patch breaks that.

Furthermore your justification for this is plain wrong, you write as if
an RT tasks can belong to any grouping (in the !RT_GROUP case), by the
above this is untrue and shows a fundamental mis-understanding of the
concepts involved.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/