diff options
| author |   Paolo Abeni <pabeni@redhat.com> | 2017-06-12 11:23:42 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2017-06-12 10:01:29 -0400 |
| commit | 0a463c78d25b9464b77311d9dda297550a2d6aa5 (patch) | |
| tree | 87152851619730150299fe4acf3a7a0eff8a6f4b /include/linux | |
| parent | net: factor out a helper to decrement the skb refcount (diff) | |
| download | linux-dev-0a463c78d25b9464b77311d9dda297550a2d6aa5.tar.xz linux-dev-0a463c78d25b9464b77311d9dda297550a2d6aa5.zip | |
udp: avoid a cache miss on dequeue
Since UDP no more uses sk->destructor, we can clear completely
the skb head state before enqueuing. Amend and use
skb_release_head_state() for that.
All head states share a single cacheline, which is not
normally used/accesses on dequeue. We can avoid entirely accessing
such cacheline implementing and using in the UDP code a specialized
skb free helper which ignores the skb head state.
This saves a cacheline miss at skb deallocation time.
v1 -> v2:
replaced secpath_reset() with skb_release_head_state()
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Acked-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/skbuff.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index decce3655a48..d66d4feaac86 100644 --- a/include/linux/skbuff.h +++ b/include/linux/skbuff.h @@ -880,10 +880,12 @@ static inline bool skb_unref(struct sk_buff *skb) return true; } +void skb_release_head_state(struct sk_buff *skb); void kfree_skb(struct sk_buff *skb); void kfree_skb_list(struct sk_buff *segs); void skb_tx_error(struct sk_buff *skb); void consume_skb(struct sk_buff *skb); +void consume_stateless_skb(struct sk_buff *skb); void __kfree_skb(struct sk_buff *skb); extern struct kmem_cache *skbuff_head_cache; |