Date: Tue, 02 Jan 2001 11:36:21 -0800 (PST) From: John Baldwin <jhb@FreeBSD.org> To: Mark Murray <mark@grondar.za> Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org Subject: Re: cvs commit: src/sys/kern kern_sig.c Message-ID: <XFMail.010102113621.jhb@FreeBSD.org> In-Reply-To: <200101021927.f02JRtO11317@gratis.grondar.za>
next in thread | previous in thread | raw e-mail | index | archive | help
On 02-Jan-01 Mark Murray wrote: >> jhb 2001/01/02 10:54:10 PST >> >> Modified files: >> sys/kern kern_sig.c >> Log: >> The previous commit wasn't entirely correct. At least one goto to the >> out: label in psignal() did not grab sched_lock before trying to release >> it. Also, the previous version had several cases where it grabbed >> sched_lock before jumping to out: unneccessarily, so rework this a bit. >> The runfast: and out: labels must be called with sched_lock released, and >> the run: label must be called with it held. Appropriate mtx_assert()'s >> have been added that should catch any bugs that may still be in this >> code. > > It this likely to catch any or all of the CURRENT deadlocks? It might fix hangs involving signal handling, though I think the previous checkin was close enough to have already fixed those. One of my SMP boxes now hangs relatively quickly (10 - 15 minutes into an installworld). I'll need to rig up a serial console, turn on a lot of KTR logging and let it run for however long KTR slows it down (my guess is multiple days :( ) until it hangs then stare the KTR logs until I can figure out where it hung. -- 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.010102113621.jhb>