[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20140820201843.083a45d9@uryu.home.lan>
Date: Wed, 20 Aug 2014 20:18:43 -0700
From: Stephen Hemminger <stephen@...workplumber.org>
To: Zhu Yanjun <zyjzyj2000@...il.com>
Cc: linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
Yue.Tao@...driver.com, alexandre.dietsch@...driver.com,
davem@...emloft.net, honkiko@...il.com, cwang@...pensource.com,
Zhu Yanjun <Yanjun.Zhu@...driver.com>
Subject: Re: [PATCH V2 0/1] ipv4: net namespace does not inherit network
configurations
On Thu, 21 Aug 2014 10:32:00 +0800
Zhu Yanjun <zyjzyj2000@...il.com> wrote:
> V2: Following the advice from Cong Wang, I submit a patch as normal.
>
> Hi,all
>
> I did a test on kernel3.16 rc6:
>
> root@...u1:~# echo 1 > /proc/sys/net/ipv6/conf/all/forwarding
> root@...u1:~# echo 1 > /proc/sys/net/ipv4/conf/all/forwarding
> root@...u1:~# ip netns list
> root@...u1:~# ip netns add fib1
> root@...u1:~# ip netns exec fib1 bash
> root@...u1:~# cat /proc/sys/net/ipv6/conf/all/forwarding
> 0
> root@...u1:~# cat /proc/sys/net/ipv4/conf/all/forwarding
> 1
>
> The behavior of ipv4 and ipv6 is very inconsistent. I checked
> the kernel source code. I found that from this patch
> [ipv6: fix bad free of addrconf_init_net], the above difference
> appeared.
>
> Since a net namespace is independent to another. That is, there
> is no any relationship between the net namespaces. So the behavior
> of ipv4 is not correct.
>
> Based on this patch [ipv6: fix bad free of addrconf_init_net], I made
> a new patch to fix this problem on ipv4.
>
> Any reply is appreciated.
>
> Zhu Yanjun (1):
> ipv4: net namespace does not inherit network configurations
>
> net/ipv4/devinet.c | 29 ++++++++++++-----------------
> 1 file changed, 12 insertions(+), 17 deletions(-)
>
This a semantic change to network namespaces and therefore is
likely to break existing applications using network namespaces.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists