Re: [nft PATCH 0/8] nft event monitor

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 14, 2014 at 02:32:52PM +0200, Pablo Neira Ayuso wrote:
> On Mon, Apr 14, 2014 at 12:17:03PM +0200, Arturo Borrero Gonzalez wrote:
> > The following series implements a basic nftables monitor via Netlink messages.
> 
> Nice series.
> 
> > Most of the work in first patches is related to refactorization/generalization
> > of code.
> > 
> > The final patch is the big one.
> 
> This time is OK with me, but next time you can probably split it in
> smaller chunks, so you add table events in one patch, then chain event
> in another, and so on. So 8/8 gets distributed is several patches. But
> I think it's also reasonable to have it in one patch for this time.
> 
> Please wait to see if Patrick and others have comments on it and then
> go revisit 8/8, thanks Arturo.

I'm fine with these patches. If you resend just 8/8 I can apply them.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Netfitler Users]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux