From owner-svn-src-head@FreeBSD.ORG Wed Oct 19 17:27:32 2011 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3BC8F106566C; Wed, 19 Oct 2011 17:27:32 +0000 (UTC) (envelope-from pluknet@gmail.com) Received: from mail-vx0-f182.google.com (mail-vx0-f182.google.com [209.85.220.182]) by mx1.freebsd.org (Postfix) with ESMTP id AA0FB8FC18; Wed, 19 Oct 2011 17:27:31 +0000 (UTC) Received: by vcbfo13 with SMTP id fo13so2653952vcb.13 for ; Wed, 19 Oct 2011 10:27:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=FYueB77FI5djx++fSwBwLwF+88KIl5Fo4DtvSY+jrXk=; b=j/G2m1xiwQpL2vrAtp/nTfDfNgGwHMg7H2mxsGSyUQoWTHQYJhvWIJV5Z9E4FxYqhK EtWhga0U4H5HSgSTqC8+coHp538+PkD93fUMDF91fzXQmN/xVJhO2Lenn3DgpiIbwfsi NwkmIzFxiigxOHhssJ6zbxte31hVeiR4Lwa2A= MIME-Version: 1.0 Received: by 10.182.110.1 with SMTP id hw1mr485632obb.38.1319045250904; Wed, 19 Oct 2011 10:27:30 -0700 (PDT) Sender: pluknet@gmail.com Received: by 10.182.216.38 with HTTP; Wed, 19 Oct 2011 10:27:30 -0700 (PDT) In-Reply-To: <201110191313.p9JDDuDP026273@svn.freebsd.org> References: <201110191313.p9JDDuDP026273@svn.freebsd.org> Date: Wed, 19 Oct 2011 21:27:30 +0400 X-Google-Sender-Auth: O6TQD0PxhdSvVFuX1U2L2Y5Uxr4 Message-ID: From: Sergey Kandaurov To: "Bjoern A. Zeeb" Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable 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 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 19 Oct 2011 17:27:32 -0000 On 19 October 2011 17:13, Bjoern A. Zeeb 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