[PATCH] set_cpus_allowed fix

Mike Kravetz pointed out that the set_cpus_allowed optimization he
suggested has a small but possible race condition wherein the system
could still be operating in the context of the task but it is not
running.  On top of this, the runqueue lock can be dropped in
load_balance and thus we can race and set task->cpu at a very unpleasant
time.

My solution is to just remove the optimization.

kernel/sched.c

@@ -1377,6 +1377,7 @@ asmlinkage long sys_sched_yield(void)
 
 	return 0;
 }
+
 asmlinkage long sys_sched_get_priority_max(int policy)
 {
 	int ret = -EINVAL;
@@ -1687,16 +1688,6 @@ void set_cpus_allowed(task_t *p, unsigned long new_mask)
 		goto out;
 	}
 
-	/*
-	 * If the task is not on a runqueue, then it is sufficient
-	 * to simply update the task's cpu field.
-	 */
-	if (!p->array) {
-		p->thread_info->cpu = __ffs(p->cpus_allowed);
-		task_rq_unlock(rq, &flags);
-		goto out;
-	}
-
 	init_MUTEX_LOCKED(&req.sem);
 	req.task = p;
 	list_add(&req.list, &rq->migration_queue);