Skip site navigation (1)Skip section navigation (2)
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>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r276747 - head/sys/netpfil/pf
Message-ID:  <20150122012709.GM15484@FreeBSD.org>
In-Reply-To: <201501060903.t06934qp081875@svn.freebsd.org>
References:  <201501060903.t06934qp081875@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150122012709.GM15484>