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

Jeremy Fitzhardinge
2007-06-25T12:50:14Z

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.)

J