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: > Fix recursive pf locking leading to panics. Splatter PF_LOCK_ASSERT()s > to document where we are expecting to be called with a lock held to > more easily catch unnoticed code paths. > This does not neccessarily improve locking in pfsync, it just tries > to avoid the panics reported. > > PR: kern/159390, kern/158873 > Submitted by: pluknet (at least something that partly resembles > my patch ignoring other cleanup, which I only saw > too late on the 2nd PR) > MFC After: 3 days Thanks. Hope my patch helped you at least a little bit. -- wbr, pluknet
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAE-mSOLiqen9pW9qiYj4mDNarPsVQRhkso8EoGp0Ac_PsGiaUg>