Re: [REGRESSION][BISECTED][X86] next-20080526 hangs on boot

From: Cyrill Gorcunov
Date: Tue May 27 2008 - 23:22:16 EST


Hi,
thanks a lot for checking this!
I'll take a look today evening.

On 5/28/08, Adrian Bunk <bunk@xxxxxxxxxx> wrote:
> On Tue, May 27, 2008 at 10:35:43AM +0400, Cyrill Gorcunov wrote:
>> On 5/26/08, Sitsofe Wheeler <sitsofe@xxxxxxxxx> wrote:
>> > <posted & mailed>
>> >
>> > Sitsofe Wheeler wrote:
>> >
>> > > Cyrill Gorcunov wrote:
>> > >
>> > >> [Sitsofe Wheeler - Mon, May 26, 2008 at 03:04:54PM +0100]
>> > >> | When using a 32 bit linux-next-20080526 the bootup process will
>> > >> hang at
>> > >> | a random point (not even sysrq helps) with no additional output on
>> > >> the
>> > >> | screen (whereas linux-next-20080523 did boot). Mysteriously,
>> > >> booting
>> > >> | with nmi_watchdog=2 allows the boot to finish (booting with
>> > >> | nmi_watchdog=1 still stalls). I have bisected it down to commit
>> > >> | [d1b946b97d71423f365fa797d1428e1847c0bec1]:
>> > >>
>> > >> Hi, so it helps by reverting only that commit? I mean all further
>> > >> commits
>> > >> are still appiled?
>> > >
>> > > Ah that I hadn't tested. I believe I might need to revert
>> > > 4b82b277707a39b97271439c475f186f63ec4692 too if later commits are
>> > > applied
>> > > (but I'm still testing)
>> > >
>> > >> and, btw, could you post your config, please?
>> > >
>> > > http://sucs.org/~sits/test/config-20080526.txt
>> >
>> > OK applying the following patch (which is more or less a revert of
>> > [4b82b277707a39b97271439c475f186f63ec4692]) resolves the problem:
>>...
>>
>> Hi Sitsofe, actually I can't imagine how this revert have helped you
>> from hang. Newly introduced nmi_watchdog_default() didn't change
>> the logic of the code being used before this merge-series.
>>
>> Thomas, Ingo, maybe I've overlooked something (hard to believe ;)?
>> Only the change is the adding of new call for nmi_watchdog_default()
>> which was say 'inlined' before.
>>...
>
> arch/x86/kernel/smpboot.c:native_smp_prepare_cpus() also calls
> the no longer empty nmi_watchdog_default() on 32bit.
>
> cu
> Adrian
>
> --
>
> "Is there not promise of rain?" Ling Tan asked suddenly out
> of the darkness. There had been need of rain for many days.
> "Only a promise," Lao Er said.
> Pearl S. Buck - Dragon Seed
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/