diff options
| author |   Eric Dumazet <edumazet@google.com> | 2015-11-18 06:30:50 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2015-11-18 16:17:37 -0500 |
| commit | 52bd2d62ce6758d811edcbd2256eb9ea7f6a56cb (patch) | |
| tree | a05cc3913f4389700efef15cccf882befbcd6d32 /include/linux/skbuff.h | |
| parent | be2net: remove local variable 'status' (diff) | |
| download | linux-dev-52bd2d62ce6758d811edcbd2256eb9ea7f6a56cb.tar.xz linux-dev-52bd2d62ce6758d811edcbd2256eb9ea7f6a56cb.zip | |
net: better skb->sender_cpu and skb->napi_id cohabitation
skb->sender_cpu and skb->napi_id share a common storage,
and we had various bugs about this.
We had to call skb_sender_cpu_clear() in some places to
not leave a prior skb->napi_id and fool netdev_pick_tx()
As suggested by Alexei, we could split the space so that
these errors can not happen.
0 value being reserved as the common (not initialized) value,
let's reserve [1 .. NR_CPUS] range for valid sender_cpu,
and [NR_CPUS+1 .. ~0U] for valid napi_id.
This will allow proper busy polling support over tunnels.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Suggested-by: Alexei Starovoitov <ast@kernel.org>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/skbuff.h')
| -rw-r--r-- | include/linux/skbuff.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index 4355129fff91..c9c394bf0771 100644 --- a/include/linux/skbuff.h +++ b/include/linux/skbuff.h @@ -1082,9 +1082,6 @@ static inline void skb_copy_hash(struct sk_buff *to, const struct sk_buff *from) static inline void skb_sender_cpu_clear(struct sk_buff *skb) { -#ifdef CONFIG_XPS - skb->sender_cpu = 0; -#endif } #ifdef NET_SKBUFF_DATA_USES_OFFSET |