From owner-cvs-all Tue Feb 5 11:27:30 2002 Delivered-To: cvs-all@freebsd.org Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by hub.freebsd.org (Postfix) with ESMTP id 0299C37B425; Tue, 5 Feb 2002 11:27:25 -0800 (PST) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id GAA31632; Wed, 6 Feb 2002 06:27:22 +1100 Date: Wed, 6 Feb 2002 06:29:49 +1100 (EST) From: Bruce Evans X-X-Sender: To: John Baldwin Cc: Bruce Evans , , Subject: RE: cvs commit: src/sys/i386/i386 db_interface.c In-Reply-To: Message-ID: <20020206061908.G260-100000@gamplex.bde.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Tue, 5 Feb 2002, John Baldwin wrote: > On 30-Jan-02 Bruce Evans wrote: > > bde 2002/01/30 10:23:32 PST > > > > Modified files: > > sys/i386/i386 db_interface.c > > Log: > > Backed out the last vestiges of rev.1.51. Don't enter a critical > > region in Debugger(), since rev.1.468 of machdep.c does this better. > > Other cosmetic backouts. > > Err, have you tested this? In my tests locally this still doesn't work > properly which is why I haven't committed it. :( PS: perhaps you are thinking of the flag in Debugger(). That is fundamentally broken too. It just protects against multiple entries to the debugger when the first entry is via the console debugger key. Try the following: - enter ddb using the key - hit the key again. No problem since the flag is set. - single step until the flag is unset - hit the key again. This causes a fine mess. The same thing mess happens when you enter ddb via a breakpoint and hit the key, except the console driver is not reentered then (unless the breakpoint is in the console driver). Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message