Date: Sat, 22 Feb 2014 15:28:38 -0800 From: Craig Rodrigues <rodrigc@FreeBSD.org> To: Gleb Smirnoff <glebius@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, Nikos Vassiliadis <nvass@gmx.com>, src-committers@freebsd.org, Martin Matuska <mm@freebsd.org> Subject: Re: svn commit: r262196 - head/sys/netpfil/pf Message-ID: <CAG=rPVfOTiH-egTigXrYJEyunR0R1UMiS8=HgT3=hFoLSQU12A@mail.gmail.com> In-Reply-To: <20140219153215.GD63039@FreeBSD.org> References: <201402182217.s1IMHCeM077356@svn.freebsd.org> <20140219101736.GX63039@glebius.int.ru> <20140219140123.Horde.zrXx5GqkiiaAfGIetc98KQ1@mail.vx.sk> <20140219153215.GD63039@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Feb 19, 2014 at 7:32 AM, Gleb Smirnoff <glebius@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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG=rPVfOTiH-egTigXrYJEyunR0R1UMiS8=HgT3=hFoLSQU12A>