diff options
| author | Andi Kleen <ak@linux.intel.com> | 2010-09-28 07:37:55 +0200 |
|---|---|---|
| committer | Andi Kleen <ak@linux.intel.com> | 2010-10-07 09:47:10 +0200 |
| commit | 47f43e7efadacc627f325aba64c6a547de0926db (patch) | |
| tree | e83afbf6be4f594d6f0cd1bf815e9469d4a14f18 /mm/memory-failure.c | |
| parent | HWPOISON: Report correct address granuality for AO huge page errors (diff) | |
| download | linux-47f43e7efadacc627f325aba64c6a547de0926db.tar.xz linux-47f43e7efadacc627f325aba64c6a547de0926db.zip | |
HWPOISON: Stop shrinking at right page count
When we call the slab shrinker to free a page we need to stop at
page count one because the caller always holds a single reference, not zero.
This avoids useless looping over slab shrinkers and freeing too much
memory.
Reviewed-by: Wu Fengguang <fengguang.wu@intel.com>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Diffstat (limited to 'mm/memory-failure.c')
| -rw-r--r-- | mm/memory-failure.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/memory-failure.c b/mm/memory-failure.c index cd7e6974de77..757f6b0accfe 100644 --- a/mm/memory-failure.c +++ b/mm/memory-failure.c @@ -235,7 +235,7 @@ void shake_page(struct page *p, int access) int nr; do { nr = shrink_slab(1000, GFP_KERNEL, 1000); - if (page_count(p) == 0) + if (page_count(p) == 1) break; } while (nr > 10); } |