Messages in this thread Patch in this message | | | Subject | [PATCH] 2.5.2: sched fixes | From | Robert Love <> | Date | 15 Jan 2002 14:48:15 -0500 |
| |
[ some people have asked that I repost this, since it seems it never got to the list. I believe it is in Linus's queue now. ]
Linus,
The following is some of the sane bits from Ingo's latest H7 patch. It contains some critical bug fixes and should all by obviously correct.
I suspect Ingo will have a full patch ready in due time, but these fixes should be merged sooner rather than later. These don't infringe on any of Davide's bits (another reason to just merge fixes for now).
Itemized Changes:
- avoid IPI lock with smp_send_reschedule in resched_task
- remove unneeded branch in try_to_wake_up, which also has a locking rule error
- check to make sure new cpus_allowed mask is valid before setting
- fix locking rule error in set_cpus_allowed
- cleanup
Robert Love
diff -urN linux-2.5.2/kernel/sched.c linux/kernel/sched.c --- linux-2.5.2/kernel/sched.c Mon Jan 14 22:46:56 2002 +++ linux/kernel/sched.c Mon Jan 14 23:28:44 2002 @@ -126,7 +126,7 @@ need_resched = p->need_resched; wmb(); p->need_resched = 1; - if (!need_resched) + if (!need_resched && (p->cpu != smp_processor_id())) smp_send_reschedule(p->cpu); } @@ -188,17 +188,9 @@ lock_task_rq(rq, p, flags); p->state = TASK_RUNNING; if (!p->array) { - if (!rt_task(p) && synchronous && (smp_processor_id() < p->cpu)) { - spin_lock(&this_rq()->lock); - p->cpu = smp_processor_id(); - activate_task(p, this_rq()); - spin_unlock(&this_rq()->lock); - } else { - activate_task(p, rq); - if ((rq->curr == rq->idle) || - (p->prio < rq->curr->prio)) - resched_task(rq->curr); - } + activate_task(p, rq); + if ((rq->curr == rq->idle) || (p->prio < rq->curr->prio)) + resched_task(rq->curr); success = 1; } unlock_task_rq(rq, p, flags); @@ -690,6 +682,8 @@ int target_cpu; new_mask &= cpu_online_map; + if (!new_mask) + BUG(); p->cpus_allowed = new_mask; /* * Can the task run on the current CPU? If not then @@ -703,8 +697,8 @@ spin_lock_irq(&target_rq->lock); spin_lock(&this_rq->lock); } else { - spin_lock_irq(&target_rq->lock); - spin_lock(&this_rq->lock); + spin_lock_irq(&this_rq->lock); + spin_lock(&target_rq->lock); } dequeue_task(p, p->array); this_rq->nr_running--; @@ -861,7 +855,7 @@ goto out_unlock; retval = -EPERM; - if ((policy == SCHED_FIFO || policy == SCHED_RR) && + if ((policy == SCHED_FIFO || policy == SCHED_RR) && !capable(CAP_SYS_NICE)) goto out_unlock; if ((current->euid != p->euid) && (current->euid != p->uid) && @@ -890,7 +884,7 @@ return retval; } -asmlinkage long sys_sched_setscheduler(pid_t pid, int policy, +asmlinkage long sys_sched_setscheduler(pid_t pid, int policy, struct sched_param *param) { return setscheduler(pid, policy, param);
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |