]> err.no Git - linux-2.6/commitdiff
sched: fix cpu hotplug
authorDmitry Adamushko <dmitry.adamushko@gmail.com>
Sat, 28 Jun 2008 22:16:56 +0000 (00:16 +0200)
committerIngo Molnar <mingo@elte.hu>
Sun, 29 Jun 2008 06:50:21 +0000 (08:50 +0200)
the CPU hotplug problems (crashes under high-volume unplug+replug
tests) seem to be related to migrate_dead_tasks().

Firstly I added traces to see all tasks being migrated with
migrate_live_tasks() and migrate_dead_tasks(). On my setup the problem
pops up (the one with "se == NULL" in the loop of
pick_next_task_fair()) shortly after the traces indicate that some has
been migrated with migrate_dead_tasks()). btw., I can reproduce it
much faster now with just a plain cpu down/up loop.

[disclaimer] Well, unless I'm really missing something important in
this late hour [/desclaimer] pick_next_task() is not something
appropriate for migrate_dead_tasks() :-)

the following change seems to eliminate the problem on my setup
(although, I kept it running only for a few minutes to get a few
messages indicating migrate_dead_tasks() does move tasks and the
system is still ok)

Signed-off-by: Ingo Molnar <mingo@elte.hu>
kernel/sched.c

index 3aaa5c8cb4214586bd283eedc2375baa3dc739c0..a66e85639de2e541dd9726245cd344d3ea2d9f3b 100644 (file)
@@ -5887,6 +5887,7 @@ static void migrate_dead_tasks(unsigned int dead_cpu)
                next = pick_next_task(rq, rq->curr);
                if (!next)
                        break;
+               next->sched_class->put_prev_task(rq, next);
                migrate_dead(dead_cpu, next);
 
        }