Re: APM shutdown oops (Re: [stable] [patch 09/12] Fix SMP poweroff hangs)

2007-10-12 Thread Rafael J. Wysocki
On Friday, 12 October 2007 01:27, Adrian Bunk wrote: > On Thu, Oct 11, 2007 at 02:42:01PM -0400, Jeff Garzik wrote: > > Linus Torvalds wrote: > >> On Wed, 10 Oct 2007, Kevin wrote: > >>> The last kernel I used was 6.2.22 the "dmesg" the file is attached: > >>> > >>> dmesg 2.6.22 line 158 > apm: ov

Re: APM shutdown oops (Re: [stable] [patch 09/12] Fix SMP poweroff hangs)

2007-10-11 Thread Adrian Bunk
On Thu, Oct 11, 2007 at 02:42:01PM -0400, Jeff Garzik wrote: > Linus Torvalds wrote: >> On Wed, 10 Oct 2007, Kevin wrote: >>> The last kernel I used was 6.2.22 the "dmesg" the file is attached: >>> >>> dmesg 2.6.22 line 158 > apm: overridden by ACPI. >>> >>> dmesg, APM on, has no line > apm: overr

Re: ARPM shutdown oops (Re: [stable] [patch 09/12] Fix SMP poweroff hangs)

2007-10-11 Thread Rafael J. Wysocki
On Thursday, 11 October 2007 20:42, Jeff Garzik wrote: > Linus Torvalds wrote: > > > > On Wed, 10 Oct 2007, Kevin wrote: > >> The last kernel I used was 6.2.22 the "dmesg" the file is attached: > >> > >> dmesg 2.6.22 line 158 > apm: overridden by ACPI. > >> > >> dmesg, APM on, has no line > apm:

Re: ARPM shutdown oops (Re: [stable] [patch 09/12] Fix SMP poweroff hangs)

2007-10-11 Thread Dave Jones
On Thu, Oct 11, 2007 at 02:42:01PM -0400, Jeff Garzik wrote: > Linus Torvalds wrote: > > > > And I think I see the problem: it's a config change. You don't have > > PM_LEGACY enabled. Your config file diff shows: > > > >-CONFIG_PM_LEGACY=y > >+# CONFIG_PM_LEGACY is not set > >

Re: ARPM shutdown oops (Re: [stable] [patch 09/12] Fix SMP poweroff hangs)

2007-10-11 Thread Jeff Garzik
Linus Torvalds wrote: On Wed, 10 Oct 2007, Kevin wrote: The last kernel I used was 6.2.22 the "dmesg" the file is attached: dmesg 2.6.22 line 158 > apm: overridden by ACPI. dmesg, APM on, has no line > apm: overridden by ACPI. Ok, this is the real reason. The APM code does: if

Re: ARPM shutdown oops (Re: [stable] [patch 09/12] Fix SMP poweroff hangs)

