From owner-cvs-src@FreeBSD.ORG Tue Dec 14 20:11:04 2004 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 1068916A4CE; Tue, 14 Dec 2004 20:11:04 +0000 (GMT) Received: from ylpvm29.prodigy.net (ylpvm29-ext.prodigy.net [207.115.57.60]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9AB7A43D7F; Tue, 14 Dec 2004 20:11:03 +0000 (GMT) (envelope-from nate@root.org) Received: from [10.0.0.34] (adsl-67-119-74-222.dsl.sntc01.pacbell.net [67.119.74.222])iBEKAw1s020792; Tue, 14 Dec 2004 15:10:58 -0500 Message-ID: <41BF48D4.8080305@root.org> Date: Tue, 14 Dec 2004 12:11:00 -0800 From: Nate Lawson User-Agent: Mozilla Thunderbird 1.0RC1 (X11/20041205) X-Accept-Language: en-us, en MIME-Version: 1.0 To: John Baldwin References: <200411300618.iAU6IkQX065609@repoman.freebsd.org> <200412141333.06213.jhb@FreeBSD.org> In-Reply-To: <200412141333.06213.jhb@FreeBSD.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/i386/i386 vm_machdep.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 14 Dec 2004 20:11:04 -0000 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