Message ID | 1298612883-18762-1-git-send-email-lucian.grijincu@gmail.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
On 02/25/2011 06:48 AM, Lucian Adrian Grijincu wrote: > Before this patch issuing these commands: > > fd = open("/proc/sys/net/ipv6/route/flush") > unshare(CLONE_NEWNET) > write(fd, "stuff") > > would flush the newly created net, not the original one. > > The equivalent ipv4 code is correct (stores the net inside ->extra1). > --- Acked-by: Daniel Lezcano <daniel.lezcano@free.fr> -- 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
From: Daniel Lezcano <daniel.lezcano@free.fr> Date: Fri, 25 Feb 2011 16:30:25 +0100 > On 02/25/2011 06:48 AM, Lucian Adrian Grijincu wrote: >> Before this patch issuing these commands: >> >> fd = open("/proc/sys/net/ipv6/route/flush") >> unshare(CLONE_NEWNET) >> write(fd, "stuff") >> >> would flush the newly created net, not the original one. >> >> The equivalent ipv4 code is correct (stores the net inside ->extra1). >> --- > > Acked-by: Daniel Lezcano <daniel.lezcano@free.fr> Applied, thanks. -- 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
Lucian Adrian Grijincu wrote, at 02/25/2011 01:48 PM: > Before this patch issuing these commands: > > fd = open("/proc/sys/net/ipv6/route/flush") > unshare(CLONE_NEWNET) > write(fd, "stuff") > > would flush the newly created net, not the original one. After appling your patch, when excuting above commands, router cache still not be flushed in init net namespace. But IPv4 is ok. Host1: Host2 ping6 Host2 (shell1) open(/proc/sys/net/ipv6/route/flush) unshare(CLONE_NEWNET) while(1) {write(fd, "stuff")} (shell2) ip -6 route show cache <==1* 1*: IPv6 will show one route cache entry. But IPv4 shows nothing. As changlog said, after this patch, shell1 only fulsh the original net namespace, not the newly created one. But from shell2, we can see that IPv6 route cache is not flushed. Have i missed something?
On 03/02/2011 04:25 AM, Shan Wei wrote: > Lucian Adrian Grijincu wrote, at 02/25/2011 01:48 PM: >> Before this patch issuing these commands: >> >> fd = open("/proc/sys/net/ipv6/route/flush") >> unshare(CLONE_NEWNET) >> write(fd, "stuff") >> >> would flush the newly created net, not the original one. > > After appling your patch, when excuting above commands, > router cache still not be flushed in init net namespace. > But IPv4 is ok. > > Host1: Host2 > ping6 Host2 > (shell1) > open(/proc/sys/net/ipv6/route/flush) > unshare(CLONE_NEWNET) > while(1) {write(fd, "stuff")} > > > (shell2) > ip -6 route show cache<==1* > 1*: IPv6 will show one route cache entry. > But IPv4 shows nothing. > > As changlog said, after this patch, shell1 only fulsh the original > net namespace, not the newly created one. But from shell2, we > can see that IPv6 route cache is not flushed. > > Have i missed something? Yes, you are right you shouldn't see any route from shell2. Are you sure 'ip -6 route show cache' does not show the route cache in a window where ping6 created a new route and 'write(fd, "stuff")' did not flushed the new route yet ? -- 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/route.c b/net/ipv6/route.c index f786aed..522563e 100644 --- a/net/ipv6/route.c +++ b/net/ipv6/route.c @@ -2604,14 +2604,16 @@ static int ipv6_sysctl_rtcache_flush(ctl_table *ctl, int write, void __user *buffer, size_t *lenp, loff_t *ppos) { - struct net *net = current->nsproxy->net_ns; - int delay = net->ipv6.sysctl.flush_delay; - if (write) { - proc_dointvec(ctl, write, buffer, lenp, ppos); - fib6_run_gc(delay <= 0 ? ~0UL : (unsigned long)delay, net); - return 0; - } else + struct net *net; + int delay; + if (!write) return -EINVAL; + + net = (struct net *)ctl->extra1; + delay = net->ipv6.sysctl.flush_delay; + proc_dointvec(ctl, write, buffer, lenp, ppos); + fib6_run_gc(delay <= 0 ? ~0UL : (unsigned long)delay, net); + return 0; } ctl_table ipv6_route_table_template[] = { @@ -2698,6 +2700,7 @@ struct ctl_table * __net_init ipv6_route_sysctl_init(struct net *net) if (table) { table[0].data = &net->ipv6.sysctl.flush_delay; + table[0].extra1 = net; table[1].data = &net->ipv6.ip6_dst_ops.gc_thresh; table[2].data = &net->ipv6.sysctl.ip6_rt_max_size; table[3].data = &net->ipv6.sysctl.ip6_rt_gc_min_interval;