Skip to content
  • Srivatsa S. Bhat's avatar
    PM / Freezer: Thaw only kernel threads if freezing of kernel threads fails · 379e0be8
    Srivatsa S. Bhat authored
    
    
    If freezing of kernel threads fails, we are expected to automatically
    thaw tasks in the error recovery path. However, at times, we encounter
    situations in which we would like the automatic error recovery path
    to thaw only the kernel threads, because we want to be able to do
    some more cleanup before we thaw userspace. Something like:
    
    error = freeze_kernel_threads();
    if (error) {
    	/* Do some cleanup */
    
    	/* Only then thaw userspace tasks*/
    	thaw_processes();
    }
    
    An example of such a situation is where we freeze/thaw filesystems
    during suspend/hibernation. There, if freezing of kernel threads
    fails, we would like to thaw the frozen filesystems before thawing
    the userspace tasks.
    
    So, modify freeze_kernel_threads() to thaw only kernel threads in
    case of freezing failure. And change suspend_freeze_processes()
    accordingly. (At the same time, let us also get rid of the rather
    cryptic usage of the conditional operator (:?) in that function.)
    
    [rjw: In fact, this patch fixes a regression introduced during the
     3.3 merge window, because without it thaw_processes() may be called
     before swsusp_free() in some situations and that may lead to massive
     memory allocation failures.]
    
    Signed-off-by: default avatarSrivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com>
    Acked-by: default avatarTejun Heo <tj@kernel.org>
    Acked-by: default avatarNigel Cunningham <nigel@tuxonice.net>
    Signed-off-by: default avatarRafael J. Wysocki <rjw@sisk.pl>
    379e0be8