diff options
| author |   Alexander Drozdov <al.drozdov@gmail.com> | 2015-03-23 09:11:12 +0300 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2015-03-23 22:00:36 -0400 |
| commit | 68c2e5de360411674d9821ee2b46f5d8ee965161 (patch) | |
| tree | e68cac5be8d3c5fb6e5b47c48e48bd9c8703e8de /net/packet | |
| parent | inet: fix double request socket freeing (diff) | |
| download | linux-dev-68c2e5de360411674d9821ee2b46f5d8ee965161.tar.xz linux-dev-68c2e5de360411674d9821ee2b46f5d8ee965161.zip | |
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>
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 b91ac5946ad1..9d854c5ce0b5 100644 --- a/net/packet/af_packet.c +++ b/net/packet/af_packet.c @@ -1916,14 +1916,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; |