From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jon Masters Subject: Re: [PATCH] netfilter: per netns nf_conntrack_cachep Date: Tue, 02 Feb 2010 13:39:42 -0500 Message-ID: <1265135982.2861.199.camel@tonnant> References: <1264813832.2793.446.camel@tonnant> <1265023437.2848.30.camel@edumazet-laptop> <1265035970.2848.50.camel@edumazet-laptop> <1265036548.2848.55.camel@edumazet-laptop> <1265108690.2861.118.camel@tonnant> <1265110504.2861.135.camel@tonnant> <1265129192.2861.141.camel@tonnant> <4B685756.8010107@trash.net> <1265130426.2861.158.camel@tonnant> <1265134598.2861.191.camel@tonnant> <4B6870AF.6060109@trash.net> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: Alexey Dobriyan , Eric Dumazet , linux-kernel , netdev , netfilter-devel , "Paul E. McKenney" To: Patrick McHardy Return-path: Received: from dallas.jonmasters.org ([72.29.103.172]:40300 "EHLO dallas.jonmasters.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756651Ab0BBSkA (ORCPT ); Tue, 2 Feb 2010 13:40:00 -0500 In-Reply-To: <4B6870AF.6060109@trash.net> Sender: netdev-owner@vger.kernel.org List-ID: On Tue, 2010-02-02 at 19:36 +0100, Patrick McHardy wrote: > Jon Masters wrote: > > On Tue, 2010-02-02 at 19:58 +0200, Alexey Dobriyan wrote: > > > >> Yes, moving to init_net-only function is fine. > > > > So moving the "setup up fake conntrack" bits to init_init_net from > > init_net still results in the panic, which means that the use count > > really is dropping to zero and we really are trying to free it when > > using multiple namespaces. Per ns is probably an easier way to go. > > Agreed, that will also avoid problems in the future with the > ct_net pointer pointing to &init_net. I'll take care of this > tommorrow. Ok. I'll leave this box running with the hack. I think at the very least that this specific issue needs to get fixed and in the stable tree, then the other bits (per namespace cachep...) are probably a good idea at the same time but that's up to you. Thanks for your help! Jon.