summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPetr Holasek <pholasek@redhat.com>2011-01-12 17:00:34 -0800
committerLinus Torvalds <torvalds@linux-foundation.org>2011-01-13 08:03:17 -0800
commita6fc86d2b43bf1086557f023a24adf91db915559 (patch)
tree4ad1a6f004d50c21bdce5b14d8f78fef3f453670
parentc6a340584607f653e10549c76dd427d4780c8f2c (diff)
downloadop-kernel-dev-a6fc86d2b43bf1086557f023a24adf91db915559.zip
op-kernel-dev-a6fc86d2b43bf1086557f023a24adf91db915559.tar.gz
kpagecount: add slab page checking because _mapcount is in a union
Add a PageSlab() check before adding the _mapcount value to /kpagecount. page->_mapcount is in a union with the SLAB structure so for pages controlled by SLAB, page_mapcount() returns nonsense. Signed-off-by: Petr Holasek <pholasek@redhat.com> Cc: Wu Fengguang <fengguang.wu@intel.com> Cc: Matt Mackall <mpm@selenic.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--fs/proc/page.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/proc/page.c b/fs/proc/page.c
index 3b8b4566..b06c674 100644
--- a/fs/proc/page.c
+++ b/fs/proc/page.c
@@ -40,7 +40,7 @@ static ssize_t kpagecount_read(struct file *file, char __user *buf,
ppage = pfn_to_page(pfn);
else
ppage = NULL;
- if (!ppage)
+ if (!ppage || PageSlab(ppage))
pcount = 0;
else
pcount = page_mapcount(ppage);
OpenPOWER on IntegriCloud