diff options
author | Hannes Frederic Sowa <hannes@stressinduktion.org> | 2013-12-07 03:33:45 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-01-15 15:27:10 -0800 |
commit | 30e65cd5055baa827b0f827c5c9415046b2c377d (patch) | |
tree | 6321711dfe735858d868abaaa987389c20f89688 | |
parent | ae8d56de3072a8a812e1400686668315fb27c21b (diff) | |
download | lwn-30e65cd5055baa827b0f827c5c9415046b2c377d.tar.gz lwn-30e65cd5055baa827b0f827c5c9415046b2c377d.zip |
ipv6: don't count addrconf generated routes against gc limit
[ Upstream commit a3300ef4bbb1f1e33ff0400e1e6cf7733d988f4f ]
Brett Ciphery reported that new ipv6 addresses failed to get installed
because the addrconf generated dsts where counted against the dst gc
limit. We don't need to count those routes like we currently don't count
administratively added routes.
Because the max_addresses check enforces a limit on unbounded address
generation first in case someone plays with router advertisments, we
are still safe here.
Reported-by: Brett Ciphery <brett.ciphery@windriver.com>
Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | net/ipv6/route.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/net/ipv6/route.c b/net/ipv6/route.c index 6ac8bc29b43e..335b16fccb9d 100644 --- a/net/ipv6/route.c +++ b/net/ipv6/route.c @@ -2114,15 +2114,11 @@ struct rt6_info *addrconf_dst_alloc(struct inet6_dev *idev, { struct net *net = dev_net(idev->dev); struct rt6_info *rt = ip6_dst_alloc(&net->ipv6.ip6_dst_ops, - net->loopback_dev, 0); + net->loopback_dev, DST_NOCOUNT); int err; - if (!rt) { - if (net_ratelimit()) - pr_warning("IPv6: Maximum number of routes reached," - " consider increasing route/max_size.\n"); + if (!rt) return ERR_PTR(-ENOMEM); - } in6_dev_hold(idev); |