Re: [ULOGD RFC 00/30] [ULOGD RFC] ulogd V2 improvements, round #1

[Patrick McHardy <kaber@trash.net>]

heitzenberger@astaro.com wrote:
> Hi,
>
> this is a patchset for you to comment on, which tries to turn ulogd V2
> into something usefull.
>
> Some of the patches are are quite intrusive, others might be to be
> discussed.  Please take a look at all of them and give some comments
> back, I'll appreciate it.
>
> In the end they turn ulogd v2 into some powerfull network accounting
> solution being capable to handle 400000 or more concurrent flows.
> That's where your post-processing starts to get important of course
> :).
>
> Some points which might be of interest to you:
>
>  * ulogd-ifi.diff: replacement for the IFINDEX plugin, as that
>  functionality IMO is better homed in the ulogd core and not in a
>  plugin (remove IFINDEX?).
>
>  * some of the patches are quite large and contain many individual 
>  changes in the end.  Should I split them?
>
>  * ulogd-NFCT-new-lib.diff: makes ulogd work with a recent
>  libnetfilter-conntrack.  This one is fairly untested, I'll test that
>  before sending the final patchset.  Note also that I'd like to use
>  libnl with ulogd rather soon, so this is only a short-term solution.
>
>  * the libnetfilter-conntrack patch I've send before is required
>  to build ulogd.
>
> Some of those patches still contain trailing whitespace.  Please
> ignore that as of now, I'll send additional patches removing
> whitespace later.
>   

Thanks for sending these patches Holger, I'll try to review all of them
tommorrow. Just for my understanding since the ones I already looked at
look like I could merge them immediately, do you want me to already
commit them or is this really just meant to gather comments?