From owner-svn-src-head@FreeBSD.ORG  Thu Jan 22 01:27:13 2015
Return-Path: <owner-svn-src-head@FreeBSD.ORG>
Delivered-To: svn-src-head@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115])
 (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by hub.freebsd.org (Postfix) with ESMTPS id 56A14BF8;
 Thu, 22 Jan 2015 01:27:13 +0000 (UTC)
Received: from cell.glebius.int.ru (glebius.int.ru [81.19.69.10])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
 (Client CN "cell.glebius.int.ru", Issuer "cell.glebius.int.ru" (not verified))
 by mx1.freebsd.org (Postfix) with ESMTPS id D054ED6D;
 Thu, 22 Jan 2015 01:27:12 +0000 (UTC)
Received: from cell.glebius.int.ru (localhost [127.0.0.1])
 by cell.glebius.int.ru (8.14.9/8.14.9) with ESMTP id t0M1R9VG083373
 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO);
 Thu, 22 Jan 2015 04:27:09 +0300 (MSK)
 (envelope-from glebius@FreeBSD.org)
Received: (from glebius@localhost)
 by cell.glebius.int.ru (8.14.9/8.14.9/Submit) id t0M1R9jp083372;
 Thu, 22 Jan 2015 04:27:09 +0300 (MSK)
 (envelope-from glebius@FreeBSD.org)
X-Authentication-Warning: cell.glebius.int.ru: glebius set sender to
 glebius@FreeBSD.org using -f
Date: Thu, 22 Jan 2015 04:27:09 +0300
From: Gleb Smirnoff <glebius@FreeBSD.org>
To: Craig Rodrigues <rodrigc@FreeBSD.org>, Nikos Vassiliadis <nvass@gmx.com>
Subject: Re: svn commit: r276747 - head/sys/netpfil/pf
Message-ID: <20150122012709.GM15484@FreeBSD.org>
References: <201501060903.t06934qp081875@svn.freebsd.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <201501060903.t06934qp081875@svn.freebsd.org>
User-Agent: Mutt/1.5.23 (2014-03-12)
Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org,
 src-committers@freebsd.org
X-BeenThere: svn-src-head@freebsd.org
X-Mailman-Version: 2.1.18-1
Precedence: list
List-Id: SVN commit messages for the src tree for head/-current
 <svn-src-head.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-head/>
List-Post: <mailto:svn-src-head@freebsd.org>
List-Help: <mailto:svn-src-head-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 22 Jan 2015 01:27:13 -0000

On Tue, Jan 06, 2015 at 09:03:04AM +0000, Craig Rodrigues wrote:
C> Author: rodrigc
C> Date: Tue Jan  6 09:03:03 2015
C> New Revision: 276747
C> URL: https://svnweb.freebsd.org/changeset/base/276747
C> 
C> Log:
C>   Instead of creating a purge thread for every vnet, create
C>   a single purge thread and clean up all vnets from this thread.
C>   
C>   PR:                     194515
C>   Differential Revision:  D1315
C>   Submitted by:           Nikos Vassiliadis <nvass@gmx.com>

Sorry guys, I backed this out due to broken kldunload of pf module, which
is critical when you are working with pf bugs.

I had to backout r276746 as well, since it has numerous build breakages,
that are addressed by later revisions.

That's my fault that I don't review in time, and I will try to improve
the situation.

Can you please replay r276746 again, addressing all the build problems
and send the patch to me? You can user reviews.freebsd.org if you want.

I'd like to get this in, but in a better quality.

-- 
Totus tuus, Glebius.