From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick McHardy Subject: Re: [ULOGD RFC 00/30] [ULOGD RFC] ulogd V2 improvements, round #1 Date: Wed, 30 Jan 2008 20:18:20 +0100 Message-ID: <47A0CD7C.2010705@trash.net> References: <20080130185847.693274384@kruemel.intranet.astaro.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Cc: netfilter-devel@vger.kernel.org, holger@eitzenberger.org To: heitzenberger@astaro.com Return-path: Received: from stinky.trash.net ([213.144.137.162]:59100 "EHLO stinky.trash.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757430AbYA3TSz (ORCPT ); Wed, 30 Jan 2008 14:18:55 -0500 In-Reply-To: <20080130185847.693274384@kruemel.intranet.astaro.de> Sender: netfilter-devel-owner@vger.kernel.org List-ID: 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?