diff options
author | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2009-09-21 20:02:30 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-09-22 10:17:33 -0400 |
commit | fe1ff49d0d1c30254dbfc84c3786eb538e0cc7d1 (patch) | |
tree | aed0b1c5355e42ce27788fa4868dc770cbbebe75 /mm/slub.c | |
parent | 398499d5f3613c47f2143b8c54a04efb5d7a6da9 (diff) |
mm: kmem_cache_create(): make it easier to catch NULL cache names
Right now, if you inadvertently pass NULL to kmem_cache_create() at boot
time, it crashes much later after boot somewhere deep inside sysfs which
makes it very non obvious to figure out what's going on.
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/slub.c')
-rw-r--r-- | mm/slub.c | 3 |
1 files changed, 3 insertions, 0 deletions
@@ -3345,6 +3345,9 @@ struct kmem_cache *kmem_cache_create(const char *name, size_t size, | |||
3345 | { | 3345 | { |
3346 | struct kmem_cache *s; | 3346 | struct kmem_cache *s; |
3347 | 3347 | ||
3348 | if (WARN_ON(!name)) | ||
3349 | return NULL; | ||
3350 | |||
3348 | down_write(&slub_lock); | 3351 | down_write(&slub_lock); |
3349 | s = find_mergeable(size, align, flags, name, ctor); | 3352 | s = find_mergeable(size, align, flags, name, ctor); |
3350 | if (s) { | 3353 | if (s) { |