Date: Fri, 22 Jun 2001 16:41:22 -0700 (PDT) From: John Baldwin <jhb@FreeBSD.org> To: John Baldwin <jhb@FreeBSD.org> Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org Subject: RE: cvs commit: src/sys/alpha/alpha trap.c src/sys/i386/i386 tra Message-ID: <XFMail.010622164122.jhb@FreeBSD.org> In-Reply-To: <200106222305.f5MN5Fc80501@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 22-Jun-01 John Baldwin wrote: > jhb 2001/06/22 16:05:14 PDT > > Modified files: > sys/alpha/alpha trap.c > sys/i386/i386 trap.c > sys/ia64/ia64 trap.c > Log: > - Grab the proc lock around CURSIG and postsig(). Don't release the proc > lock until after grabbing the sched_lock to avoid CURSIG racing with > psignal. > - Don't grab Giant for addupc_task() as it isn't needed. > > Reported by: tegge (signal race), bde (addupc_task a while back) At some point userret() and ast() really should be moved to a MI location. userret() is already identical on all platforms, and ast() just needs one MD macro for assigning the proper member of p_md to the passed in trapframe so signals work for it to be MI. It would make updating these easier as well as hopefully making it easier for new ports. -- John Baldwin <jhb@FreeBSD.org> -- http://www.FreeBSD.org/~jhb/ PGP Key: http://www.baldwin.cx/~john/pgpkey.asc "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?XFMail.010622164122.jhb>