gso: Do not perform partial GSO if number of partial segments is 1 or less
authorAlexander Duyck <aduyck@mirantis.com>
Mon, 2 May 2016 16:38:12 +0000 (09:38 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 4 May 2016 17:32:26 +0000 (13:32 -0400)
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>
net/core/skbuff.c

index 7a1d489..b8dd2d2 100644 (file)
@@ -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);