From owner-cvs-all Wed Feb 6 1:40:59 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 D2EA337B420; Wed, 6 Feb 2002 01:40:43 -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 UAA29459; Wed, 6 Feb 2002 20:40:41 +1100 Date: Wed, 6 Feb 2002 20:43:10 +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: <20020206201716.B2295-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 05-Feb-02 Bruce Evans wrote: > > On Tue, 5 Feb 2002, John Baldwin wrote: > >> 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 > > ... > > Hmm, the problem I was having is that interrupts were still firing while I was > in ddb. I could tell because new KTR entries due to clock interrutps kept > showing up. This can't happen :-). Except for bugs which would affect most forms of interrupt disablement. Perhaps there is a path through trap() which enables interrupts even for debugger traps, but only when they are enabled when the trap occurs. Ah, I see a related broken path, not for debugger traps but for pagefaults. Interrupts are enabled for pagefaults almost unconditionally, so a pagefault in ddb would cause problems. I think this causes the "Context switches not allowed in the Debugger" message. I thought that this was caused by a more fundamental bug. BTW, why does the spinlocking in kern_clock.c use MTX_QUIET? nanotime() for witness timestamps should work normally there. I noticed this when I uninlined mtx_*_spin_*(). The `flags' versions are not used anywhere else. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message