diff options
| author | Alexander Drozdov <al.drozdov@gmail.com> | 2015-03-23 09:11:12 +0300 |
|---|---|---|
| committer | Mister Oyster <oysterized@gmail.com> | 2017-05-23 20:04:58 +0200 |
| commit | 21f886d7827851e7b499158da5933a37237225e5 (patch) | |
| tree | 724057c697085f5a7a712437291c9d9ffd014d23 /net/packet | |
| parent | ac7d172ae53206b3197db4d07644a978aaac359c (diff) | |
af_packet: make tpacket_rcv to not set status value before run_filter
It is just an optimization. We don't need the value of status variable
if the packet is filtered.
Signed-off-by: Alexander Drozdov <al.drozdov@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: mydongistiny <jaysonedson@gmail.com>
Diffstat (limited to 'net/packet')
| -rw-r--r-- | net/packet/af_packet.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c index 18fef23f7..2b6757c2d 100644 --- a/net/packet/af_packet.c +++ b/net/packet/af_packet.c @@ -1761,14 +1761,15 @@ static int tpacket_rcv(struct sk_buff *skb, struct net_device *dev, } } - if (skb->ip_summed == CHECKSUM_PARTIAL) - status |= TP_STATUS_CSUMNOTREADY; - snaplen = skb->len; res = run_filter(skb, sk, snaplen); if (!res) goto drop_n_restore; + + if (skb->ip_summed == CHECKSUM_PARTIAL) + status |= TP_STATUS_CSUMNOTREADY; + if (snaplen > res) snaplen = res; |
