summaryrefslogtreecommitdiffstats
path: root/mm/memory.c
diff options
context:
space:
mode:
authorAndi Kleen <ak@linux.intel.com>2009-10-14 01:51:41 +0200
committerAndi Kleen <ak@linux.intel.com>2009-10-19 07:29:20 +0200
commit4779cb31c0ee3b355116745edca3f3e5fe865553 (patch)
tree7409cb0762ca55afe11aa981db4624d3496ed6fb /mm/memory.c
parente43c3afb367112a5b357f9adfac7817255129c88 (diff)
downloadop-kernel-dev-4779cb31c0ee3b355116745edca3f3e5fe865553.zip
op-kernel-dev-4779cb31c0ee3b355116745edca3f3e5fe865553.tar.gz
HWPOISON: Fix page count leak in hwpoison late kill in do_swap_page
When returning due to a poisoned page drop the page count. It wasn't a fatal problem because noone cares about the page count on a poisoned page (except when it wraps), but it's cleaner to fix it. Pointed out by Linus. Signed-off-by: Andi Kleen <ak@linux.intel.com>
Diffstat (limited to 'mm/memory.c')
-rw-r--r--mm/memory.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/mm/memory.c b/mm/memory.c
index 7e91b5f..7a3b0ad 100644
--- a/mm/memory.c
+++ b/mm/memory.c
@@ -2539,7 +2539,7 @@ static int do_swap_page(struct mm_struct *mm, struct vm_area_struct *vma,
} else if (PageHWPoison(page)) {
ret = VM_FAULT_HWPOISON;
delayacct_clear_flag(DELAYACCT_PF_SWAPIN);
- goto out;
+ goto out_release;
}
lock_page(page);
@@ -2611,6 +2611,7 @@ out_nomap:
pte_unmap_unlock(page_table, ptl);
out_page:
unlock_page(page);
+out_release:
page_cache_release(page);
return ret;
}
OpenPOWER on IntegriCloud