diff options
author | Eric Dumazet <eric.dumazet@gmail.com> | 2010-05-18 01:36:55 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-05-18 01:36:55 -0400 |
commit | d19d56ddc88e7895429ef118db9c83c7bbe3ce6a (patch) | |
tree | c0db76f3527c88c95a8793c871f62d628fb3fd1d /net/ipv6 | |
parent | de213e5eedecdfb1b1eea7e6be28bc64cac5c078 (diff) |
net: Introduce skb_tunnel_rx() helper
skb rxhash should be cleared when a skb is handled by a tunnel before
being delivered again, so that correct packet steering can take place.
There are other cleanups and accounting that we can factorize in a new
helper, skb_tunnel_rx()
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6')
-rw-r--r-- | net/ipv6/ip6_tunnel.c | 8 | ||||
-rw-r--r-- | net/ipv6/ip6mr.c | 8 | ||||
-rw-r--r-- | net/ipv6/sit.c | 8 |
3 files changed, 8 insertions, 16 deletions
diff --git a/net/ipv6/ip6_tunnel.c b/net/ipv6/ip6_tunnel.c index 2599870747ec..8f39893d8081 100644 --- a/net/ipv6/ip6_tunnel.c +++ b/net/ipv6/ip6_tunnel.c | |||
@@ -723,14 +723,10 @@ static int ip6_tnl_rcv(struct sk_buff *skb, __u16 protocol, | |||
723 | skb->protocol = htons(protocol); | 723 | skb->protocol = htons(protocol); |
724 | skb->pkt_type = PACKET_HOST; | 724 | skb->pkt_type = PACKET_HOST; |
725 | memset(skb->cb, 0, sizeof(struct inet6_skb_parm)); | 725 | memset(skb->cb, 0, sizeof(struct inet6_skb_parm)); |
726 | skb->dev = t->dev; | ||
727 | skb_dst_drop(skb); | ||
728 | nf_reset(skb); | ||
729 | 726 | ||
730 | dscp_ecn_decapsulate(t, ipv6h, skb); | 727 | skb_tunnel_rx(skb, t->dev); |
731 | 728 | ||
732 | t->dev->stats.rx_packets++; | 729 | dscp_ecn_decapsulate(t, ipv6h, skb); |
733 | t->dev->stats.rx_bytes += skb->len; | ||
734 | netif_rx(skb); | 730 | netif_rx(skb); |
735 | rcu_read_unlock(); | 731 | rcu_read_unlock(); |
736 | return 0; | 732 | return 0; |
diff --git a/net/ipv6/ip6mr.c b/net/ipv6/ip6mr.c index 163850e22b11..bd9e7d3e9c8e 100644 --- a/net/ipv6/ip6mr.c +++ b/net/ipv6/ip6mr.c | |||
@@ -658,14 +658,12 @@ static int pim6_rcv(struct sk_buff *skb) | |||
658 | skb->mac_header = skb->network_header; | 658 | skb->mac_header = skb->network_header; |
659 | skb_pull(skb, (u8 *)encap - skb->data); | 659 | skb_pull(skb, (u8 *)encap - skb->data); |
660 | skb_reset_network_header(skb); | 660 | skb_reset_network_header(skb); |
661 | skb->dev = reg_dev; | ||
662 | skb->protocol = htons(ETH_P_IPV6); | 661 | skb->protocol = htons(ETH_P_IPV6); |
663 | skb->ip_summed = 0; | 662 | skb->ip_summed = 0; |
664 | skb->pkt_type = PACKET_HOST; | 663 | skb->pkt_type = PACKET_HOST; |
665 | skb_dst_drop(skb); | 664 | |
666 | reg_dev->stats.rx_bytes += skb->len; | 665 | skb_tunnel_rx(skb, reg_dev); |
667 | reg_dev->stats.rx_packets++; | 666 | |
668 | nf_reset(skb); | ||
669 | netif_rx(skb); | 667 | netif_rx(skb); |
670 | dev_put(reg_dev); | 668 | dev_put(reg_dev); |
671 | return 0; | 669 | return 0; |
diff --git a/net/ipv6/sit.c b/net/ipv6/sit.c index 5abae10cd884..e51e650ea80b 100644 --- a/net/ipv6/sit.c +++ b/net/ipv6/sit.c | |||
@@ -566,11 +566,9 @@ static int ipip6_rcv(struct sk_buff *skb) | |||
566 | kfree_skb(skb); | 566 | kfree_skb(skb); |
567 | return 0; | 567 | return 0; |
568 | } | 568 | } |
569 | tunnel->dev->stats.rx_packets++; | 569 | |
570 | tunnel->dev->stats.rx_bytes += skb->len; | 570 | skb_tunnel_rx(skb, tunnel->dev); |
571 | skb->dev = tunnel->dev; | 571 | |
572 | skb_dst_drop(skb); | ||
573 | nf_reset(skb); | ||
574 | ipip6_ecn_decapsulate(iph, skb); | 572 | ipip6_ecn_decapsulate(iph, skb); |
575 | netif_rx(skb); | 573 | netif_rx(skb); |
576 | rcu_read_unlock(); | 574 | rcu_read_unlock(); |