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/skbuff.c | |
| parent | gre: change gre_parse_header to return the header length (diff) | |
| download | linux-dev-d7fb5a80492169cd737d51042f4ee03b09be0ef6.tar.xz linux-dev-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 '')
| -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); |