diff options
author | Christoph Lameter <clameter@sgi.com> | 2008-04-14 18:52:18 +0300 |
---|---|---|
committer | Pekka Enberg <penberg@cs.helsinki.fi> | 2008-04-14 18:52:18 +0300 |
commit | 49bd5221ce8fb55d12c04a3ffd375201c5bbfb7a (patch) | |
tree | 8569f6898326ae3ccbd24309dc0dd53707b11220 /mm | |
parent | 50ef37b96c11e76625067ae413dc54585ea22585 (diff) | |
download | lwn-49bd5221ce8fb55d12c04a3ffd375201c5bbfb7a.tar.gz lwn-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.c | 4 |
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); } |