diff options
author | Daniel Yeisley <dan.yeisley@unisys.com> | 2006-11-03 01:07:14 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-11-03 15:27:58 -0500 |
commit | 7f6b8876c7e66b0d15af134e2a5b87e55514eb6d (patch) | |
tree | b15c14ff1d4b31f80071299ecdfef8c29236a4d5 /mm/slab.c | |
parent | d13adb604693374c5fce47cd1a2017bcf3178eae (diff) |
[PATCH] init_reap_node() initialization fix
It looks like there is a bug in init_reap_node() in slab.c that can cause
multiple oops's on certain ES7000 configurations. The variable reap_node
is defined per cpu, but only initialized on a single CPU. This causes an
oops in next_reap_node() when __get_cpu_var(reap_node) returns the wrong
value. Fix is below.
Signed-off-by: Dan Yeisley <dan.yeisley@unisys.com>
Cc: Andi Kleen <ak@suse.de>
Acked-by: Christoph Lameter <clameter@engr.sgi.com>
Cc: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: Manfred Spraul <manfred@colorfullife.com>
Cc: <stable@kernel.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'mm/slab.c')
-rw-r--r-- | mm/slab.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -883,7 +883,7 @@ static void init_reap_node(int cpu) | |||
883 | if (node == MAX_NUMNODES) | 883 | if (node == MAX_NUMNODES) |
884 | node = first_node(node_online_map); | 884 | node = first_node(node_online_map); |
885 | 885 | ||
886 | __get_cpu_var(reap_node) = node; | 886 | per_cpu(reap_node, cpu) = node; |
887 | } | 887 | } |
888 | 888 | ||
889 | static void next_reap_node(void) | 889 | static void next_reap_node(void) |