From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Subject: Re: [PATCH 0/3] Fix Network namespace shutdown take 2 Date: Wed, 25 Feb 2009 13:43:29 +0100 Message-ID: <49A53CF1.5060503@free.fr> References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: netdev-owner@vger.kernel.org To: David Miller Cc: "Eric W. Biederman" , Linux Containers , netdev@vger.kernel.org, Alexey Dobriyan , "Denis V. Lunev" List-Id: containers.vger.kernel.org Eric W. Biederman wrote: > 6 months ago when I introduced net_alive I fixed the symptoms > but I failed to properly fix network namespace shutdown. > > I realized this when I received a bug report on Tuesday about a > failure in icmp_send caused by packets in the arp_gueue. > > It turns out that the net_alive check in netif_receive_skb > is completely unnecessary and just masked the real problem. > > If we remove all network devices from a network namespace before we > shutdown network subsystems and protocols then as designed we cannot > have packets in flight causing problems. > > It turns out that the root cause of these problems is that the icmp > code was calling register_pernet_device instead of > register_pernet_subsys and so it's cleanup was happening much too > early. > > The following patchset which should work against both 2.6.29-rcX > and net-next fixes the registration problems and removes the > unncessary net_alive check, making the code simpler and hopefully > more comprehensible. > Hi Dave, I don't see these patches in the net-2.6 tree. Shouldn't they be in net-2.6 too ? -- Daniel