tcp: remove SKBTX_ACK_TSTAMP since it is redundant
authorSoheil Hassas Yeganeh <soheil@google.com>
Thu, 28 Apr 2016 03:39:01 +0000 (23:39 -0400)
committerDavid S. Miller <davem@davemloft.net>
Thu, 28 Apr 2016 20:06:10 +0000 (16:06 -0400)
commit0a2cf20c3fb62ad4717276b5303bf831f7b29d54
tree8ac718cd55a6ec72241c647e38856d904797a078
parent863c1fd9814618eefba02218f8fadf8a430c2a17
tcp: remove SKBTX_ACK_TSTAMP since it is redundant

The SKBTX_ACK_TSTAMP flag is set in skb_shinfo->tx_flags when
the timestamp of the TCP acknowledgement should be reported on
error queue. Since accessing skb_shinfo is likely to incur a
cache-line miss at the time of receiving the ack, the
txstamp_ack bit was added in tcp_skb_cb, which is set iff
the SKBTX_ACK_TSTAMP flag is set for an skb. This makes
SKBTX_ACK_TSTAMP flag redundant.

Remove the SKBTX_ACK_TSTAMP and instead use the txstamp_ack bit
everywhere.

Note that this frees one bit in shinfo->tx_flags.

Signed-off-by: Soheil Hassas Yeganeh <soheil@google.com>
Acked-by: Martin KaFai Lau <kafai@fb.com>
Suggested-by: Willem de Bruijn <willemb@google.com>
Acked-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/skbuff.h
net/ipv4/tcp.c
net/ipv4/tcp_input.c
net/ipv4/tcp_output.c
net/socket.c