diff options
author | Steven Rostedt <rostedt@goodmis.org> | 2005-07-25 16:28:39 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-07-26 18:40:00 -0400 |
commit | d46523ea32a79fbc8cd1237f9441f45cc3f02456 (patch) | |
tree | 0afe4590170ad25e38005f72431680ba47ba6860 | |
parent | 18586e721636527cb5177467fb17e2350615978a (diff) |
[PATCH] fix MAX_USER_RT_PRIO and MAX_RT_PRIO
Here's the patch again to fix the code to handle if the values between
MAX_USER_RT_PRIO and MAX_RT_PRIO are different.
Without this patch, an SMP system will crash if the values are
different.
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Cc: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Dean Nelson <dcn@sgi.com>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | arch/ia64/sn/kernel/xpc_main.c | 2 | ||||
-rw-r--r-- | kernel/sched.c | 5 |
2 files changed, 4 insertions, 3 deletions
diff --git a/arch/ia64/sn/kernel/xpc_main.c b/arch/ia64/sn/kernel/xpc_main.c index d580adcad927..bb1d5cf30440 100644 --- a/arch/ia64/sn/kernel/xpc_main.c +++ b/arch/ia64/sn/kernel/xpc_main.c | |||
@@ -420,7 +420,7 @@ xpc_activating(void *__partid) | |||
420 | partid_t partid = (u64) __partid; | 420 | partid_t partid = (u64) __partid; |
421 | struct xpc_partition *part = &xpc_partitions[partid]; | 421 | struct xpc_partition *part = &xpc_partitions[partid]; |
422 | unsigned long irq_flags; | 422 | unsigned long irq_flags; |
423 | struct sched_param param = { sched_priority: MAX_USER_RT_PRIO - 1 }; | 423 | struct sched_param param = { sched_priority: MAX_RT_PRIO - 1 }; |
424 | int ret; | 424 | int ret; |
425 | 425 | ||
426 | 426 | ||
diff --git a/kernel/sched.c b/kernel/sched.c index a5fb654ea590..a646e4f36c41 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
@@ -3486,7 +3486,7 @@ static void __setscheduler(struct task_struct *p, int policy, int prio) | |||
3486 | p->policy = policy; | 3486 | p->policy = policy; |
3487 | p->rt_priority = prio; | 3487 | p->rt_priority = prio; |
3488 | if (policy != SCHED_NORMAL) | 3488 | if (policy != SCHED_NORMAL) |
3489 | p->prio = MAX_USER_RT_PRIO-1 - p->rt_priority; | 3489 | p->prio = MAX_RT_PRIO-1 - p->rt_priority; |
3490 | else | 3490 | else |
3491 | p->prio = p->static_prio; | 3491 | p->prio = p->static_prio; |
3492 | } | 3492 | } |
@@ -3518,7 +3518,8 @@ recheck: | |||
3518 | * 1..MAX_USER_RT_PRIO-1, valid priority for SCHED_NORMAL is 0. | 3518 | * 1..MAX_USER_RT_PRIO-1, valid priority for SCHED_NORMAL is 0. |
3519 | */ | 3519 | */ |
3520 | if (param->sched_priority < 0 || | 3520 | if (param->sched_priority < 0 || |
3521 | param->sched_priority > MAX_USER_RT_PRIO-1) | 3521 | (p->mm && param->sched_priority > MAX_USER_RT_PRIO-1) || |
3522 | (!p->mm && param->sched_priority > MAX_RT_PRIO-1)) | ||
3522 | return -EINVAL; | 3523 | return -EINVAL; |
3523 | if ((policy == SCHED_NORMAL) != (param->sched_priority == 0)) | 3524 | if ((policy == SCHED_NORMAL) != (param->sched_priority == 0)) |
3524 | return -EINVAL; | 3525 | return -EINVAL; |