Re: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi()
* Ingo Molnar <[EMAIL PROTECTED]> wrote: > * Karsten Wiese <[EMAIL PROTECTED]> wrote: > > > Am Donnerstag, 7. Dezember 2006 03:28 schrieb Sergio Monteiro Basto: > > > On Wed, 2006-12-06 at 23:30 +0100, Ingo Molnar wrote: > > > > Subject: [patch] ACPI, i686, x86_64: fix laptop bootup hang in > > > > init_acpi() > > > > From: Ingo Molnar <[EMAIL PROTECTED]> > > > Hi Ingo, > > > Just curiosity ,have we this patch on 2.6.19-1.rt6 ? > > > > No, its in rt7. > > yeah, it's in -rt7. [ -rt6 is more than 1 day old now, so it's truly > ancient stuff ;-) ] did i really say -rt7? The ancient -rt7 stuff is 12+ hours old already, and there's the 1 minute old -rt8 available (patch and yum rpms too). Sheesh ... ;-) Ingo - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi()
* Len Brown <[EMAIL PROTECTED]> wrote: > c. disable the NMI whenever the ACPI interpeter is running >(who knows, maybe this isn't limited to the _INI case, but > could cause a hang at some other time -- only the > BIOS AML writers knows) i have tested this by forcing the NMI frequency to 10,000 per second, and never saw any other problem. So at least this particular laptop should be OK. So i /think/ this should be enough - the _INI case should be limited to bootup - or can it trigger during module load too? The IO-APIC based NMI watchdog should really only involve the southbridge (whose initialization package has this problem, in my guesstimation - do you agree?) and not random other devices - so once we have booted up we should be fine from this particular issue. acpi_nmi_disable()/enable() does a cross-IPI to all CPUs, so it can be quite heavy-handed - i'm not sure we want it for every interpreter invocation. Ingo - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi()
* Karsten Wiese <[EMAIL PROTECTED]> wrote: > Am Donnerstag, 7. Dezember 2006 03:28 schrieb Sergio Monteiro Basto: > > On Wed, 2006-12-06 at 23:30 +0100, Ingo Molnar wrote: > > > Subject: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi() > > > From: Ingo Molnar <[EMAIL PROTECTED]> > > Hi Ingo, > > Just curiosity ,have we this patch on 2.6.19-1.rt6 ? > > No, its in rt7. yeah, it's in -rt7. [ -rt6 is more than 1 day old now, so it's truly ancient stuff ;-) ] Ingo - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi()
Am Donnerstag, 7. Dezember 2006 03:28 schrieb Sergio Monteiro Basto: > On Wed, 2006-12-06 at 23:30 +0100, Ingo Molnar wrote: > > Subject: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi() > > From: Ingo Molnar <[EMAIL PROTECTED]> > Hi Ingo, > Just curiosity ,have we this patch on 2.6.19-1.rt6 ? No, its in rt7. Karsten - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi()
On Wed, 2006-12-06 at 23:30 +0100, Ingo Molnar wrote: > Subject: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi() > From: Ingo Molnar <[EMAIL PROTECTED]> Hi Ingo, Just curiosity ,have we this patch on 2.6.19-1.rt6 ? Thanks, -- Sérgio M.B. smime.p7s Description: S/MIME cryptographic signature
Re: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi()
On Wednesday 06 December 2006 17:30, Ingo Molnar wrote: > Subject: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi() > From: Ingo Molnar <[EMAIL PROTECTED]> > > during kernel bootup, a new T60 laptop (CoreDuo, 32-bit) hangs about > 10%-20% of the time in acpi_init(): > > Calling initcall 0xc055ce1a: topology_init+0x0/0x2f() > Calling initcall 0xc055d75e: mtrr_init_finialize+0x0/0x2c() > Calling initcall 0xc05664f3: param_sysfs_init+0x0/0x175() > Calling initcall 0xc014cb65: pm_sysrq_init+0x0/0x17() > Calling initcall 0xc0569f99: init_bio+0x0/0xf4() > Calling initcall 0xc056b865: genhd_device_init+0x0/0x50() > Calling initcall 0xc056c4bd: fbmem_init+0x0/0x87() > Calling initcall 0xc056dd74: acpi_init+0x0/0x1ee() > > it's a hard hang that not even an NMI could punch through! > Frustratingly, adding printks or function tracing to the ACPI code made > the hangs go away ... > > after some time an additional detail emerged: disabling the NMI watchdog > made these occasional hangs go away. > > So i spent the better part of today trying to debug this and trying out > various theories when i finally found the likely reason for the hang: if > acpi_ns_initialize_devices() executes an _INI AML method and an NMI > happens to hit that AML execution in the wrong moment, the machine would > hang. (my theory is that this must be some sort of chipset setup method > doing stores to chipset mmio registers?) > > Unfortunately given the characteristics of the hang it was sheer > impossible to figure out which of the numerous AML methods is impacted > by this problem. > > as a workaround i wrote an interface to disable chipset-based NMIs while > executing _INI sections - and indeed this fixed the hang. I did a > boot-loop of 100 separate reboots and none hung - while without the > patch it would hang every 5-10 attempts. Out of caution i did not touch > the nmi_watchdog=2 case (it's not related to the chipset anyway and > didnt hang). > > I implemented this for both x86_64 and i686, tested the i686 laptop both > with nmi_watchdog=1 [which triggered the hangs] and nmi_watchdog=2, and > tested an Athlon64 box with the 64-bit kernel as well. Everything builds > and works with the patch applied. Good work root-causing this failure! Personally I have never been a big fan of having the NMI watchdog running by default on all systems -- but Andi insists that it helps him debug failures, so tick it does... Clearly this laptop was validated with Windows, and Windows doesn't have this problem -- suggesting that we may be somewhat out on a limb... Some alternatives to consider... a. don't enable NMI watchdog by default b. enable NMI watchdog, but later during kernel boot (assumption here that the issue is only during _INI) c. disable the NMI whenever the ACPI interpeter is running (who knows, maybe this isn't limited to the _INI case, but could cause a hang at some other time -- only the BIOS AML writers knows) thoughts? -Len > Signed-off-by: Ingo Molnar <[EMAIL PROTECTED]> > --- > arch/i386/kernel/nmi.c | 28 > arch/x86_64/kernel/nmi.c| 27 +++ > drivers/acpi/namespace/nsinit.c |9 + > include/linux/nmi.h |9 - > 4 files changed, 72 insertions(+), 1 deletion(-) > > Index: linux/arch/i386/kernel/nmi.c > === > --- linux.orig/arch/i386/kernel/nmi.c > +++ linux/arch/i386/kernel/nmi.c > @@ -376,6 +376,34 @@ void enable_timer_nmi_watchdog(void) > } > } > > +static void __acpi_nmi_disable(void *__unused) > +{ > + apic_write_around(APIC_LVT0, APIC_DM_NMI | APIC_LVT_MASKED); > +} > + > +/* > + * Disable timer based NMIs on all CPUs: > + */ > +void acpi_nmi_disable(void) > +{ > + if (atomic_read(&nmi_active) && nmi_watchdog == NMI_IO_APIC) > + on_each_cpu(__acpi_nmi_disable, NULL, 0, 1); > +} > + > +static void __acpi_nmi_enable(void *__unused) > +{ > + apic_write_around(APIC_LVT0, APIC_DM_NMI); > +} > + > +/* > + * Enable timer based NMIs on all CPUs: > + */ > +void acpi_nmi_enable(void) > +{ > + if (atomic_read(&nmi_active) && nmi_watchdog == NMI_IO_APIC) > + on_each_cpu(__acpi_nmi_enable, NULL, 0, 1); > +} > + > #ifdef CONFIG_PM > > static int nmi_pm_active; /* nmi_active before suspend */ > Index: linux/arch/x86_64/kernel/nmi.c > === > --- linux.orig/arch/x86_64/kernel/nmi.c > +++ linux/arch/x86_64/kernel/nmi.c > @@ -360,6 +360
[patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi()
Subject: [patch] ACPI, i686, x86_64: fix laptop bootup hang in init_acpi() From: Ingo Molnar <[EMAIL PROTECTED]> during kernel bootup, a new T60 laptop (CoreDuo, 32-bit) hangs about 10%-20% of the time in acpi_init(): Calling initcall 0xc055ce1a: topology_init+0x0/0x2f() Calling initcall 0xc055d75e: mtrr_init_finialize+0x0/0x2c() Calling initcall 0xc05664f3: param_sysfs_init+0x0/0x175() Calling initcall 0xc014cb65: pm_sysrq_init+0x0/0x17() Calling initcall 0xc0569f99: init_bio+0x0/0xf4() Calling initcall 0xc056b865: genhd_device_init+0x0/0x50() Calling initcall 0xc056c4bd: fbmem_init+0x0/0x87() Calling initcall 0xc056dd74: acpi_init+0x0/0x1ee() it's a hard hang that not even an NMI could punch through! Frustratingly, adding printks or function tracing to the ACPI code made the hangs go away ... after some time an additional detail emerged: disabling the NMI watchdog made these occasional hangs go away. So i spent the better part of today trying to debug this and trying out various theories when i finally found the likely reason for the hang: if acpi_ns_initialize_devices() executes an _INI AML method and an NMI happens to hit that AML execution in the wrong moment, the machine would hang. (my theory is that this must be some sort of chipset setup method doing stores to chipset mmio registers?) Unfortunately given the characteristics of the hang it was sheer impossible to figure out which of the numerous AML methods is impacted by this problem. as a workaround i wrote an interface to disable chipset-based NMIs while executing _INI sections - and indeed this fixed the hang. I did a boot-loop of 100 separate reboots and none hung - while without the patch it would hang every 5-10 attempts. Out of caution i did not touch the nmi_watchdog=2 case (it's not related to the chipset anyway and didnt hang). I implemented this for both x86_64 and i686, tested the i686 laptop both with nmi_watchdog=1 [which triggered the hangs] and nmi_watchdog=2, and tested an Athlon64 box with the 64-bit kernel as well. Everything builds and works with the patch applied. Signed-off-by: Ingo Molnar <[EMAIL PROTECTED]> --- arch/i386/kernel/nmi.c | 28 arch/x86_64/kernel/nmi.c| 27 +++ drivers/acpi/namespace/nsinit.c |9 + include/linux/nmi.h |9 - 4 files changed, 72 insertions(+), 1 deletion(-) Index: linux/arch/i386/kernel/nmi.c === --- linux.orig/arch/i386/kernel/nmi.c +++ linux/arch/i386/kernel/nmi.c @@ -376,6 +376,34 @@ void enable_timer_nmi_watchdog(void) } } +static void __acpi_nmi_disable(void *__unused) +{ + apic_write_around(APIC_LVT0, APIC_DM_NMI | APIC_LVT_MASKED); +} + +/* + * Disable timer based NMIs on all CPUs: + */ +void acpi_nmi_disable(void) +{ + if (atomic_read(&nmi_active) && nmi_watchdog == NMI_IO_APIC) + on_each_cpu(__acpi_nmi_disable, NULL, 0, 1); +} + +static void __acpi_nmi_enable(void *__unused) +{ + apic_write_around(APIC_LVT0, APIC_DM_NMI); +} + +/* + * Enable timer based NMIs on all CPUs: + */ +void acpi_nmi_enable(void) +{ + if (atomic_read(&nmi_active) && nmi_watchdog == NMI_IO_APIC) + on_each_cpu(__acpi_nmi_enable, NULL, 0, 1); +} + #ifdef CONFIG_PM static int nmi_pm_active; /* nmi_active before suspend */ Index: linux/arch/x86_64/kernel/nmi.c === --- linux.orig/arch/x86_64/kernel/nmi.c +++ linux/arch/x86_64/kernel/nmi.c @@ -360,6 +360,33 @@ void enable_timer_nmi_watchdog(void) } } +static void __acpi_nmi_disable(void *__unused) +{ + apic_write(APIC_LVT0, APIC_DM_NMI | APIC_LVT_MASKED); +} + +/* + * Disable timer based NMIs on all CPUs: + */ +void acpi_nmi_disable(void) +{ + if (atomic_read(&nmi_active) && nmi_watchdog == NMI_IO_APIC) + on_each_cpu(__acpi_nmi_disable, NULL, 0, 1); +} + +static void __acpi_nmi_enable(void *__unused) +{ + apic_write(APIC_LVT0, APIC_DM_NMI); +} + +/* + * Enable timer based NMIs on all CPUs: + */ +void acpi_nmi_enable(void) +{ + if (atomic_read(&nmi_active) && nmi_watchdog == NMI_IO_APIC) + on_each_cpu(__acpi_nmi_enable, NULL, 0, 1); +} #ifdef CONFIG_PM static int nmi_pm_active; /* nmi_active before suspend */ Index: linux/drivers/acpi/namespace/nsinit.c === --- linux.orig/drivers/acpi/namespace/nsinit.c +++ linux/drivers/acpi/namespace/nsinit.c @@ -45,6 +45,7 @@ #include #include #include +#include #define _COMPONENT ACPI_NAMESPACE ACPI_MODULE_NAME("nsinit") @@ -537,7 +538,15 @@ acpi_ns_init_one_device(acpi_handle obj_ info->parameter_type = ACPI_PARAM_ARGS; info->flags = ACPI_IGNOR