summaryrefslogtreecommitdiffstats
path: root/mm
diff options
context:
space:
mode:
authorChristoph Lameter <clameter@sgi.com>2008-04-14 18:52:18 +0300
committerPekka Enberg <penberg@cs.helsinki.fi>2008-04-14 18:52:18 +0300
commit49bd5221ce8fb55d12c04a3ffd375201c5bbfb7a (patch)
tree8569f6898326ae3ccbd24309dc0dd53707b11220 /mm
parent50ef37b96c11e76625067ae413dc54585ea22585 (diff)
downloadlinux-49bd5221ce8fb55d12c04a3ffd375201c5bbfb7a.tar.gz
linux-49bd5221ce8fb55d12c04a3ffd375201c5bbfb7a.tar.bz2
linux-49bd5221ce8fb55d12c04a3ffd375201c5bbfb7a.zip
slub: Move map/flag clearing to __free_slab
__free_slab does some diagnostics. The resetting of mapcount etc in discard_slab() can interfere with debug processing. So move the reset immediately before the page is freed. Signed-off-by: Christoph Lameter <clameter@sgi.com> Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Diffstat (limited to 'mm')
-rw-r--r--mm/slub.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/mm/slub.c b/mm/slub.c
index 23e5ee7b149f..f924cffb29e7 100644
--- a/mm/slub.c
+++ b/mm/slub.c
@@ -1125,6 +1125,8 @@ static void __free_slab(struct kmem_cache *s, struct page *page)
NR_SLAB_RECLAIMABLE : NR_SLAB_UNRECLAIMABLE,
-pages);
+ __ClearPageSlab(page);
+ reset_page_mapcount(page);
__free_pages(page, s->order);
}
@@ -1154,8 +1156,6 @@ static void discard_slab(struct kmem_cache *s, struct page *page)
struct kmem_cache_node *n = get_node(s, page_to_nid(page));
atomic_long_dec(&n->nr_slabs);
- reset_page_mapcount(page);
- __ClearPageSlab(page);
free_slab(s, page);
}