From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shaohua Li Subject: [RFC] change suspend_finish order slightly Date: Fri, 02 Dec 2005 19:15:48 -0800 Message-ID: <1133579748.7835.6.camel@linux.site> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============062723711486388556==" Return-path: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.osdl.org Errors-To: linux-pm-bounces@lists.osdl.org To: linux-pm@lists.osdl.org List-Id: linux-pm@vger.kernel.org --===============062723711486388556== Content-Type: text/plain Content-Transfer-Encoding: 7bit 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 --- linux-2.6.14/kernel/power/main.c~smp_s3_order 2005-11-03 14:18:29.000000000 +0800 +++ linux-2.6.14-root/kernel/power/main.c 2005-11-03 14:21:05.000000000 +0800 @@ -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(); } _ --===============062723711486388556== Content-Type: text/plain; charset="iso-8859-1" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline --===============062723711486388556==--