lkml.org 
[lkml]   [2010]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[066/145] PM / Hibernate: Fix preallocating of memory
    2.6.32-stable review patch.  If anyone has any objections, please let me know.

    ----------------
    From: Rafael J. Wysocki <rjw@sisk.pl>

    commit a9c9b4429df437982d2fbfab1f4b46b01329e9ed upstream.

    The hibernate memory preallocation code allocates memory to push some
    user space data out of physical RAM, so that the hibernation image is
    not too large. It allocates more memory than necessary for creating
    the image, so it has to release some pages to make room for
    allocations made while suspending devices and disabling nonboot CPUs,
    or the system will hang due to the lack of free pages to allocate
    from. Unfortunately, the function used for freeing these pages,
    free_unnecessary_pages(), contains a bug that prevents it from doing
    the job on all systems without highmem.

    Fix this problem, which is a regression from the 2.6.30 kernel, by
    using the right condition for the termination of the loop in
    free_unnecessary_pages().

    Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
    Reported-and-tested-by: Alan Jenkins <sourcejedi.lkml@googlemail.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    kernel/power/snapshot.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    --- a/kernel/power/snapshot.c
    +++ b/kernel/power/snapshot.c
    @@ -1181,7 +1181,7 @@ static void free_unnecessary_pages(void)

    memory_bm_position_reset(&copy_bm);

    - while (to_free_normal > 0 && to_free_highmem > 0) {
    + while (to_free_normal > 0 || to_free_highmem > 0) {
    unsigned long pfn = memory_bm_next_pfn(&copy_bm);
    struct page *page = pfn_to_page(pfn);




    \
     
     \ /
      Last update: 2010-03-13 01:55    [W:0.026 / U:1.080 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site