[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200709252207.02532.rjw@sisk.pl>
Date: Tue, 25 Sep 2007 22:07:01 +0200
From: "Rafael J. Wysocki" <rjw@...k.pl>
To: Thomas Gleixner <tglx@...utronix.de>
Cc: Andi Kleen <ak@...e.de>, Andrew Morton <akpm@...ux-foundation.org>,
LKML <linux-kernel@...r.kernel.org>, Ingo Molnar <mingo@...e.hu>
Subject: Re: 2.6.23-rc4-mm1 and -rc6-mm1: boot failure on HP nx6325, related to clockevents
On Tuesday, 25 September 2007 15:17, Thomas Gleixner wrote:
> On Tue, 2007-09-25 at 15:16 +0200, Rafael J. Wysocki wrote:
[--snip--]
>
> I start to get desperate. Below is a patch, which moves the apic timer
> disable check after the calibration routine. Can you please apply on top
> of -hrt and add "noapictimer" to the command line ? Does it boot ?
2.6.23-rc7 with patch-2.6.23-rc7-hrt1.patch and the patch below applied boots
with noapictimer and doesn't boot without it.
Also, attached is the output of
# cat /proc/interrupts; sleep 10; cat /proc/interrupts
from the current mainline.
Greetings,
Rafael
> Index: linux-2.6.23-rc7/arch/x86_64/kernel/apic.c
> ===================================================================
> --- linux-2.6.23-rc7.orig/arch/x86_64/kernel/apic.c 2007-09-24 20:30:00.000000000 +0200
> +++ linux-2.6.23-rc7/arch/x86_64/kernel/apic.c 2007-09-25 15:05:32.000000000 +0200
> @@ -927,6 +927,7 @@ static void __init calibrate_APIC_clock(
>
> void __init setup_boot_APIC_clock (void)
> {
> +#if 0
> /*
> * The local apic timer can be disabled via the kernel commandline.
> * Register the lapic timer as a dummy clock event source on SMP
> @@ -940,7 +941,7 @@ void __init setup_boot_APIC_clock (void)
> setup_APIC_timer();
> return;
> }
> -
> +#endif
> printk(KERN_INFO "Using local APIC timer interrupts.\n");
> calibrate_APIC_clock();
>
> @@ -949,11 +950,13 @@ void __init setup_boot_APIC_clock (void)
> * PIT/HPET going. Otherwise register lapic as a dummy
> * device.
> */
> - if (nmi_watchdog != NMI_IO_APIC)
> + if (!disable_apic_timer && nmi_watchdog != NMI_IO_APIC)
> lapic_clockevent.features &= ~CLOCK_EVT_FEAT_DUMMY;
> +#if 0
> else
> printk(KERN_WARNING "APIC timer registered as dummy,"
> " due to nmi_watchdog=1!\n");
> +#endif
>
> setup_APIC_timer();
> }
>
>
>
>
--
"Premature optimization is the root of all evil." - Donald Knuth
View attachment "interrupts.txt" of type "text/x-log" (1396 bytes)
Powered by blists - more mailing lists