ipv6: sr: use dst_cache in seg6_input
authorDavid Lebrun <david.lebrun@uclouvain.be>
Fri, 24 Mar 2017 09:46:27 +0000 (10:46 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 24 Mar 2017 21:47:32 +0000 (14:47 -0700)
We already use dst_cache in seg6_output, when handling locally generated
packets. We extend it in seg6_input, to also handle forwarded packets, and avoid
unnecessary fib lookups.

Performances for SRH encapsulation before the patch:
Result: OK: 5656067(c5655678+d388) usec, 5000000 (1000byte,0frags)
  884006pps 7072Mb/sec (7072048000bps) errors: 0

Performances after the patch:
Result: OK: 4774543(c4774084+d459) usec, 5000000 (1000byte,0frags)
  1047220pps 8377Mb/sec (8377760000bps) errors: 0

Signed-off-by: David Lebrun <david.lebrun@uclouvain.be>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/seg6_iptunnel.c

index dda2c5147ef0e4596a061cfe3c1f1a687c646e37..30ef1d1d182e91e11ae96ead7fda6600a3ef0994 100644 (file)
@@ -237,6 +237,9 @@ static int seg6_do_srh(struct sk_buff *skb)
 
 static int seg6_input(struct sk_buff *skb)
 {
+       struct dst_entry *orig_dst = skb_dst(skb);
+       struct dst_entry *dst = NULL;
+       struct seg6_lwt *slwt;
        int err;
 
        err = seg6_do_srh(skb);
@@ -245,8 +248,30 @@ static int seg6_input(struct sk_buff *skb)
                return err;
        }
 
+       slwt = seg6_lwt_lwtunnel(orig_dst->lwtstate);
+
+#ifdef CONFIG_DST_CACHE
+       preempt_disable();
+       dst = dst_cache_get(&slwt->cache);
+       preempt_enable();
+#endif
+
        skb_dst_drop(skb);
-       ip6_route_input(skb);
+
+       if (!dst) {
+               ip6_route_input(skb);
+#ifdef CONFIG_DST_CACHE
+               dst = skb_dst(skb);
+               if (!dst->error) {
+                       preempt_disable();
+                       dst_cache_set_ip6(&slwt->cache, dst,
+                                         &ipv6_hdr(skb)->saddr);
+                       preempt_enable();
+               }
+#endif
+       } else {
+               skb_dst_set(skb, dst);
+       }
 
        return dst_input(skb);
 }