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 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.
--
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