From owner-freebsd-arch Sat Mar 17 12:45:27 2001 Delivered-To: freebsd-arch@freebsd.org Received: from feral.com (feral.com [192.67.166.1]) by hub.freebsd.org (Postfix) with ESMTP id 658B237B719; Sat, 17 Mar 2001 12:45:24 -0800 (PST) (envelope-from mjacob@feral.com) Received: from zeppo.feral.com (IDENT:mjacob@zeppo [192.67.166.71]) by feral.com (8.9.3/8.9.3) with ESMTP id MAA28210; Sat, 17 Mar 2001 12:45:27 -0800 Date: Sat, 17 Mar 2001 12:45:23 -0800 (PST) From: Matthew Jacob Reply-To: mjacob@feral.com To: John Baldwin Cc: Warner Losh , arch@FreeBSD.ORG Subject: Re: man pages In-Reply-To: Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG > >: critical region of control > > ... > >: The driver may not call any functions external to itself in between > >: the time it calls ddi_enter_critical() and the time it calls > >: ddi_exit_critical(). > > ... > > > > Hmmm. That's what I need, with the above exception being too > > restrictive. I'd need to call functions that frob the hardware. :-) > > This would allow us to get rid of most of the (ab)uses of *_intr as it is right > now. If we were to use a more abstract name for this than just > intr_restore/disable, would critical_enter() and critical_exit() work? If I do > this, I may not even mess with the current set of foo_intr() functions as they > may turn out to no longer be useful. Some things like the ACPI code would need > to be taught to not assume cli/sti though. (Intel's ACPICA code is > Linux-centric in some places, but I think they are very open to fixes, so I > don't expect that will be a problem.) Personally, I like the 'intr' aspect- this is, in fact, what we are doing. ddi_enter_critical/ddi_exit_critical as an attempt to be a bit more generic, and the restriction against calling other functions is too restrictive now (IMO)- you could interpret this as prohibiting calls to the PROM, for instance. -matt To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message