2007-10-11 Thread Linus Torvalds
On Wed, 10 Oct 2007, Kevin wrote: > > The last kernel I used was 6.2.22 the "dmesg" the file is attached: > > dmesg 2.6.22 line 158 > apm: overridden by ACPI. > > dmesg, APM on, has no line > apm: overridden by ACPI. Ok, this is the real reason. The APM code does: if (PM_IS_ACTIVE

Re: [stable] [patch 09/12] Fix SMP poweroff hangs

2007-10-10 Thread Paul Mackerras
Linus Torvalds writes: > On Wed, 10 Oct 2007, Thomas Gleixner wrote: > > > > Wrapping it into a #ifdef CONFIG_X86 would be sufficient. > > Well, the ppc oops seems to be a ppc bug regardless. Sure. And Milton and Olof have figured out what the problem is and proposed patches to fix it. Howeve

Re: ARPM shutdown oops (Re: [stable] [patch 09/12] Fix SMP poweroff hangs)

2007-10-10 Thread H. Peter Anvin
Linus Torvalds wrote: We did have some APM *detection* changes, and maybe APM wasn't even detected for you before, or it was detected differently. That would be due to the bootup changes, I'm Cc'ing Peter Anvin (and the kernel mailing list, in case somebody else see a pattern to this). We

ARPM shutdown oops (Re: [stable] [patch 09/12] Fix SMP poweroff hangs)

2007-10-10 Thread Linus Torvalds
On Tue, 9 Oct 2007, Kevin wrote: > > I don't own a digital camera but I did jot the info down by hand. Heh, yeah, that's what I used to do too (and still do if a camera isn't handy). > Call Trace: > [] apm_bios_call_simple+0x92/0x110 > [] acpi_hw_clear_gpe_block+00/0x32 > [] set_system_power_

Re: [stable] [patch 09/12] Fix SMP poweroff hangs

2007-10-10 Thread Rafael J. Wysocki
On Wednesday, 10 October 2007 12:06, Milton Miller wrote: > > On Tue, Oct 9 2007 Thomas Gleixner wrote: > >On Tue, 9 Oct 2007, Linus Torvalds wrote: > >>On Wed, 10 Oct 2007, Thomas Gleixner wrote: > >>> > >>> Wrapping it into a #ifdef CONFIG_X86 would be sufficient. > >> > >> Well, the ppc oops se

Re: [stable] [patch 09/12] Fix SMP poweroff hangs

2007-10-10 Thread Milton Miller
On Tue, Oct 9 2007 Thomas Gleixner wrote: >On Tue, 9 Oct 2007, Linus Torvalds wrote: >>On Wed, 10 Oct 2007, Thomas Gleixner wrote: >>> >>> Wrapping it into a #ifdef CONFIG_X86 would be sufficient. >> >> Well, the ppc oops seems to be a ppc bug regardless. >> >> If CPU_HOTPLUG isn't defined, the th

Re: [stable] [patch 09/12] Fix SMP poweroff hangs

2007-10-09 Thread Linus Torvalds
On Wed, 10 Oct 2007, Thomas Gleixner wrote: > > Fair enough. OTOH for the affected PPC users it's a regression and that's > what I'm concerned of. Hmm.. I just got the appended (Kevin, better not just send me email, other people can be interested too): From: Kevin <[EMAIL PROTECTED]>: Subjec

Re: [stable] [patch 09/12] Fix SMP poweroff hangs

2007-10-09 Thread Olof Johansson
On Tue, Oct 09, 2007 at 04:27:06PM -0700, Linus Torvalds wrote: > > > On Wed, 10 Oct 2007, Thomas Gleixner wrote: > > > > Wrapping it into a #ifdef CONFIG_X86 would be sufficient. > > Well, the ppc oops seems to be a ppc bug regardless. > > If CPU_HOTPLUG isn't defined, the thing does nothing.

Re: [stable] [patch 09/12] Fix SMP poweroff hangs

2007-10-09 Thread Thomas Gleixner
On Tue, 9 Oct 2007, Linus Torvalds wrote: > > > On Wed, 10 Oct 2007, Thomas Gleixner wrote: > > > > Wrapping it into a #ifdef CONFIG_X86 would be sufficient. > > Well, the ppc oops seems to be a ppc bug regardless. > > If CPU_HOTPLUG isn't defined, the thing does nothing. And if it is > defin

Re: [stable] [patch 09/12] Fix SMP poweroff hangs

2007-10-09 Thread Greg KH
On Wed, Oct 10, 2007 at 01:21:18AM +0200, Thomas Gleixner wrote: > > > On Tue, 9 Oct 2007, Greg KH wrote: > > > On Tue, Oct 09, 2007 at 10:17:02AM -0500, Olof Johansson wrote: > > > On Mon, Oct 08, 2007 at 11:06:33AM -0700, Greg KH wrote: > > > > From: Mark Lord <[EMAIL PROTECTED]> > > > > > >

Re: [stable] [patch 09/12] Fix SMP poweroff hangs

2007-10-09 Thread Linus Torvalds
On Wed, 10 Oct 2007, Thomas Gleixner wrote: > > Wrapping it into a #ifdef CONFIG_X86 would be sufficient. Well, the ppc oops seems to be a ppc bug regardless. If CPU_HOTPLUG isn't defined, the thing does nothing. And if it is defined, I don't see why/how ppc can validly oops. So I think the f

Re: [stable] [patch 09/12] Fix SMP poweroff hangs

2007-10-09 Thread Thomas Gleixner
On Tue, 9 Oct 2007, Greg KH wrote: > On Tue, Oct 09, 2007 at 10:17:02AM -0500, Olof Johansson wrote: > > On Mon, Oct 08, 2007 at 11:06:33AM -0700, Greg KH wrote: > > > From: Mark Lord <[EMAIL PROTECTED]> > > > > > > commit 4047727e5ae33f9b8d2b7766d1994ea6e5ec2991 from upstream > > > > > > We n

Re: [stable] [patch 09/12] Fix SMP poweroff hangs

2007-10-09 Thread Greg KH
On Tue, Oct 09, 2007 at 10:17:02AM -0500, Olof Johansson wrote: > On Mon, Oct 08, 2007 at 11:06:33AM -0700, Greg KH wrote: > > From: Mark Lord <[EMAIL PROTECTED]> > > > > commit 4047727e5ae33f9b8d2b7766d1994ea6e5ec2991 from upstream > > > > We need to disable all CPUs other than the boot CPU (usu

Re: [patch 09/12] Fix SMP poweroff hangs

2007-10-09 Thread Olof Johansson
On Mon, Oct 08, 2007 at 11:06:33AM -0700, Greg KH wrote: > From: Mark Lord <[EMAIL PROTECTED]> > > commit 4047727e5ae33f9b8d2b7766d1994ea6e5ec2991 from upstream > > We need to disable all CPUs other than the boot CPU (usually 0) before > attempting to power-off modern SMP machines. This fixes th

[patch 09/12] Fix SMP poweroff hangs

2007-10-08 Thread Greg KH
From: Mark Lord <[EMAIL PROTECTED]> commit 4047727e5ae33f9b8d2b7766d1994ea6e5ec2991 from upstream 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 Gle