diff options
author | Jerry Zhou <uulinux@gmail.com> | 2013-09-11 17:20:26 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-09-11 18:57:12 -0400 |
commit | a7e833182a926ae5bc03204dbd00b0bb5539088b (patch) | |
tree | 9349bb816c8be88a13316d35f41b8e29b341b9d0 /mm | |
parent | 3dbb95f7895e378514ffefa93cc887fb1bc9df94 (diff) |
mm: fix negative left shift count when PAGE_SHIFT > 20
When PAGE_SHIFT > 20, the result of "20 - PAGE_SHIFT" is negative. The
previous calculating here will generate an unexpected result. In
addition, if PAGE_SIZE >= 1MB, The memory size of "numentries" was
already integral multiple of 1MB.
Signed-off-by: Jerry Zhou <uulinux@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/page_alloc.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/mm/page_alloc.c b/mm/page_alloc.c index c2b59dbda196..116bab1c2cf5 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c | |||
@@ -5745,9 +5745,10 @@ void *__init alloc_large_system_hash(const char *tablename, | |||
5745 | if (!numentries) { | 5745 | if (!numentries) { |
5746 | /* round applicable memory size up to nearest megabyte */ | 5746 | /* round applicable memory size up to nearest megabyte */ |
5747 | numentries = nr_kernel_pages; | 5747 | numentries = nr_kernel_pages; |
5748 | numentries += (1UL << (20 - PAGE_SHIFT)) - 1; | 5748 | |
5749 | numentries >>= 20 - PAGE_SHIFT; | 5749 | /* It isn't necessary when PAGE_SIZE >= 1MB */ |
5750 | numentries <<= 20 - PAGE_SHIFT; | 5750 | if (PAGE_SHIFT < 20) |
5751 | numentries = round_up(numentries, (1<<20)/PAGE_SIZE); | ||
5751 | 5752 | ||
5752 | /* limit to 1 bucket per 2^scale bytes of low memory */ | 5753 | /* limit to 1 bucket per 2^scale bytes of low memory */ |
5753 | if (scale > PAGE_SHIFT) | 5754 | if (scale > PAGE_SHIFT) |