cvs commit: src/sys/i386/i386 vm_machdep.c

[Nate Lawson]

Bruce Evans wrote:
> On Tue, 14 Dec 2004, Nate Lawson wrote:
>>John Baldwin wrote:
>>
>>>On Tuesday 14 December 2004 03:11 pm, Nate Lawson wrote:
>>>>John Baldwin wrote:
>>>>...
>>>>
>>>>>FYI, this breaks the 'reset' command from ddb if you panic on a cpu other
>>>>>than the BSP.  boot() isn't the only function that calls cpu_reset(), so
>>>>>perhaps this should be reverted (same for amd64)
>>>>
>>>>No, I think we should move forward instead of backward.  Entering the
>>>>debugger should happen on the BSP and possibly other cpus need to be
>>>>stopped by panic().
>>>
>>>Erm, well, that's not always easy since sometimes when you panic you can't
>>>talk to the other CPUs for whatever reason.  Putting back the proxy reset
> 
> The most common reason is that at least one other CPU is looping with
> interrupts disabled.  Then it won't see IPIs and stop_cpus() will loop
> forever.

Yes, the NMI change would fix this.  Please repost your ddb sync patch.

I'm ok with John putting the cpu proxy reset stuff back in for now. 
He's right in that it's the best we can do for now.

-- 
Nate