Date: Wed, 19 Oct 2011 21:27:30 +0400 From: Sergey Kandaurov <pluknet@freebsd.org> To: "Bjoern A. Zeeb" <bz@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r226544 - head/sys/contrib/pf/net Message-ID: <CAE-mSOLiqen9pW9qiYj4mDNarPsVQRhkso8EoGp0Ac_PsGiaUg@mail.gmail.com> In-Reply-To: <201110191313.p9JDDuDP026273@svn.freebsd.org> References: <201110191313.p9JDDuDP026273@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 19 October 2011 17:13, Bjoern A. Zeeb <bz@freebsd.org> wrote: > Author: bz > Date: Wed Oct 19 13:13:56 2011 > New Revision: 226544 > URL: http://svn.freebsd.org/changeset/base/226544 > > Log: > =A0Fix recursive pf locking leading to panics. =A0Splatter PF_LOCK_ASSERT= ()s > =A0to document where we are expecting to be called with a lock held to > =A0more easily catch unnoticed code paths. > =A0This does not neccessarily improve locking in pfsync, it just tries > =A0to avoid the panics reported. > > =A0PR: =A0 =A0 =A0 =A0 =A0 kern/159390, kern/158873 > =A0Submitted by: pluknet (at least something that partly resembles > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0my patch ignoring other cleanup, which I o= nly saw > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0too late on the 2nd PR) > =A0MFC After: =A0 =A03 days Thanks. Hope my patch helped you at least a little bit. --=20 wbr, pluknet
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAE-mSOLiqen9pW9qiYj4mDNarPsVQRhkso8EoGp0Ac_PsGiaUg>