diff options
| author | Sage Weil <sage@newdream.net> | 2009-09-21 20:02:56 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-09-22 10:17:35 -0400 |
| commit | bba78819548a59a52e60f0b259997bbd011164ae (patch) | |
| tree | 1d788069848ba86906269d8eb8b1b13119260e5a | |
| parent | bbba809e96539672f775a3d70102657d05816a5b (diff) | |
mm: remove broken 'kzalloc' mempool
The kzalloc mempool zeros items when they are initially allocated, but
does not rezero used items that are returned to the pool. Consequently
mempool_alloc()s may return non-zeroed memory.
Since there are/were only two in-tree users for
mempool_create_kzalloc_pool(), and 'fixing' this in a way that will
re-zero used (but not new) items before first use is non-trivial, just
remove it.
Signed-off-by: Sage Weil <sage@newdream.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | include/linux/mempool.h | 10 | ||||
| -rw-r--r-- | mm/mempool.c | 7 |
2 files changed, 2 insertions, 15 deletions
diff --git a/include/linux/mempool.h b/include/linux/mempool.h index 9be484d11283..7c08052e3321 100644 --- a/include/linux/mempool.h +++ b/include/linux/mempool.h | |||
| @@ -47,22 +47,16 @@ mempool_create_slab_pool(int min_nr, struct kmem_cache *kc) | |||
| 47 | } | 47 | } |
| 48 | 48 | ||
| 49 | /* | 49 | /* |
| 50 | * 2 mempool_alloc_t's and a mempool_free_t to kmalloc/kzalloc and kfree | 50 | * a mempool_alloc_t and a mempool_free_t to kmalloc and kfree the |
| 51 | * the amount of memory specified by pool_data | 51 | * amount of memory specified by pool_data |
| 52 | */ | 52 | */ |
| 53 | void *mempool_kmalloc(gfp_t gfp_mask, void *pool_data); | 53 | void *mempool_kmalloc(gfp_t gfp_mask, void *pool_data); |
| 54 | void *mempool_kzalloc(gfp_t gfp_mask, void *pool_data); | ||
| 55 | void mempool_kfree(void *element, void *pool_data); | 54 | void mempool_kfree(void *element, void *pool_data); |
| 56 | static inline mempool_t *mempool_create_kmalloc_pool(int min_nr, size_t size) | 55 | static inline mempool_t *mempool_create_kmalloc_pool(int min_nr, size_t size) |
| 57 | { | 56 | { |
| 58 | return mempool_create(min_nr, mempool_kmalloc, mempool_kfree, | 57 | return mempool_create(min_nr, mempool_kmalloc, mempool_kfree, |
| 59 | (void *) size); | 58 | (void *) size); |
| 60 | } | 59 | } |
| 61 | static inline mempool_t *mempool_create_kzalloc_pool(int min_nr, size_t size) | ||
| 62 | { | ||
| 63 | return mempool_create(min_nr, mempool_kzalloc, mempool_kfree, | ||
| 64 | (void *) size); | ||
| 65 | } | ||
| 66 | 60 | ||
| 67 | /* | 61 | /* |
| 68 | * A mempool_alloc_t and mempool_free_t for a simple page allocator that | 62 | * A mempool_alloc_t and mempool_free_t for a simple page allocator that |
diff --git a/mm/mempool.c b/mm/mempool.c index 32e75d400503..1a3bc3d4d554 100644 --- a/mm/mempool.c +++ b/mm/mempool.c | |||
| @@ -308,13 +308,6 @@ void *mempool_kmalloc(gfp_t gfp_mask, void *pool_data) | |||
| 308 | } | 308 | } |
| 309 | EXPORT_SYMBOL(mempool_kmalloc); | 309 | EXPORT_SYMBOL(mempool_kmalloc); |
| 310 | 310 | ||
| 311 | void *mempool_kzalloc(gfp_t gfp_mask, void *pool_data) | ||
| 312 | { | ||
| 313 | size_t size = (size_t)pool_data; | ||
| 314 | return kzalloc(size, gfp_mask); | ||
| 315 | } | ||
| 316 | EXPORT_SYMBOL(mempool_kzalloc); | ||
| 317 | |||
| 318 | void mempool_kfree(void *element, void *pool_data) | 311 | void mempool_kfree(void *element, void *pool_data) |
| 319 | { | 312 | { |
| 320 | kfree(element); | 313 | kfree(element); |
