sched: CPU hotplug events must not destroy scheduler domains created by the cpusets
authorMax Krasnyansky <maxk@qualcomm.com>
Thu, 29 May 2008 18:17:01 +0000 (11:17 -0700)
committerIngo Molnar <mingo@elte.hu>
Fri, 6 Jun 2008 13:25:00 +0000 (15:25 +0200)
commit5c8e1ed1d204a6770ca2854cd3b3597070fe7e5a
tree72a2bb4394d1f0b8492ea9566990a39ca597b840
parent1100ac91b6af02d8639d518fad5b434b1bf44ed6
sched: CPU hotplug events must not destroy scheduler domains created by the cpusets

First issue is not related to the cpusets. We're simply leaking doms_cur.
It's allocated in arch_init_sched_domains() which is called for every
hotplug event. So we just keep reallocation doms_cur without freeing it.
I introduced free_sched_domains() function that cleans things up.

Second issue is that sched domains created by the cpusets are
completely destroyed by the CPU hotplug events. For all CPU hotplug
events scheduler attaches all CPUs to the NULL domain and then puts
them all into the single domain thereby destroying domains created
by the cpusets (partition_sched_domains).
The solution is simple, when cpusets are enabled scheduler should not
create default domain and instead let cpusets do that. Which is
exactly what the patch does.

Signed-off-by: Max Krasnyansky <maxk@qualcomm.com>
Cc: pj@sgi.com
Cc: menage@google.com
Cc: rostedt@goodmis.org
Cc: mingo@elte.hu
Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
kernel/cpuset.c
kernel/sched.c