diff options
author | Alexander Duyck <aduyck@mirantis.com> | 2016-05-02 09:38:12 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-05-04 13:32:26 -0400 |
commit | d7fb5a80492169cd737d51042f4ee03b09be0ef6 (patch) | |
tree | 86b2631c7d53100b87166f8b051447b5e9eb6987 /net/core | |
parent | f132ae7c46370c981412a68ccec9f2145812a9b6 (diff) | |
download | linux-stable-d7fb5a80492169cd737d51042f4ee03b09be0ef6.tar.gz linux-stable-d7fb5a80492169cd737d51042f4ee03b09be0ef6.tar.bz2 linux-stable-d7fb5a80492169cd737d51042f4ee03b09be0ef6.zip |
gso: Do not perform partial GSO if number of partial segments is 1 or less
In the event that the number of partial segments is equal to 1 we don't
really need to perform partial segmentation offload. As such we should
skip multiplying the MSS and instead just clear the partial_segs value
since it will not provide any gain to advertise the frame as being GSO when
it is a single frame.
Signed-off-by: Alexander Duyck <aduyck@mirantis.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core')
-rw-r--r-- | net/core/skbuff.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/net/core/skbuff.c b/net/core/skbuff.c index 7a1d48983f81..b8dd2d2e2256 100644 --- a/net/core/skbuff.c +++ b/net/core/skbuff.c @@ -3101,7 +3101,10 @@ struct sk_buff *skb_segment(struct sk_buff *head_skb, */ if (features & NETIF_F_GSO_PARTIAL) { partial_segs = len / mss; - mss *= partial_segs; + if (partial_segs > 1) + mss *= partial_segs; + else + partial_segs = 0; } headroom = skb_headroom(head_skb); |