diff options
author | Stanislaw Gruszka <sgruszka@redhat.com> | 2012-01-10 18:07:31 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-01-10 19:30:42 -0500 |
commit | c6968e73b90c2a2fb9a32d4bad249f8f70f70125 (patch) | |
tree | 67ec90fc8e187fcf316298dd9d514bbbbbc8f187 /kernel/power | |
parent | c0a32fc5a2e470d0b02597b23ad79a317735253e (diff) |
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>
Diffstat (limited to 'kernel/power')
-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 cbe2c1441392..1cf88900ec4f 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) | |||
858 | PageReserved(page)) | 858 | PageReserved(page)) |
859 | return NULL; | 859 | return NULL; |
860 | 860 | ||
861 | if (page_is_guard(page)) | ||
862 | return NULL; | ||
863 | |||
861 | return page; | 864 | return page; |
862 | } | 865 | } |
863 | 866 | ||
@@ -920,6 +923,9 @@ static struct page *saveable_page(struct zone *zone, unsigned long pfn) | |||
920 | && (!kernel_page_present(page) || pfn_is_nosave(pfn))) | 923 | && (!kernel_page_present(page) || pfn_is_nosave(pfn))) |
921 | return NULL; | 924 | return NULL; |
922 | 925 | ||
926 | if (page_is_guard(page)) | ||
927 | return NULL; | ||
928 | |||
923 | return page; | 929 | return page; |
924 | } | 930 | } |
925 | 931 | ||