diff options
author | Oleg Nesterov <oleg@redhat.com> | 2010-03-30 12:58:29 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-04-02 14:11:05 -0400 |
commit | 47a70985e5c093ae03d8ccf633c70a93761d86f2 (patch) | |
tree | cabda69d97cd880d9b3de382d41b6b7736575a8f /kernel | |
parent | 269484a492d9177072ee11ec8c9bff71d256837a (diff) |
sched: set_cpus_allowed_ptr(): Don't use rq->migration_thread after unlock
Trivial typo fix. rq->migration_thread can be NULL after
task_rq_unlock(), this is why we have "mt" which should be
used instead.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <20100330165829.GA18284@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/sched.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index 49d2fa7b687..528a10592c1 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
@@ -5387,7 +5387,7 @@ int set_cpus_allowed_ptr(struct task_struct *p, const struct cpumask *new_mask) | |||
5387 | 5387 | ||
5388 | get_task_struct(mt); | 5388 | get_task_struct(mt); |
5389 | task_rq_unlock(rq, &flags); | 5389 | task_rq_unlock(rq, &flags); |
5390 | wake_up_process(rq->migration_thread); | 5390 | wake_up_process(mt); |
5391 | put_task_struct(mt); | 5391 | put_task_struct(mt); |
5392 | wait_for_completion(&req.done); | 5392 | wait_for_completion(&req.done); |
5393 | tlb_migrate_finish(p->mm); | 5393 | tlb_migrate_finish(p->mm); |