svn commit: r262196 - head/sys/netpfil/pf

[Craig Rodrigues]

On Wed, Feb 19, 2014 at 7:32 AM, Gleb Smirnoff <glebius at freebsd.org> wrote:

> Third. As you already know, there is projects/pf branch, where Nicos
> is getting things right wrt pf+VIMAGE. The patches should first go
> to this branch and tested in it. Committing to head (even a good
> code), you are creating conflicts for Nicos. You are fixing two
> particular problems that hurt you, while Nicos tries to get things
> right in general, for everyones sake. My approach on taskqueue
> context (that was rejected by Marko), was also an attempt to
> create a good and generic way of dealing with the problem. Unfortunately,
> Marko didn't suggest good alternatives.
>
>
 Gleb,

Much thanks to you and Nicos for working on pf + VIMAGE.
I work on FreeNAS at iXsystems, and in FreeNAS we enable
VIMAGE, because we use jails quiet heavily.

I encouraged Martin to go with his patch, because
it seemed to solve some problems we had with pf + VIMAGE kernel panics
such as this one:

https://bugs.freenas.org/issues/4153

Do you have a rough idea when you and Nicos will be ready to merge your
projects/pf branch to HEAD?
Would you be open to forking the https://github.com/trueos repository on
github
and putting your pf + VIMAGE patches in there?

We could make special builds of FreeNAS and aggressively test your VIMAGE
patches.  FreeNAS isn't based on FreeBSD CURRENT/FreeBSD 10 yet.....
we have some work to do to get up to there.

Thanks.


--
Craig