Commit 4f226674 authored by Eric Dumazet's avatar Eric Dumazet Committed by David S. Miller
Browse files

tcp: do not clear skb->csum if already zero 



Freshly allocated skbs have their csum field cleared already.

Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent a52fe46e

net/ipv4/tcp.c

+0 −1
Original line number Diff line number Diff line
@@ -658,7 +658,6 @@ void tcp_skb_entail(struct sock *sk, struct sk_buff *skb) 
	struct tcp_sock *tp = tcp_sk(sk);

	struct tcp_skb_cb *tcb = TCP_SKB_CB(skb);



	skb->csum    = 0;

	tcb->seq     = tcb->end_seq = tp->write_seq;

	tcb->tcp_flags = TCPHDR_ACK;

	tcb->sacked  = 0;

net/ipv4/tcp_output.c

+0 −1
Original line number Diff line number Diff line
@@ -2398,7 +2398,6 @@ static int tcp_mtu_probe(struct sock *sk) 
	TCP_SKB_CB(nskb)->end_seq = TCP_SKB_CB(skb)->seq + probe_size;

	TCP_SKB_CB(nskb)->tcp_flags = TCPHDR_ACK;

	TCP_SKB_CB(nskb)->sacked = 0;

	nskb->csum = 0;



	tcp_insert_write_queue_before(nskb, skb, sk);

	tcp_highest_sack_replace(sk, skb, nskb);

net/ipv6/tcp_ipv6.c

+0 −1
Original line number Diff line number Diff line
@@ -969,7 +969,6 @@ static void tcp_v6_send_response(const struct sock *sk, struct sk_buff *skb, u32 
	fl6.flowlabel = label;



	buff->ip_summed = CHECKSUM_PARTIAL;

	buff->csum = 0;



	__tcp_v6_send_check(buff, &fl6.saddr, &fl6.daddr);