From owner-cvs-all Sat Dec 2 20:15:55 2000 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 ACF8737B400; Sat, 2 Dec 2000 20:15:48 -0800 (PST) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.8.7/8.8.7) with ESMTP id PAA03069; Sun, 3 Dec 2000 15:15:28 +1100 Date: Sun, 3 Dec 2000 15:05:08 +1100 (EST) From: Bruce Evans X-Sender: bde@besplex.bde.org To: Matthew Jacob Cc: Alfred Perlstein , Doug Rabson , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/alpha/alpha clock.c db_interface.c interrupt.c mp_machdep.c prom.c vm_machdep.c In-Reply-To: Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG On Sun, 19 Nov 2000, Matthew Jacob wrote: > On Sun, 19 Nov 2000, Alfred Perlstein wrote: > > > * Doug Rabson [001119 04:29] wrote: > > > dfr 2000/11/19 04:28:42 PST > > > > > > Modified files: > > > sys/alpha/alpha clock.c db_interface.c interrupt.c > > > mp_machdep.c prom.c vm_machdep.c > > > Log: > > > Convert various calls to splhigh() to disable_intr() since splhigh() is > > > now a no-op. > > > > Afaik the concensus was that spl's were not to be removed even > > though they are no-ops. > > > > You should probably put them back. > > No, actually they're needed and they are there. Um, "they" (the spls) were replaced by disable_intr()/enable_intr(), so they are not there. > This is all specifically > machine dependent code areas, and places where, for example, the prom is > entered, interrupts really do need to be disabled. Disabling interrupts for one CPU is probably insufficient for the SMP case. The old code was broken if it actually required interrupts to be disabled. splhigh() might not disable interrupts. In fact, it can't disable interrupts if fast interrupts actually work. Similarly for new locking functions. Don't assume that mtx_enter() disables interrupts, although this is a documented (mis)feature for the MTX_SPIN case, since this is an implementation detail (bug). Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message