[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080527225042.GG11310@cs181133002.pp.htv.fi>
Date: Wed, 28 May 2008 01:50:42 +0300
From: Adrian Bunk <bunk@...nel.org>
To: Cyrill Gorcunov <gorcunov@...il.com>
Cc: Sitsofe Wheeler <sitsofe@...oo.com>, linux-kernel@...r.kernel.org,
kernel-testers@...r.kernel.org,
Thomas Gleixner <tglx@...utronix.de>,
Ingo Molnar <mingo@...e.hu>
Subject: Re: [REGRESSION][BISECTED][X86] next-20080526 hangs on boot
On Tue, May 27, 2008 at 10:35:43AM +0400, Cyrill Gorcunov wrote:
> On 5/26/08, Sitsofe Wheeler <sitsofe@...oo.com> 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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists