PM / hibernate: Untangle power_down()
The power_down() routine in the core hibernation code is not exactly straightforward (to put it lightly), so clean it up to make it avoid invoking itself recursively, among other things. Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
parent
00bb399882
commit
81d45bdf89
|
@ -608,6 +608,22 @@ static void power_down(void)
|
||||||
{
|
{
|
||||||
#ifdef CONFIG_SUSPEND
|
#ifdef CONFIG_SUSPEND
|
||||||
int error;
|
int error;
|
||||||
|
|
||||||
|
if (hibernation_mode == HIBERNATION_SUSPEND) {
|
||||||
|
error = suspend_devices_and_enter(PM_SUSPEND_MEM);
|
||||||
|
if (error) {
|
||||||
|
hibernation_mode = hibernation_ops ?
|
||||||
|
HIBERNATION_PLATFORM :
|
||||||
|
HIBERNATION_SHUTDOWN;
|
||||||
|
} else {
|
||||||
|
/* Restore swap signature. */
|
||||||
|
error = swsusp_unmark();
|
||||||
|
if (error)
|
||||||
|
pr_err("PM: Swap will be unusable! Try swapon -a.\n");
|
||||||
|
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
switch (hibernation_mode) {
|
switch (hibernation_mode) {
|
||||||
|
@ -620,25 +636,6 @@ static void power_down(void)
|
||||||
if (pm_power_off)
|
if (pm_power_off)
|
||||||
kernel_power_off();
|
kernel_power_off();
|
||||||
break;
|
break;
|
||||||
#ifdef CONFIG_SUSPEND
|
|
||||||
case HIBERNATION_SUSPEND:
|
|
||||||
error = suspend_devices_and_enter(PM_SUSPEND_MEM);
|
|
||||||
if (error) {
|
|
||||||
if (hibernation_ops)
|
|
||||||
hibernation_mode = HIBERNATION_PLATFORM;
|
|
||||||
else
|
|
||||||
hibernation_mode = HIBERNATION_SHUTDOWN;
|
|
||||||
power_down();
|
|
||||||
}
|
|
||||||
/*
|
|
||||||
* Restore swap signature.
|
|
||||||
*/
|
|
||||||
error = swsusp_unmark();
|
|
||||||
if (error)
|
|
||||||
printk(KERN_ERR "PM: Swap will be unusable! "
|
|
||||||
"Try swapon -a.\n");
|
|
||||||
return;
|
|
||||||
#endif
|
|
||||||
}
|
}
|
||||||
kernel_halt();
|
kernel_halt();
|
||||||
/*
|
/*
|
||||||
|
|
Loading…
Reference in New Issue