From owner-cvs-all Mon Apr 1 15:51:31 2002 Delivered-To: cvs-all@freebsd.org Received: from freefall.freebsd.org (freefall.FreeBSD.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id 3CCFE37B417; Mon, 1 Apr 2002 15:51:24 -0800 (PST) Received: (from dillon@localhost) by freefall.freebsd.org (8.11.6/8.11.6) id g31NpO890339; Mon, 1 Apr 2002 15:51:24 -0800 (PST) (envelope-from dillon) Message-Id: <200204012351.g31NpO890339@freefall.freebsd.org> From: Matt Dillon Date: Mon, 1 Apr 2002 15:51:24 -0800 (PST) To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/i386/i386 critical.c src/sys/i386/include cpufunc.h critical.h src/sys/i386/isa apic_vector.s icu_vector.s src/sys/kern kern_fork.c kern_proc.c kern_switch.c src/sys/alpha/alpha critical.c src/sys/alpha/include cpufunc.h ... X-FreeBSD-CVS-Branch: HEAD 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 dillon 2002/04/01 15:51:24 PST Modified files: sys/i386/i386 critical.c sys/i386/include cpufunc.h sys/i386/isa apic_vector.s icu_vector.s sys/kern kern_fork.c kern_proc.c kern_switch.c sys/alpha/alpha critical.c sys/alpha/include cpufunc.h sys/ia64/ia64 critical.c sys/ia64/include cpufunc.h sys/powerpc/include cpufunc.h sys/powerpc/powerpc critical.c sys/sparc64/include cpufunc.h sys/sparc64/sparc64 critical.c Added files: sys/i386/include critical.h sys/alpha/include critical.h sys/ia64/include critical.h sys/powerpc/include critical.h sys/sparc64/include critical.h Log: Stage-2 commit of the critical*() code. This re-inlines cpu_critical_enter() and cpu_critical_exit() and moves associated critical prototypes into their own header file, //critical.h, which is only included by the three MI source files that need it. Backout and re-apply improperly comitted syntactical cleanups made to files that were still under active development. Backout improperly comitted program structure changes that moved localized declarations to the top of two procedures. Partially re-apply one of the program structure changes to move 'mask' into an intermediate block rather then in three separate sub-blocks to make the code more readable. Re-integrate bug fixes that Jake made to the sparc64 code. Note: In general, developers should not gratuitously move declarations out of sub-blocks. They are where they are for reasons of structure, grouping, readability, compiler-localizability, and to avoid developer-introduced bugs similar to several found in recent years in the VFS and VM code. Reviewed by: jake Revision Changes Path 1.3 +1 -18 src/sys/alpha/alpha/critical.c 1.15 +0 -5 src/sys/alpha/include/cpufunc.h 1.1 +72 -0 src/sys/alpha/include/critical.h (new) 1.3 +19 -74 src/sys/i386/i386/critical.c 1.119 +0 -4 src/sys/i386/include/cpufunc.h 1.1 +111 -0 src/sys/i386/include/critical.h (new) 1.81 +2 -2 src/sys/i386/isa/apic_vector.s 1.36 +2 -2 src/sys/i386/isa/icu_vector.s 1.3 +1 -18 src/sys/ia64/ia64/critical.c 1.12 +0 -5 src/sys/ia64/include/cpufunc.h 1.1 +73 -0 src/sys/ia64/include/critical.h (new) 1.143 +1 -0 src/sys/kern/kern_fork.c 1.125 +1 -0 src/sys/kern/kern_proc.c 1.25 +1 -0 src/sys/kern/kern_switch.c 1.12 +0 -6 src/sys/powerpc/include/cpufunc.h 1.1 +76 -0 src/sys/powerpc/include/critical.h (new) 1.2 +0 -20 src/sys/powerpc/powerpc/critical.c 1.14 +0 -5 src/sys/sparc64/include/cpufunc.h 1.1 +75 -0 src/sys/sparc64/include/critical.h (new) 1.5 +0 -21 src/sys/sparc64/sparc64/critical.c To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message