On Wednesday, 25 October 2006 18:35, Alexey Starikovskiy wrote: > Here is a least intrusive patch by my opinion... > Just copy pm_disk_mode from pm_ops supplied by platform. > ACPI sets it to PM_DISK_PLATFORM if it has S4, > and it should be zero otherwise, patch does not copy it then. Well, I think the appended patch is sufficient. If "platform" is unavailable it has no effect and works like "shutdown". Greetings, Rafael Signed-off-by: Rafael J. Wysocki <rjw@xxxxxxx> --- kernel/power/disk.c | 8 +++++--- kernel/power/main.c | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) Index: linux-2.6.19-rc2-mm2/kernel/power/main.c =================================================================== --- linux-2.6.19-rc2-mm2.orig/kernel/power/main.c +++ linux-2.6.19-rc2-mm2/kernel/power/main.c @@ -28,7 +28,7 @@ DECLARE_MUTEX(pm_sem); struct pm_ops *pm_ops; -suspend_disk_method_t pm_disk_mode = PM_DISK_SHUTDOWN; +suspend_disk_method_t pm_disk_mode = PM_DISK_PLATFORM; /** * pm_set_ops - Set the global power method table. Index: linux-2.6.19-rc2-mm2/kernel/power/disk.c =================================================================== --- linux-2.6.19-rc2-mm2.orig/kernel/power/disk.c +++ linux-2.6.19-rc2-mm2/kernel/power/disk.c @@ -61,9 +61,11 @@ static void power_down(suspend_disk_meth switch(mode) { case PM_DISK_PLATFORM: - kernel_shutdown_prepare(SYSTEM_SUSPEND_DISK); - error = pm_ops->enter(PM_SUSPEND_DISK); - break; + if (pm_ops && pm_ops->enter) { + kernel_shutdown_prepare(SYSTEM_SUSPEND_DISK); + error = pm_ops->enter(PM_SUSPEND_DISK); + break; + } case PM_DISK_SHUTDOWN: kernel_power_off(); break; - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html