From owner-freebsd-smp Wed Jun 21 8:28:59 2000 Delivered-To: freebsd-smp@freebsd.org Received: from apollo.backplane.com (apollo.backplane.com [216.240.41.2]) by hub.freebsd.org (Postfix) with ESMTP id CC1BE37B7B4 for ; Wed, 21 Jun 2000 08:28:56 -0700 (PDT) (envelope-from dillon@apollo.backplane.com) Received: (from dillon@localhost) by apollo.backplane.com (8.9.3/8.9.1) id IAA94804; Wed, 21 Jun 2000 08:28:49 -0700 (PDT) (envelope-from dillon) Date: Wed, 21 Jun 2000 08:28:49 -0700 (PDT) From: Matthew Dillon Message-Id: <200006211528.IAA94804@apollo.backplane.com> To: Bruce Evans Cc: Jason Evans , Greg Lehey , Warner Losh , The Hermit Hacker , freebsd-smp@FreeBSD.ORG Subject: Re: SP Patchset #1 up References: Sender: owner-freebsd-smp@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.org :These were more or less correct, and unrelated to sio. : :Note that the low level console drivers must not use any locks or switch :context, since they need to work in debugger traps which may occur with :locks in any state. Old reentrancy bugs in syscons will probably be :more obvious now. : :Bruce Yah. That's why I replaced them all with mtx_enter_sched_quick(), which is the closest equivalent to those requirements. The spl*() calls only work if you are holding GiantMutex, and that obviously is not necessarily the case for the kernel printf(). Believe me, I hit those problems early--- when I could ctl-alt-esc into DDB or when a panic would occur and not even print the panic message. I think I have those cases fixed in patch set #1. -Matt Matthew Dillon To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-smp" in the body of the message