slab: NUMA slab allocator migration bugfix

NUMA slab allocator cpu migration bugfix

The NUMA slab allocator (specifically, cache_alloc_refill)
is not refreshing its local copies of what cpu and what
numa node it is on, when it drops and reacquires the irq
block that it inherited from its caller. As a result
those values become invalid if an attempt to migrate the
process to another numa node occured while the irq block
had been dropped.

The solution is to make cache_alloc_refill reload these
variables whenever it drops and reacquires the irq block.

The error is very difficult to hit. When it does occur,
one gets the following oops + stack traceback bits in
check_spinlock_acquired:

kernel BUG at mm/slab.c:2417
cache_alloc_refill+0xe6
kmem_cache_alloc+0xd0
...

This patch was developed against 2.6.23, ported to and
compiled-tested only against 2.6.25-rc4.

Signed-off-by: Joe Korty <joe.korty@ccur.com>
Signed-off-by: Christoph Lameter <clameter@sgi.com>

authored by Joe Korty and committed by Christoph Lameter 6d2144d3 b6210386

+3 -4
+3 -4
mm/slab.c
··· 2964 struct array_cache *ac; 2965 int node; 2966 2967 - node = numa_node_id(); 2968 - 2969 - check_irq_off(); 2970 - ac = cpu_cache_get(cachep); 2971 retry: 2972 batchcount = ac->batchcount; 2973 if (!ac->touched && batchcount > BATCHREFILL_LIMIT) { 2974 /*
··· 2964 struct array_cache *ac; 2965 int node; 2966 2967 retry: 2968 + check_irq_off(); 2969 + node = numa_node_id(); 2970 + ac = cpu_cache_get(cachep); 2971 batchcount = ac->batchcount; 2972 if (!ac->touched && batchcount > BATCHREFILL_LIMIT) { 2973 /*