From patchwork Thu Jan 2 07:58:22 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: chenweilong X-Patchwork-Id: 306068 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 15CBC2C00A0 for ; Thu, 2 Jan 2014 18:58:42 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750987AbaABH6i (ORCPT ); Thu, 2 Jan 2014 02:58:38 -0500 Received: from szxga02-in.huawei.com ([119.145.14.65]:8602 "EHLO szxga02-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750721AbaABH6h (ORCPT ); Thu, 2 Jan 2014 02:58:37 -0500 Received: from 172.24.2.119 (EHLO szxeml213-edg.china.huawei.com) ([172.24.2.119]) by szxrg02-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id BOA43564; Thu, 02 Jan 2014 15:58:30 +0800 (CST) Received: from SZXEML412-HUB.china.huawei.com (10.82.67.91) by szxeml213-edg.china.huawei.com (172.24.2.30) with Microsoft SMTP Server (TLS) id 14.3.158.1; Thu, 2 Jan 2014 15:58:29 +0800 Received: from [127.0.0.1] (10.135.74.202) by szxeml412-hub.china.huawei.com (10.82.67.91) with Microsoft SMTP Server id 14.3.158.1; Thu, 2 Jan 2014 15:58:25 +0800 Message-ID: <52C51C1E.9040603@huawei.com> Date: Thu, 2 Jan 2014 15:58:22 +0800 From: chenweilong User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:24.0) Gecko/20100101 Thunderbird/24.0.1 MIME-Version: 1.0 To: David Miller , , , , Hannes Frederic Sowa Subject: Re: [PATCH] ipv6: don't call addrconf_dst_alloc again when enable lo References: <1371352470-3226-1-git-send-email-gaofeng@cn.fujitsu.com> <20130619.230532.1329281330568271336.davem@davemloft.net> <20131231035718.GB27636@order.stressinduktion.org> <52C4FDBE.5030607@huawei.com> <20140102065429.GB22494@order.stressinduktion.org> In-Reply-To: <20140102065429.GB22494@order.stressinduktion.org> X-Originating-IP: [10.135.74.202] X-CFilter-Loop: Reflected Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On 2014/1/2 14:54, Hannes Frederic Sowa wrote: > On Thu, Jan 02, 2014 at 01:48:46PM +0800, chenweilong wrote: >> On 2013/12/31 11:57, Hannes Frederic Sowa wrote: >>> On Wed, Jun 19, 2013 at 11:05:32PM -0700, David Miller wrote: >>>> From: Gao feng >>>> Date: Sun, 16 Jun 2013 11:14:30 +0800 >>>> >>>>> If we disable all of the net interfaces, and enable >>>>> un-lo interface before lo interface, we already allocated >>>>> the addrconf dst in ipv6_add_addr. So we shouldn't allocate >>>>> it again when we enable lo interface. >>>>> >>>>> Otherwise the message below will be triggered. >>>>> unregister_netdevice: waiting for sit1 to become free. Usage count = 1 >>>>> >>>>> This problem is introduced by commit 25fb6ca4ed9cad72f14f61629b68dc03c0d9713f >>>>> "net IPv6 : Fix broken IPv6 routing table after loopback down-up" >>>>> >>>>> Signed-off-by: Gao feng >>>> >>>> This is the second such regression added by that commit :-/ >>>> >>>> Applied and queue up for -stable, thanks. >>> >>> Hmm, and this change also has a regression and breaks the original fix. :/ >>> >>> https://bugzilla.kernel.org/show_bug.cgi?id=67951 >>> >>> I tried to track it down but it seems pretty complicated. Maybe we have to >>> special-case the take-down of the loopback device. >>> >>> >>> >> >> When I did the tests,If 'ifconfig lo down',all IPv6 connection broken, >> but IPv4 connection were still OK. >> >> Is it designed like that or a bug? > > This seems to solve the loopback up/down problem, but there are still > some issues with up/down of interfaces and routing table interactions. > > We enable routes over interfaces when interface is actually down and > kick manually specified on-link routes when we actually should try to > keep them and just disable them. > > diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c > index 6c16345..61d752a 100644 > --- a/net/ipv6/addrconf.c > +++ b/net/ipv6/addrconf.c > @@ -2629,8 +2629,10 @@ static void init_loopback(struct net_device *dev) > if (sp_ifa->flags & (IFA_F_DADFAILED | IFA_F_TENTATIVE)) > continue; > > - if (sp_ifa->rt) > + if (sp_ifa->rt) { > + ip6_ins_rt(sp_ifa->rt); > continue; > + } > > sp_rt = addrconf_dst_alloc(idev, &sp_ifa->addr, false); > > > . > I test the patch,it has the problem Gao feng reported. How about this: --- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c index d5fa5b8..5e2db6e 100644 --- a/net/ipv6/addrconf.c +++ b/net/ipv6/addrconf.c @@ -2609,10 +2609,13 @@ static void init_loopback(struct net_device *dev) if (sp_ifa->flags & (IFA_F_DADFAILED | IFA_F_TENTATIVE)) continue; - if (sp_ifa->rt) - continue; + if (sp_ifa->rt && sp_ifa->rt->dst.dev == dev){ + ip6_del_rt(sp_ifa->rt); + } sp_rt = addrconf_dst_alloc(idev, &sp_ifa->addr, false); /* Failure cases are ignored */