Prev: HTC Dream drivers was Re: [linux-pm] suspend blockers & Android integration
Next: [S+Q2 06/19] slub: Check kasprintf results in kmem_cache_init()
From: Christoph Lameter on 9 Jul 2010 15:20
The sequence of unlocking a slab and freeing occurs multiple times.
Put the common into a single function. There is no intend here
to use this to enforce a sequencing of events leading up
to the discarding of a slab page.

Signed-off-by: Christoph Lameter <cl(a)linux-foundation.org>

---
mm/slub.c | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)

Index: linux-2.6/mm/slub.c
===================================================================
--- linux-2.6.orig/mm/slub.c 2010-06-01 08:58:50.000000000 -0500
+++ linux-2.6/mm/slub.c 2010-06-01 08:58:54.000000000 -0500
@@ -1260,6 +1260,13 @@ static __always_inline int slab_trylock(
return rc;
}

+static void discard_slab_unlock(struct kmem_cache *s,
+ struct page *page)
+{
+ slab_unlock(page);
+ discard_slab(s, page);
+}
+
/*
* Management of partially allocated slabs
*/
@@ -1437,9 +1444,8 @@ static void unfreeze_slab(struct kmem_ca
add_partial(n, page, 1);
slab_unlock(page);
} else {
- slab_unlock(page);
stat(s, FREE_SLAB);
- discard_slab(s, page);
+ discard_slab_unlock(s, page);
}
}
}
@@ -1822,9 +1828,8 @@ slab_empty:
remove_partial(s, page);
stat(s, FREE_REMOVE_PARTIAL);
}
- slab_unlock(page);
stat(s, FREE_SLAB);
- discard_slab(s, page);
+ discard_slab_unlock(s, page);
return;

debug:
@@ -2893,8 +2898,7 @@ int kmem_cache_shrink(struct kmem_cache
*/
list_del(&page->lru);
n->nr_partial--;
- slab_unlock(page);
- discard_slab(s, page);
+ discard_slab_unlock(s, page);
} else {
list_move(&page->lru,
slabs_by_inuse + page->inuse);

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: HTC Dream drivers was Re: [linux-pm] suspend blockers & Android integration
Next: [S+Q2 06/19] slub: Check kasprintf results in kmem_cache_init()