diff options
author | Vincent Guittot <vincent.guittot@linaro.org> | 2014-05-13 05:11:01 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2014-05-22 05:16:29 -0400 |
commit | c515db8cd311ef77b2dc7cbd6b695022655bb0f3 (patch) | |
tree | b3bdc7320f117e3c2b68e2a8ea6eef0892d370ce /kernel/sched/core.c | |
parent | 8bf21433f38b020c3d8a3805d1d7fb73d7b40c01 (diff) |
sched/numa: Fix initialization of sched_domain_topology for NUMA
Jet Chen has reported a kernel panics when booting qemu-system-x86_64 with
kvm64 cpu. A panic occured while building the sched_domain.
In sched_init_numa, we create a new topology table in which both default
levels and numa levels are copied. The last row of the table must have a null
pointer in the mask field.
The current implementation doesn't add this last row in the computation of the
table size. So we add 1 row in the allocation size that will be used as the
last row of the table. The kzalloc will ensure that the mask field is NULL.
Reported-by: Jet Chen <jet.chen@intel.com>
Tested-by: Jet Chen <jet.chen@intel.com>
Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org>
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Cc: fengguang.wu@intel.com
Link: http://lkml.kernel.org/r/1399972261-25693-1-git-send-email-vincent.guittot@linaro.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'kernel/sched/core.c')
-rw-r--r-- | kernel/sched/core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched/core.c b/kernel/sched/core.c index a78c5b62a470..45d077ed24fb 100644 --- a/kernel/sched/core.c +++ b/kernel/sched/core.c | |||
@@ -6231,7 +6231,7 @@ static void sched_init_numa(void) | |||
6231 | /* Compute default topology size */ | 6231 | /* Compute default topology size */ |
6232 | for (i = 0; sched_domain_topology[i].mask; i++); | 6232 | for (i = 0; sched_domain_topology[i].mask; i++); |
6233 | 6233 | ||
6234 | tl = kzalloc((i + level) * | 6234 | tl = kzalloc((i + level + 1) * |
6235 | sizeof(struct sched_domain_topology_level), GFP_KERNEL); | 6235 | sizeof(struct sched_domain_topology_level), GFP_KERNEL); |
6236 | if (!tl) | 6236 | if (!tl) |
6237 | return; | 6237 | return; |