Re: [patch, 2.6.22-rc6] fix nmi_watchdog=2 bootup hang

Björn Steinbrink
2007-06-25T13:05:44Z

On 2007.06.25 08:49:05 -0400, Jeremy Fitzhardinge wrote:
> Björn Steinbrink wrote:
> >On 2007.06.25 10:26:52 +0200, Ingo Molnar wrote:
> >
> >>* Ingo Molnar <mingo@elte.hu> wrote:
> >>
> >>
> >>>the winner is ...
> >>>
> >>> f8822f42019eceed19cc6c0f985a489e17796ed8 is first bad commit
> >>> commit f8822f42019eceed19cc6c0f985a489e17796ed8
> >>> Author: Jeremy Fitzhardinge <jeremy@goop.org>
> >>> Date: Wed May 2 19:27:14 2007 +0200
> >>>
> >>> [PATCH] i386: PARAVIRT: Consistently wrap paravirt ops callsites to
> >>> make them patchable
> >>>
> >>and of course i'm happy to test any patch that is simpler than the
> >>brutal revert i sent.
> >>
> >
> >wrmsrl() looks broken, dropping the upper 32bits of the value to be
> >written. Does this help?
> >
>
> Crap. That's embarrassing. Does it help, because it seems likely?
> (Esp since Ingo didn't even have CONFIG_PARAVIRT enabled, so most of his
> revert would have been dead code anyway.)

He has. The config Ingo sent was for x86_64, which (AFAICT) doesn't have
CONFIG_PARAVIRT, so the config was unfortunately useless. But his
bootlog tells us:

Booting paravirtualized kernel on bare hardware

Björn