Date: Sat, 20 Jun 2015 09:53:34 -0400 From: John Baldwin <jhb@FreeBSD.org> To: Sean Bruno <sbruno@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r284602 - head/sys/kern Message-ID: <5585705E.9040004@FreeBSD.org> In-Reply-To: <201506191857.t5JIvbdP094203@svn.freebsd.org> References: <201506191857.t5JIvbdP094203@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 6/19/15 2:57 PM, Sean Bruno wrote: > Author: sbruno > Date: Fri Jun 19 18:57:36 2015 > New Revision: 284602 > URL: https://svnweb.freebsd.org/changeset/base/284602 > > Log: > Feedback from commit r284535 > > davide: imgact_binmisc_clear_entry() needs to use atomic ops to remove > the enable bit. I had asked you to remove the atomics and instead use an exclusive lock in the enable/disable routines as these are not hotpaths and that is simpler. Stacey agreed with that assessment in his review comment. It looks like you removed the atomics but forgot to switch the shared locks to exclusive locks. I would still prefer using exclusive locks and removing the atomics. -- John Baldwin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5585705E.9040004>