change suspend_finish order slightly

The BIOS guys here require ACPI _WAK method which is called in pm_ops->finish is invoked after all CPUs are resumed. Detail info can be found at http://bugzilla.kernel.org/show_bug.cgi?id=5651. To do this, this patch tries to change the order slightly. Does this break other platforms?

 linux-2.6.14-root/kernel/power/main.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff -puN kernel/power/main.c~smp_s3_order kernel/power/main.c

kernel/power/main.c renamed from kernel/power/main.c~smp_s3_order +2 -2

@@ -133,10 +133,10 @@ static int suspend_enter(suspend_state_t
 static void suspend_finish(suspend_state_t state)
 {
 	device_resume();
-	if (pm_ops && pm_ops->finish)
-		pm_ops->finish(state);
 	thaw_processes();
 	enable_nonboot_cpus();
+	if (pm_ops && pm_ops->finish)
+		pm_ops->finish(state);
 	pm_restore_console();
 }
 

_

The BIOS guys here require ACPI _WAK method which is called in pm_ops->finish is invoked after all CPUs are resumed. Detail info can be found at http://bugzilla.kernel.org/show_bug.cgi?id=5651. To do this, this patch tries to change the order slightly. Does this break other platforms?

Thanks,
Shaohua
---

 linux-2.6.14-root/kernel/power/main.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff -puN kernel/power/main.c~smp_s3_order kernel/power/main.c

kernel/power/main.c renamed from kernel/power/main.c~smp_s3_order +2 -2

@@ -133,10 +133,10 @@ static int suspend_enter(suspend_state_t
 static void suspend_finish(suspend_state_t state)
 {
 	device_resume();
-	if (pm_ops && pm_ops->finish)
-		pm_ops->finish(state);
 	thaw_processes();
 	enable_nonboot_cpus();
+	if (pm_ops && pm_ops->finish)
+		pm_ops->finish(state);
 	pm_restore_console();
 }
 

_

______________________________________________________________________
_______________________________________________
linux-pm mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/linux-pm