Date: Tue, 14 Dec 2004 12:11:00 -0800 From: Nate Lawson <nate@root.org> To: John Baldwin <jhb@FreeBSD.org> Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/i386/i386 vm_machdep.c Message-ID: <41BF48D4.8080305@root.org> In-Reply-To: <200412141333.06213.jhb@FreeBSD.org> References: <200411300618.iAU6IkQX065609@repoman.freebsd.org> <200412141333.06213.jhb@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
John Baldwin wrote: > On Tuesday 30 November 2004 01:18 am, Nate Lawson wrote: > >>njl 2004-11-30 06:18:46 UTC >> >> FreeBSD src repository >> >> Modified files: >> sys/i386/i386 vm_machdep.c >> Log: >> MFamd64: Remove the cpu_reset_proxy cruft now that we run boot() on >> cpu 0. Also, restructure cpu_reset to be cleaner (no functional change.) > > > 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(). -- Nate
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?41BF48D4.8080305>