diff options
author | Herbert Xu <herbert@gondor.apana.org.au> | 2017-01-10 15:24:01 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-01-15 07:42:55 -0500 |
commit | ec0fdcb88c6f451dc3d436406de69fbf48a2e1da (patch) | |
tree | a5c3b2f8c7eb41fadf79b8415bc2d4bee3b45c5f /net/core | |
parent | 33364eee1fe4b1bdff89312645ee3a624858dbc9 (diff) |
gro: Enter slow-path if there is no tailroom
[ Upstream commit 1272ce87fa017ca4cf32920764d879656b7a005a ]
The GRO path has a fast-path where we avoid calling pskb_may_pull
and pskb_expand by directly accessing frag0. However, this should
only be done if we have enough tailroom in the skb as otherwise
we'll have to expand it later anyway.
This patch adds the check by capping frag0_len with the skb tailroom.
Fixes: cb18978cbf45 ("gro: Open-code final pskb_may_pull")
Reported-by: Slava Shwartsman <slavash@mellanox.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/core')
-rw-r--r-- | net/core/dev.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/core/dev.c b/net/core/dev.c index 6666b28b6815..7b4fa1e4e4d8 100644 --- a/net/core/dev.c +++ b/net/core/dev.c | |||
@@ -4453,7 +4453,8 @@ static void skb_gro_reset_offset(struct sk_buff *skb) | |||
4453 | pinfo->nr_frags && | 4453 | pinfo->nr_frags && |
4454 | !PageHighMem(skb_frag_page(frag0))) { | 4454 | !PageHighMem(skb_frag_page(frag0))) { |
4455 | NAPI_GRO_CB(skb)->frag0 = skb_frag_address(frag0); | 4455 | NAPI_GRO_CB(skb)->frag0 = skb_frag_address(frag0); |
4456 | NAPI_GRO_CB(skb)->frag0_len = skb_frag_size(frag0); | 4456 | NAPI_GRO_CB(skb)->frag0_len = min(skb_frag_size(frag0), |
4457 | skb->end - skb->tail); | ||
4457 | } | 4458 | } |
4458 | } | 4459 | } |
4459 | 4460 | ||