net/ipv6: Move dst flags to booleans in fib entries
authorDavid Ahern <dsahern@gmail.com>
Wed, 18 Apr 2018 00:33:20 +0000 (17:33 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 18 Apr 2018 03:41:17 +0000 (23:41 -0400)
Continuing to wean FIB paths off of dst_entry, use a bool to hold
requests for certain dst settings. Add a helper to convert the
flags to DST flags when a FIB entry is converted to a dst_entry.

Signed-off-by: David Ahern <dsahern@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/ip6_fib.h
net/ipv6/addrconf.c
net/ipv6/route.c

index c73b985734f5fd211a2d97e443baef0abed502b6..159f651dee554e1f95be099d3bc1ac33ebc6cbdb 100644 (file)
@@ -177,7 +177,10 @@ struct rt6_info {
        u8                              fib6_type;
        u8                              exception_bucket_flushed:1,
                                        should_flush:1,
-                                       unused:6;
+                                       dst_nocount:1,
+                                       dst_nopolicy:1,
+                                       dst_host:1,
+                                       unused:3;
 
        unsigned long                   expires;
        struct dst_metrics              *fib6_metrics;
index a156f1a0b1a783b011b5663fa4da4a4da18e1309..c8df5c6499db04fa984004a94ca7007ede01d091 100644 (file)
@@ -1046,7 +1046,7 @@ ipv6_add_addr(struct inet6_dev *idev, const struct in6_addr *addr,
 
        if (net->ipv6.devconf_all->disable_policy ||
            idev->cnf.disable_policy)
-               rt->dst.flags |= DST_NOPOLICY;
+               rt->dst_nopolicy = true;
 
        neigh_parms_data_state_setall(idev->nd_parms);
 
@@ -5981,7 +5981,7 @@ void addrconf_disable_policy_idev(struct inet6_dev *idev, int val)
                        int cpu;
 
                        rcu_read_lock();
-                       addrconf_set_nopolicy(ifa->rt, val);
+                       ifa->rt->dst_nopolicy = val ? true : false;
                        if (rt->rt6i_pcpu) {
                                for_each_possible_cpu(cpu) {
                                        struct rt6_info **rtp;
index e293692174bae52e279ff67b89a9e38e2e3f0964..1a3e0db31b34691b7da1b886c727b39d25888efb 100644 (file)
@@ -937,6 +937,20 @@ static int ip6_rt_type_to_error(u8 fib6_type)
        return fib6_prop[fib6_type];
 }
 
+static unsigned short fib6_info_dst_flags(struct rt6_info *rt)
+{
+       unsigned short flags = 0;
+
+       if (rt->dst_nocount)
+               flags |= DST_NOCOUNT;
+       if (rt->dst_nopolicy)
+               flags |= DST_NOPOLICY;
+       if (rt->dst_host)
+               flags |= DST_HOST;
+
+       return flags;
+}
+
 static void ip6_rt_init_dst_reject(struct rt6_info *rt, struct rt6_info *ort)
 {
        rt->dst.error = ip6_rt_type_to_error(ort->fib6_type);
@@ -961,6 +975,8 @@ static void ip6_rt_init_dst_reject(struct rt6_info *rt, struct rt6_info *ort)
 
 static void ip6_rt_init_dst(struct rt6_info *rt, struct rt6_info *ort)
 {
+       rt->dst.flags |= fib6_info_dst_flags(ort);
+
        if (ort->rt6i_flags & RTF_REJECT) {
                ip6_rt_init_dst_reject(rt, ort);
                return;
@@ -970,7 +986,6 @@ static void ip6_rt_init_dst(struct rt6_info *rt, struct rt6_info *ort)
        rt->dst.output = ip6_output;
 
        if (ort->fib6_type == RTN_LOCAL) {
-               rt->dst.flags |= DST_HOST;
                rt->dst.input = ip6_input;
        } else if (ipv6_addr_type(&ort->rt6i_dst.addr) & IPV6_ADDR_MULTICAST) {
                rt->dst.input = ip6_mc_input;
@@ -1058,10 +1073,11 @@ static bool ip6_hold_safe(struct net *net, struct rt6_info **prt,
 /* called with rcu_lock held */
 static struct rt6_info *ip6_create_rt_rcu(struct rt6_info *rt)
 {
+       unsigned short flags = fib6_info_dst_flags(rt);
        struct net_device *dev = rt->fib6_nh.nh_dev;
        struct rt6_info *nrt;
 
-       nrt = __ip6_dst_alloc(dev_net(dev), dev, 0);
+       nrt = __ip6_dst_alloc(dev_net(dev), dev, flags);
        if (nrt)
                ip6_rt_copy_init(nrt, rt);
 
@@ -1229,12 +1245,13 @@ static struct rt6_info *ip6_rt_cache_alloc(struct rt6_info *ort,
 
 static struct rt6_info *ip6_rt_pcpu_alloc(struct rt6_info *rt)
 {
+       unsigned short flags = fib6_info_dst_flags(rt);
        struct net_device *dev;
        struct rt6_info *pcpu_rt;
 
        rcu_read_lock();
        dev = ip6_rt_get_dev_rcu(rt);
-       pcpu_rt = __ip6_dst_alloc(dev_net(dev), dev, rt->dst.flags);
+       pcpu_rt = __ip6_dst_alloc(dev_net(dev), dev, flags);
        rcu_read_unlock();
        if (!pcpu_rt)
                return NULL;
@@ -2965,7 +2982,7 @@ static struct rt6_info *ip6_route_info_create(struct fib6_config *cfg,
        ipv6_addr_prefix(&rt->rt6i_dst.addr, &cfg->fc_dst, cfg->fc_dst_len);
        rt->rt6i_dst.plen = cfg->fc_dst_len;
        if (rt->rt6i_dst.plen == 128)
-               rt->dst.flags |= DST_HOST;
+               rt->dst_host = true;
 
 #ifdef CONFIG_IPV6_SUBTREES
        ipv6_addr_prefix(&rt->rt6i_src.addr, &cfg->fc_src, cfg->fc_src_len);
@@ -3626,10 +3643,12 @@ struct rt6_info *addrconf_dst_alloc(struct net *net,
        if (!rt)
                return ERR_PTR(-ENOMEM);
 
+       rt->dst_nocount = true;
+
        in6_dev_hold(idev);
        rt->rt6i_idev = idev;
 
-       rt->dst.flags |= DST_HOST;
+       rt->dst_host = true;
        rt->rt6i_protocol = RTPROT_KERNEL;
        rt->rt6i_flags = RTF_UP | RTF_NONEXTHOP;
        if (anycast) {