From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick McHardy Subject: Re: [PATCH] simplify the code to dump the conntrack table Date: Thu, 14 Sep 2006 13:15:34 +0200 Message-ID: <450939D6.3040802@trash.net> References: <45001B82.4000504@netfilter.org> <4500370F.5050002@trash.net> <45049837.3050208@netfilter.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: Harald Welte , Netfilter Development Mailinglist , Jozsef Kadlecsik Return-path: To: Pablo Neira Ayuso In-Reply-To: <45049837.3050208@netfilter.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: netfilter-devel-bounces@lists.netfilter.org Errors-To: netfilter-devel-bounces@lists.netfilter.org List-Id: netfilter-devel.vger.kernel.org Pablo Neira Ayuso wrote: > Patrick McHardy wrote: > >>Pablo Neira Ayuso wrote: >> >>>Merge the bits to dump the conntrack table and the ones to dump and >>>zero counters in a single piece of code. This patch does not change >>>the default behaviour if accounting is not enabled. >> >>Nice cleanup, thanks. And it saves me from removing the id based >>dumping, which I forgot to change for the counter case :) > > > Thanks. So I guess that the next question is if there is any plan to > remove the id ;) I think that question has been discussed enough. Lets get rid of it. Small complaint about this patch btw, it didn't compile without conntrack accounting since the counters are only present when it is enabled. >>BTW, what about my last question regarding your early_drop patch? > > > Sorry, I forgot to do it, please mangle the patch and use '>' instead of > '>=', otherwise the maximum number of conntracks will be ip_conntrack_max-1. Done, thanks. > BTW, did you have the chance to have a look these patches? > > http://lists.netfilter.org/pipermail/netfilter-devel/2006-August/025345.html Not yet. > http://lists.netfilter.org/pipermail/netfilter-devel/2006-August/025353.html There were a couple of questions, still waiting for your response.