diff options
author | Namhyung Kim <namhyung@gmail.com> | 2010-09-29 21:02:13 +0900 |
---|---|---|
committer | Pekka Enberg <penberg@kernel.org> | 2010-10-02 10:47:52 +0300 |
commit | a5dd5c117cbf620378d693963ffc42239297fac4 (patch) | |
tree | 12dc4b1bc7da97acebb04afabd07b20cc6642a3b | |
parent | 62e346a83026a28526fc9799337bcc6154819f25 (diff) | |
download | op-kernel-dev-a5dd5c117cbf620378d693963ffc42239297fac4.zip op-kernel-dev-a5dd5c117cbf620378d693963ffc42239297fac4.tar.gz |
slub: Fix signedness warnings
The bit-ops routines require its arg to be a pointer to unsigned long.
This leads sparse to complain about different signedness as follows:
mm/slub.c:2425:49: warning: incorrect type in argument 2 (different signedness)
mm/slub.c:2425:49: expected unsigned long volatile *addr
mm/slub.c:2425:49: got long *map
Acked-by: Christoph Lameter <cl@linux.com>
Acked-by: David Rientjes <rientjes@google.com>
Signed-off-by: Namhyung Kim <namhyung@gmail.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
-rw-r--r-- | mm/slub.c | 7 |
1 files changed, 3 insertions, 4 deletions
@@ -2433,9 +2433,8 @@ static void list_slab_objects(struct kmem_cache *s, struct page *page, #ifdef CONFIG_SLUB_DEBUG void *addr = page_address(page); void *p; - long *map = kzalloc(BITS_TO_LONGS(page->objects) * sizeof(long), - GFP_ATOMIC); - + unsigned long *map = kzalloc(BITS_TO_LONGS(page->objects) * + sizeof(long), GFP_ATOMIC); if (!map) return; slab_err(s, page, "%s", text); @@ -3660,7 +3659,7 @@ static int add_location(struct loc_track *t, struct kmem_cache *s, static void process_slab(struct loc_track *t, struct kmem_cache *s, struct page *page, enum track_item alloc, - long *map) + unsigned long *map) { void *addr = page_address(page); void *p; |