diff options
| author | Stanislaw Gruszka <sgruszka@redhat.com> | 2012-01-10 15:07:31 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-01-10 16:30:42 -0800 |
| commit | c6968e73b90c2a2fb9a32d4bad249f8f70f70125 (patch) | |
| tree | 67ec90fc8e187fcf316298dd9d514bbbbbc8f187 | |
| parent | c0a32fc5a2e470d0b02597b23ad79a317735253e (diff) | |
| download | kernel_goldelico_gta04-c6968e73b90c2a2fb9a32d4bad249f8f70f70125.zip kernel_goldelico_gta04-c6968e73b90c2a2fb9a32d4bad249f8f70f70125.tar.gz kernel_goldelico_gta04-c6968e73b90c2a2fb9a32d4bad249f8f70f70125.tar.bz2 | |
PM/Hibernate: do not count debug pages as savable
When debugging with CONFIG_DEBUG_PAGEALLOC and debug_guardpage_minorder >
0, we have lot of free pages that are not marked so. Snapshot code
account them as savable, what cause hibernate memory preallocation
failure.
It is pretty hard to make hibernate allocation succeed with
debug_guardpage_minorder=1. This change at least make it possible when
system has relatively big amount of RAM.
Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
Acked-by: Rafael J. Wysocki <rjw@sisk.pl>
Cc: Andrea Arcangeli <aarcange@redhat.com>
Cc: Christoph Lameter <cl@linux-foundation.org>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | kernel/power/snapshot.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/kernel/power/snapshot.c b/kernel/power/snapshot.c index cbe2c14..1cf8890 100644 --- a/kernel/power/snapshot.c +++ b/kernel/power/snapshot.c @@ -858,6 +858,9 @@ static struct page *saveable_highmem_page(struct zone *zone, unsigned long pfn) PageReserved(page)) return NULL; + if (page_is_guard(page)) + return NULL; + return page; } @@ -920,6 +923,9 @@ static struct page *saveable_page(struct zone *zone, unsigned long pfn) && (!kernel_page_present(page) || pfn_is_nosave(pfn))) return NULL; + if (page_is_guard(page)) + return NULL; + return page; } |