diff options
author | Simon Horman <horms@verge.net.au> | 2010-08-22 13:35:32 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-08-23 00:09:04 -0400 |
commit | c2368e795cec561229ef66a04ac51629b918a9e8 (patch) | |
tree | 2fd08484e8948dc2969ac2d01b00af175b93abb5 /net/bridge/br_input.c | |
parent | fcb12fd2236f49aa8fdc1568ed4ebdfe4fddc6b5 (diff) |
bridge: is PACKET_LOOPBACK unlikely()?
While looking at using netdev_rx_handler_register for openvswitch Jesse
Gross suggested that an unlikely() might be worthwhile in that code.
I'm interested to see if its appropriate for the bridge code.
Cc: Jesse Gross <jesse@nicira.com>
Signed-off-by: Simon Horman <horms@verge.net.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/bridge/br_input.c')
-rw-r--r-- | net/bridge/br_input.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/bridge/br_input.c b/net/bridge/br_input.c index 826cd5221536..6d04cfdf4541 100644 --- a/net/bridge/br_input.c +++ b/net/bridge/br_input.c | |||
@@ -141,7 +141,7 @@ struct sk_buff *br_handle_frame(struct sk_buff *skb) | |||
141 | const unsigned char *dest = eth_hdr(skb)->h_dest; | 141 | const unsigned char *dest = eth_hdr(skb)->h_dest; |
142 | int (*rhook)(struct sk_buff *skb); | 142 | int (*rhook)(struct sk_buff *skb); |
143 | 143 | ||
144 | if (skb->pkt_type == PACKET_LOOPBACK) | 144 | if (unlikely(skb->pkt_type == PACKET_LOOPBACK)) |
145 | return skb; | 145 | return skb; |
146 | 146 | ||
147 | if (!is_valid_ether_addr(eth_hdr(skb)->h_source)) | 147 | if (!is_valid_ether_addr(eth_hdr(skb)->h_source)) |