diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-06-04 19:11:07 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-06-04 19:54:13 -0400 |
commit | 50417c55562c03e6746b13aee650c2bbb048fea3 (patch) | |
tree | 986c01bd15db89953aca177bd91653029aa137ed /mm/zbud.c | |
parent | 38515c73398a4c58059ecf1087e844561b58ee0f (diff) |
mm/zbud.c: make size unsigned like unique callsite
zbud_alloc is only called by zswap_frontswap_store with unsigned int len.
Change function parameter + update >= 0 check.
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Acked-by: Seth Jennings <sjennings@variantweb.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/zbud.c')
-rw-r--r-- | mm/zbud.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -247,7 +247,7 @@ void zbud_destroy_pool(struct zbud_pool *pool) | |||
247 | * gfp arguments are invalid or -ENOMEM if the pool was unable to allocate | 247 | * gfp arguments are invalid or -ENOMEM if the pool was unable to allocate |
248 | * a new page. | 248 | * a new page. |
249 | */ | 249 | */ |
250 | int zbud_alloc(struct zbud_pool *pool, int size, gfp_t gfp, | 250 | int zbud_alloc(struct zbud_pool *pool, unsigned int size, gfp_t gfp, |
251 | unsigned long *handle) | 251 | unsigned long *handle) |
252 | { | 252 | { |
253 | int chunks, i, freechunks; | 253 | int chunks, i, freechunks; |
@@ -255,7 +255,7 @@ int zbud_alloc(struct zbud_pool *pool, int size, gfp_t gfp, | |||
255 | enum buddy bud; | 255 | enum buddy bud; |
256 | struct page *page; | 256 | struct page *page; |
257 | 257 | ||
258 | if (size <= 0 || gfp & __GFP_HIGHMEM) | 258 | if (!size || (gfp & __GFP_HIGHMEM)) |
259 | return -EINVAL; | 259 | return -EINVAL; |
260 | if (size > PAGE_SIZE - ZHDR_SIZE_ALIGNED - CHUNK_SIZE) | 260 | if (size > PAGE_SIZE - ZHDR_SIZE_ALIGNED - CHUNK_SIZE) |
261 | return -ENOSPC; | 261 | return -ENOSPC; |