From owner-freebsd-current Thu Apr 4 10:54:23 2002 Delivered-To: freebsd-current@freebsd.org Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by hub.freebsd.org (Postfix) with ESMTP id 5908E37B41C for ; Thu, 4 Apr 2002 10:54:16 -0800 (PST) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id EAA27937 for ; Fri, 5 Apr 2002 04:54:14 +1000 Date: Fri, 5 Apr 2002 04:53:50 +1000 (EST) From: Bruce Evans X-X-Sender: bde@gamplex.bde.org To: current@freebsd.org Subject: Re: cvs commit: src/sys/alpha/alpha machdep.c src/sys/alpha/osf1 osf1_signal.c src/sys/coda coda_psdev.c src/sys/compat/linux linux_signal.c src/sys/compat/svr4 svr4_signal.c src/sys/i386/ In-Reply-To: <200204041749.g34Hnng99475@freefall.freebsd.org> Message-ID: <20020405045116.F2040-100000@gamplex.bde.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Thu, 4 Apr 2002, Bruce Evans wrote: > bde 2002/04/04 09:49:49 PST > > Modified files: > sys/alpha/alpha machdep.c > sys/alpha/osf1 osf1_signal.c > sys/coda coda_psdev.c > sys/compat/linux linux_signal.c > sys/compat/svr4 svr4_signal.c > sys/i386/i386 machdep.c > sys/i386/ibcs2 ibcs2_signal.c > sys/i386/linux linux_sysvec.c > sys/i386/svr4 svr4_machdep.c > sys/ia64/ia64 machdep.c > sys/kern kern_sig.c subr_trap.c > sys/pc98/i386 machdep.c > sys/sparc64/sparc64 machdep.c > sys/sys proc.h signalvar.h > Log: > Moved signal handling and rescheduling from userret() to ast() so that > they aren't in the usual path of execution for syscalls and traps. > The main complication for this is that we have to set flags to control > ast() everywhere that changes the signal mask. > > Avoid locking in userret() in most of the remaining cases. > > Submitted by: luoqi (first part only, long ago, reorganized by me) > Reminded by: dillon Tha invariants check in this (in userret()) failed for the first time 10 minutes after I committed it. I think this is harmless (due to a cached flag in ast()) but don't have time to check today. Turn off the check if it spams the console but signals appear to work despite this. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message