Commit abbdb5a7 authored by Eric Dumazet's avatar Eric Dumazet Committed by David S. Miller

net: remove a dubious unlikely() clause

TCP protocol is still used these days, and TCP uses
clones in its transmit path. We can not optimize linux
stack assuming it is mostly used in routers, or that TCP
is dead.

Fixes: 795bb1c0 ("net: bulk free infrastructure for NAPI context, use napi_consume_skb")
Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Cc: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 5a779c4f
...@@ -815,7 +815,7 @@ void napi_consume_skb(struct sk_buff *skb, int budget) ...@@ -815,7 +815,7 @@ void napi_consume_skb(struct sk_buff *skb, int budget)
trace_consume_skb(skb); trace_consume_skb(skb);
/* if SKB is a clone, don't handle this case */ /* if SKB is a clone, don't handle this case */
if (unlikely(skb->fclone != SKB_FCLONE_UNAVAILABLE)) { if (skb->fclone != SKB_FCLONE_UNAVAILABLE) {
__kfree_skb(skb); __kfree_skb(skb);
return; return;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment