Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 21 Nov 2012 11:56:42 +0400
From:      Gleb Smirnoff <glebius@FreeBSD.org>
To:        Mark Martinec <Mark.Martinec+freebsd@ijs.si>
Cc:        freebsd-current@FreeBSD.org, freebsd-pf@FreeBSD.org
Subject:   Re: Upgrading FreeBSD to use the NEW pf syntax.
Message-ID:  <20121121075642.GR67660@FreeBSD.org>
In-Reply-To: <201211201543.17903.Mark.Martinec%2Bfreebsd@ijs.si>
References:  <op.wn1vktomjfousr@box.dlink.com> <CAPBZQG2R%2BLXTo8xXZNhfWg%2BS4wtkDc1cAuhoHqdgyiGDGZuXOw@mail.gmail.com> <CAEW%2BogbUkHTaef98=CusV%2BV3qTFHqj-7x-_icKaom_0d2gv69g@mail.gmail.com> <201211201543.17903.Mark.Martinec%2Bfreebsd@ijs.si>

next in thread | previous in thread | raw e-mail | index | archive | help
  Mark,

On Tue, Nov 20, 2012 at 03:43:17PM +0100, Mark Martinec wrote:
M> For one thing, I'm desperately awaiting NAT64 support (the 'af-to'
M> translation rule in newer pf (5.1?), committed on 2011-10).

Backport this exact feature to FreeBSD and send patch.

M> Other: packet normalization (scrub) has been reworked and simplified,
M> and is now a rulset option. Considering that scrub is currently broken
M> (9.1, see list of PF bugs in FreeBSD), along with several other
M> bugs that need fixing, it seems the (scarce) manpower would better
M> be spent in moving on, than keeping the already leaky (buggy) pf
M> afloat.

Yes, scrub improvements can be cherry picked and added to FreeBSD, too.

But if you think that bulk import of new version would close all current
bugs without opening new problems, then you are mistaking. Last bulk
import introduced much more bugs than it closed. And this statement isn't
a accusation towards the person who did the import. This is just a generic
rule. If you take 100k lines of code that were developed for another
operating system kernel and without thourough reviewing it just make it
compile and link with another kernel, then you are about to miss many
rough edges that will show up later, when the code would be utilized.

Thus, cherry-picking is preferred over bulk imports.

-- 
Totus tuus, Glebius.



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