diff options
author | Christoph Lameter <clameter@sgi.com> | 2007-05-06 17:49:38 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-05-07 15:12:53 -0400 |
commit | 614410d5892af5f86d0ec14e28f9f6d5f4ac9e9b (patch) | |
tree | 616c8437f45590a0ca6e2efdc2a5dfb61799d0ec /include | |
parent | 47bfdc0d5a18a4b760ffb6a332932aaa5c0859e0 (diff) |
SLUB: allocate smallest object size if the user asks for 0 bytes
Makes SLUB behave like SLAB in this area to avoid issues....
Throw a stack dump to alert people.
At some point the behavior should be switched back. NULL is no memory as
far as I can tell and if the use asked for 0 bytes then he need to get no
memory.
Signed-off-by: Christoph Lameter <clameter@sgi.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/slub_def.h | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/include/linux/slub_def.h b/include/linux/slub_def.h index 30b154ce7289..f8e0c86c48a9 100644 --- a/include/linux/slub_def.h +++ b/include/linux/slub_def.h | |||
@@ -80,8 +80,12 @@ extern struct kmem_cache kmalloc_caches[KMALLOC_SHIFT_HIGH + 1]; | |||
80 | */ | 80 | */ |
81 | static inline int kmalloc_index(int size) | 81 | static inline int kmalloc_index(int size) |
82 | { | 82 | { |
83 | if (size == 0) | 83 | /* |
84 | return 0; | 84 | * We should return 0 if size == 0 but we use the smallest object |
85 | * here for SLAB legacy reasons. | ||
86 | */ | ||
87 | WARN_ON_ONCE(size == 0); | ||
88 | |||
85 | if (size > 64 && size <= 96) | 89 | if (size > 64 && size <= 96) |
86 | return 1; | 90 | return 1; |
87 | if (size > 128 && size <= 192) | 91 | if (size > 128 && size <= 192) |