lkml.org 
[lkml]   [2012]   [Jan]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v2 3/4] PM/Hibernate: Thaw kernel threads in hibernation_snapshot() in error/test path
    Date
    In the hibernation call path, the kernel threads are frozen inside
    hibernation_snapshot(). If we happen to encounter an error further down
    the road or if we are exiting early due to a successful freezer test,
    then thaw kernel threads before returning to the caller.

    Signed-off-by: Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com>
    ---

    kernel/power/hibernate.c | 9 ++++++---
    kernel/power/user.c | 8 ++------
    2 files changed, 8 insertions(+), 9 deletions(-)

    diff --git a/kernel/power/hibernate.c b/kernel/power/hibernate.c
    index 1670387..0ce1220 100644
    --- a/kernel/power/hibernate.c
    +++ b/kernel/power/hibernate.c
    @@ -343,13 +343,13 @@ int hibernation_snapshot(int platform_mode)
    * successful freezer test.
    */
    freezer_test_done = true;
    - goto Cleanup;
    + goto Thaw;
    }

    error = dpm_prepare(PMSG_FREEZE);
    if (error) {
    dpm_complete(PMSG_RECOVER);
    - goto Cleanup;
    + goto Thaw;
    }

    suspend_console();
    @@ -385,6 +385,8 @@ int hibernation_snapshot(int platform_mode)
    platform_end(platform_mode);
    return error;

    + Thaw:
    + thaw_kernel_threads();
    Cleanup:
    swsusp_free();
    goto Close;
    @@ -655,12 +657,13 @@ int hibernate(void)
    pr_debug("PM: Image restored successfully.\n");
    }

    - Thaw:
    /*
    * Kernel threads had been frozen inside hibernation_snapshot().
    * So thaw them here, on the success path.
    */
    thaw_kernel_threads();
    +
    + Thaw:
    thaw_processes();
    Finish:
    free_basic_memory_bitmaps();
    diff --git a/kernel/power/user.c b/kernel/power/user.c
    index eaf5c97..62dbe93 100644
    --- a/kernel/power/user.c
    +++ b/kernel/power/user.c
    @@ -257,16 +257,12 @@ static long snapshot_ioctl(struct file *filp, unsigned int cmd,
    }
    pm_restore_gfp_mask();
    error = hibernation_snapshot(data->platform_support);
    - if (error) {
    - thaw_kernel_threads();
    - } else {
    + if (!error) {
    error = put_user(in_suspend, (int __user *)arg);
    if (!error && !freezer_test_done)
    data->ready = 1;
    - if (freezer_test_done) {
    + if (freezer_test_done)
    freezer_test_done = false;
    - thaw_kernel_threads();
    - }
    }
    break;



    \
     
     \ /
      Last update: 2012-01-31 00:17    [W:3.057 / U:0.692 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site