Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754072AbYLQPq5 (ORCPT ); Wed, 17 Dec 2008 10:46:57 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751442AbYLQPqr (ORCPT ); Wed, 17 Dec 2008 10:46:47 -0500 Received: from rtr.ca ([76.10.145.34]:40302 "EHLO mail.rtr.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751393AbYLQPqp (ORCPT ); Wed, 17 Dec 2008 10:46:45 -0500 Message-ID: <49491F32.1040400@rtr.ca> Date: Wed, 17 Dec 2008 10:48:02 -0500 From: Mark Lord Organization: Real-Time Remedies Inc. User-Agent: Thunderbird 2.0.0.18 (X11/20081125) MIME-Version: 1.0 To: Linux Kernel Cc: Greg KH , akpm@linux-foundation.org, rjw@sisk.pl, tglx@linutronix.de, rjw@sisk.pl, lenb@kernel.org, linux-pm@lists.linux-foundation.org, davej@redhat.com Subject: SMP poweroff hangs: it's baaaack! But on x86_64 this time. References: <20071213053020.E28AB1454450@imap.suse.de> <4761601B.3030408@rtr.ca> <20071213164545.GA10593@suse.de> <47616C15.7030604@rtr.ca> <47616F8C.9030803@rtr.ca> In-Reply-To: <47616F8C.9030803@rtr.ca> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > Subject: Fix SMP poweroff hangs > From: Mark Lord > > We need to disable all CPUs other than the boot CPU (usually 0) before > attempting to power-off modern SMP machines. This fixes the > hang-on-poweroff issue on my MythTV SMP box, and also on Thomas Gleixner's > new toybox. > > Signed-off-by: Mark Lord > Acked-by: Thomas Gleixner > Cc: "Rafael J. Wysocki" > Signed-off-by: Andrew Morton > --- > > kernel/sys.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff -puN kernel/sys.c~fix-smp-poweroff-hangs kernel/sys.c > --- a/kernel/sys.c~fix-smp-poweroff-hangs > +++ a/kernel/sys.c > @@ -32,6 +32,7 @@ > #include > #include > #include > +#include > > #include > #include > @@ -878,6 +879,7 @@ void kernel_power_off(void) > kernel_shutdown_prepare(SYSTEM_POWER_OFF); > if (pm_power_off_prepare) > pm_power_off_prepare(); > + disable_nonboot_cpus(); > sysdev_shutdown(); > printk(KERN_EMERG "Power down.\n"); > machine_power_off(); .. This bug has returned here now, but on x86_86 this time around. Same machine as before, just upgraded toa 64-bit kernel/user (2.6.27.9) from the original 32-bit kernel/user that was originally fixed (above). One hang at poweroff over the past 10 days. Not much, but enough to destroy confidence in "unattended" operation. I lack opportunity to dig further into the code for now, but just wanted to flag the problem, in case similar reports from others might already be out there. In the meanwhile, I'm experimenting with this simple patch, garnered from the 32-bit investigations last time around. We should know in a few weeks whether it has any effect or not. --- old/kernel/sys.c 2008-10-18 13:57:22.000000000 -0400 +++ linux-2.6.27.9/kernel/sys.c 2008-12-17 09:42:17.000000000 -0500 @@ -303,6 +303,8 @@ static void kernel_shutdown_prepare(enum system_states state) { + set_cpus_allowed(current, cpumask_of_cpu(first_cpu(cpu_online_map))); + disable_nonboot_cpus(); blocking_notifier_call_chain(&reboot_notifier_list, (state == SYSTEM_HALT)?SYS_HALT:SYS_POWER_OFF, NULL); system_state = state; @@ -333,7 +335,6 @@ kernel_shutdown_prepare(SYSTEM_POWER_OFF); if (pm_power_off_prepare) pm_power_off_prepare(); - disable_nonboot_cpus(); sysdev_shutdown(); printk(KERN_EMERG "Power down.\n"); machine_power_off(); -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/