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 /mm | |
| parent | slub: extract common code to remove objects from partial list without locking (diff) | |
| download | linux-dev-a5dd5c117cbf620378d693963ffc42239297fac4.tar.xz linux-dev-a5dd5c117cbf620378d693963ffc42239297fac4.zip | |
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>
Diffstat (limited to 'mm')
| -rw-r--r-- | mm/slub.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/mm/slub.c b/mm/slub.c index aad00ba486f2..ac236b1ced16 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -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; |