diff options
author | Wu Fengguang <fengguang.wu@intel.com> | 2009-12-16 12:19:57 +0100 |
---|---|---|
committer | Andi Kleen <ak@linux.intel.com> | 2009-12-16 12:19:57 +0100 |
commit | bd1ce5f91f545730df4af492f774d9d32f5da3cb (patch) | |
tree | 7d25f35cda1f37e30fd21bb2375f2693837d5ec5 /mm/madvise.c | |
parent | a7560fc80f33cab33176ee78f146df22b28e3338 (diff) | |
download | op-kernel-dev-bd1ce5f91f545730df4af492f774d9d32f5da3cb.zip op-kernel-dev-bd1ce5f91f545730df4af492f774d9d32f5da3cb.tar.gz |
HWPOISON: avoid grabbing the page count multiple times during madvise injection
If page is double referenced in madvise_hwpoison() and __memory_failure(),
remove_mapping() will fail because it expects page_count=2. Fix it by
not grabbing extra page count in __memory_failure().
Signed-off-by: Wu Fengguang <fengguang.wu@intel.com>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Diffstat (limited to 'mm/madvise.c')
-rw-r--r-- | mm/madvise.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/mm/madvise.c b/mm/madvise.c index 35b1479b..18970ae 100644 --- a/mm/madvise.c +++ b/mm/madvise.c @@ -238,7 +238,6 @@ static int madvise_hwpoison(unsigned long start, unsigned long end) page_to_pfn(p), start); /* Ignore return value for now */ __memory_failure(page_to_pfn(p), 0, 1); - put_page(p); } return ret; } |