diff options
| author | Simon Horman <horms@verge.net.au> | 2010-08-22 17:35:32 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-08-22 21:09:04 -0700 |
| commit | c2368e795cec561229ef66a04ac51629b918a9e8 (patch) | |
| tree | 2fd08484e8948dc2969ac2d01b00af175b93abb5 /net/bridge | |
| parent | fcb12fd2236f49aa8fdc1568ed4ebdfe4fddc6b5 (diff) | |
| download | op-kernel-dev-c2368e795cec561229ef66a04ac51629b918a9e8.zip op-kernel-dev-c2368e795cec561229ef66a04ac51629b918a9e8.tar.gz | |
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')
| -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 826cd52..6d04cfd 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) const unsigned char *dest = eth_hdr(skb)->h_dest; int (*rhook)(struct sk_buff *skb); - if (skb->pkt_type == PACKET_LOOPBACK) + if (unlikely(skb->pkt_type == PACKET_LOOPBACK)) return skb; if (!is_valid_ether_addr(eth_hdr(skb)->h_source)) |
