Messages in this thread | | | Subject | Re: [PATCH] sched: Don't allow priority switch to realtime when the task doesn't belong to init_task_group and when CONFIG_RT_GROUP_SCHED isn't set | From | Peter Zijlstra <> | Date | Sun, 23 Nov 2008 02:11:16 +0100 |
| |
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.
| |