On 06/06/2013 02:28 PM, Gleb Smirnoff wrote:
M> > module unload is broken:( Maybe it can be fixed at a (bit) later date?
M>
M> I don't think Gleb will be happy with this. Some time ago he removed
M> some vimage related stuff to prevent crashing on module unload (see
M> r229849). Actually your pat
Hi, I quite agree with Gleb Smirnoff on the possibility of avoiding a
reboot after each compile.
I don't now if it's a good moment to speak of this, but I observed that if
pfsync is active in the kernel, it will use its state table even if nothing
is set in the rc.conf.
I think that if no paramet
On Thu, Jun 06, 2013 at 03:24:10PM +0300, Mikolaj Golub wrote:
M> > >> -VNET_DEFINE(u_long, pf_srchashsize);
M> > >> -#defineV_pf_srchashsizeVNET(pf_srchashsize)
M> > >> -SYSCTL_VNET_UINT(_net_pf, OID_AUTO, source_nodes_hashsize,
CTLFLAG_RDTUN,
M> > >> -&VNET_NAME(pf_srchashsiz
On Thu, Jun 06, 2013 at 12:35:33PM +0200, Nikos Vassiliadis wrote:
> >> -VNET_DEFINE(u_long, pf_srchashsize);
> >> -#define V_pf_srchashsizeVNET(pf_srchashsize)
> >> -SYSCTL_VNET_UINT(_net_pf, OID_AUTO, source_nodes_hashsize, CTLFLAG_RDTUN,
> >> -&VNET_NAME(pf_srchashsize), 0, "Size
Hi,
Comments below.
On 06/05/2013 10:52 AM, Mikolaj Golub wrote:
1) It looks like the patch can be split on several parts. A log
message to every change describing why it is needed and what problem
solves would be very helpful. As a tool to maintain such changes I
personally prefer git.
I'll
Hi,
On Mon, Jun 03, 2013 at 01:58:38PM +0200, Nikos Vassiliadis wrote:
> Hi,
>
> Please review this patch. It fixes some problems with pf and vimage.
> For the time being only pf works. ALTQ, pflog, pfsync are not changed
> nor tested but as time permits, I'll work on them. Basic packet
> filteri
Hi,
Please review this patch. It fixes some problems with pf and vimage.
For the time being only pf works. ALTQ, pflog, pfsync are not changed
nor tested but as time permits, I'll work on them. Basic packet
filtering functionality per VNET should be ok.
Thanks in advance for reviewing,
Nikos
In