diff options
| author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2009-07-21 13:56:38 +0200 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2009-08-02 14:26:15 +0200 |
| commit | 693525e3bea25cf2ee6cf2b862ba7c148e891df2 (patch) | |
| tree | 7bb3ba6262c1adad1ffb8a52df9ff0924caec76a /kernel/sched.c | |
| parent | 8f48894fcc89ddec62e1762f73a0825793e59e91 (diff) | |
| download | op-kernel-dev-693525e3bea25cf2ee6cf2b862ba7c148e891df2.zip op-kernel-dev-693525e3bea25cf2ee6cf2b862ba7c148e891df2.tar.gz | |
sched: Ensure the migration task doesn't go away during use
Like sched_migrate_task(), set_cpus_allowed_ptr() should hold
onto the migration thread too.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/sched.c')
| -rw-r--r-- | kernel/sched.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index 475138c..7f83be3 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -7078,8 +7078,12 @@ int set_cpus_allowed_ptr(struct task_struct *p, const struct cpumask *new_mask) if (migrate_task(p, cpumask_any_and(cpu_online_mask, new_mask), &req)) { /* Need help from migration thread: drop lock and wait. */ + struct task_struct *mt = rq->migration_thread; + + get_task_struct(mt); task_rq_unlock(rq, &flags); wake_up_process(rq->migration_thread); + put_task_struct(mt); wait_for_completion(&req.done); tlb_migrate_finish(p->mm); return 0; |
