sched: set_cpus_allowed_ptr(): Don't use rq->migration_thread after unlock

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linux Kernel Mailing List
Date: Sunday, April 4, 2010 - 12:59 pm

Gitweb:     http://git.kernel.org/linus/47a70985e5c093ae03d8ccf633c70a93761d86f2
Commit:     47a70985e5c093ae03d8ccf633c70a93761d86f2
Parent:     269484a492d9177072ee11ec8c9bff71d256837a
Author:     Oleg Nesterov <oleg@redhat.com>
AuthorDate: Tue Mar 30 18:58:29 2010 +0200
Committer:  Ingo Molnar <mingo@elte.hu>
CommitDate: Fri Apr 2 20:11:05 2010 +0200

    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>
---
 kernel/sched.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/kernel/sched.c b/kernel/sched.c
index 49d2fa7..528a105 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)
 
 		get_task_struct(mt);
 		task_rq_unlock(rq, &flags);
-		wake_up_process(rq->migration_thread);
+		wake_up_process(mt);
 		put_task_struct(mt);
 		wait_for_completion(&req.done);
 		tlb_migrate_finish(p->mm);
--
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
sched: set_cpus_allowed_ptr(): Don't use rq->migration_thr ..., Linux Kernel Mailing ..., (Sun Apr 4, 12:59 pm)