From owner-cvs-sys Fri Nov 21 10:16:34 1997 Return-Path: Received: (from root@localhost) by hub.freebsd.org (8.8.7/8.8.7) id KAA27963 for cvs-sys-outgoing; Fri, 21 Nov 1997 10:16:34 -0800 (PST) (envelope-from owner-cvs-sys) Received: from freefall.freebsd.org (freefall.FreeBSD.ORG [204.216.27.21]) by hub.freebsd.org (8.8.7/8.8.7) with ESMTP id KAA27757; Fri, 21 Nov 1997 10:14:21 -0800 (PST) (envelope-from bde@FreeBSD.org) From: Bruce Evans Received: (from bde@localhost) by freefall.freebsd.org (8.8.6/8.8.5) id KAA05962; Fri, 21 Nov 1997 10:14:02 -0800 (PST) Date: Fri, 21 Nov 1997 10:14:02 -0800 (PST) Message-Id: <199711211814.KAA05962@freefall.freebsd.org> To: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG, cvs-sys@FreeBSD.ORG Subject: cvs commit: src/sys/i386/isa isa.c src/sys/i386/i386 autoconf.c Sender: owner-cvs-sys@FreeBSD.ORG X-Loop: FreeBSD.org Precedence: bulk bde 1997/11/21 10:14:02 PST Modified files: sys/i386/isa isa.c sys/i386/i386 autoconf.c Log: Moved splhigh()/spl0() calls from isa_configure() to configure() so that there is a natural place to initialize `safepri' in a future commit. Spinoffs: - spl0() gets called in the unlikely event that isa is not configured. - configure() has better control over enabling interrupts. - it is now less unclear that interrupts aren't actually enabled early. Rev.1.48 of autoconf.c seems to have done the opposite of what was intended - moving the isa_configure() call delayed the spl0() side effect. Added some comments about the bogons. Removed the splhigh() call since it is a no-op. Revision Changes Path 1.107 +1 -3 src/sys/i386/isa/isa.c 1.81 +13 -1 src/sys/i386/i386/autoconf.c