[IPV6]: Segmentation offload not set correctly on TCP children
authorStephen Hemminger <shemminger@osdl.org>
Fri, 25 Aug 2006 22:55:43 +0000 (15:55 -0700)
committerDavid S. Miller <davem@sunset.davemloft.net>
Sun, 27 Aug 2006 01:42:01 +0000 (18:42 -0700)
TCP over IPV6 would incorrectly inherit the GSO settings.
This would cause kernel to send Tcp Segmentation Offload packets for
IPV6 data to devices that can't handle it. It caused the sky2 driver
to lock http://bugzilla.kernel.org/show_bug.cgi?id=7050
and the e1000 would generate bogus packets. I can't blame the
hardware for gagging if the upper layers feed it garbage.

This was a new bug in 2.6.18 introduced with GSO support.

Signed-off-by: Stephen Hemminger <shemminger@osdl.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/tcp_ipv6.c

index b843a650be7156ba31ee696854b2130b2a6ea0fd..802a1a6b103762c8c3639017c295cef86348b8be 100644 (file)
@@ -944,7 +944,7 @@ static struct sock * tcp_v6_syn_recv_sock(struct sock *sk, struct sk_buff *skb,
         * comment in that function for the gory details. -acme
         */
 
-       sk->sk_gso_type = SKB_GSO_TCPV6;
+       newsk->sk_gso_type = SKB_GSO_TCPV6;
        __ip6_dst_store(newsk, dst, NULL);
 
        newtcp6sk = (struct tcp6_sock *)newsk;