diff options
author | Vlad Yasevich <vyasevic@redhat.com> | 2014-03-27 17:32:30 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-03-31 16:04:04 -0400 |
commit | f6367b4660dde412f9b7af94763efb1d89cefb74 (patch) | |
tree | fe623c6c6c86cde1718238329a34b728185cc4f3 /net | |
parent | 1ee481fb4cf8044632fa869bafc41345afa5957b (diff) |
bridge: use is_skb_forwardable in forward path
Use existing function instead of trying to use our own.
Signed-off-by: Vlad Yasevich <vyasevic@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r-- | net/bridge/br_forward.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/net/bridge/br_forward.c b/net/bridge/br_forward.c index d3409e6b5453..056b67b0e277 100644 --- a/net/bridge/br_forward.c +++ b/net/bridge/br_forward.c | |||
@@ -35,16 +35,11 @@ static inline int should_deliver(const struct net_bridge_port *p, | |||
35 | p->state == BR_STATE_FORWARDING; | 35 | p->state == BR_STATE_FORWARDING; |
36 | } | 36 | } |
37 | 37 | ||
38 | static inline unsigned int packet_length(const struct sk_buff *skb) | ||
39 | { | ||
40 | return skb->len - (skb->protocol == htons(ETH_P_8021Q) ? VLAN_HLEN : 0); | ||
41 | } | ||
42 | |||
43 | int br_dev_queue_push_xmit(struct sk_buff *skb) | 38 | int br_dev_queue_push_xmit(struct sk_buff *skb) |
44 | { | 39 | { |
45 | /* ip_fragment doesn't copy the MAC header */ | 40 | /* ip_fragment doesn't copy the MAC header */ |
46 | if (nf_bridge_maybe_copy_header(skb) || | 41 | if (nf_bridge_maybe_copy_header(skb) || |
47 | (packet_length(skb) > skb->dev->mtu && !skb_is_gso(skb))) { | 42 | !is_skb_forwardable(skb->dev, skb)) { |
48 | kfree_skb(skb); | 43 | kfree_skb(skb); |
49 | } else { | 44 | } else { |
50 | skb_push(skb, ETH_HLEN); | 45 | skb_push(skb, ETH_HLEN); |
@@ -71,7 +66,7 @@ static void __br_deliver(const struct net_bridge_port *to, struct sk_buff *skb) | |||
71 | skb->dev = to->dev; | 66 | skb->dev = to->dev; |
72 | 67 | ||
73 | if (unlikely(netpoll_tx_running(to->br->dev))) { | 68 | if (unlikely(netpoll_tx_running(to->br->dev))) { |
74 | if (packet_length(skb) > skb->dev->mtu && !skb_is_gso(skb)) | 69 | if (!is_skb_forwardable(skb->dev, skb)) |
75 | kfree_skb(skb); | 70 | kfree_skb(skb); |
76 | else { | 71 | else { |
77 | skb_push(skb, ETH_HLEN); | 72 | skb_push(skb, ETH_HLEN); |