On Mon, Apr 25, 2016 at 04:51:13PM -0400, Soheil Hassas Yeganeh wrote: > From: Soheil Hassas Yeganeh <soh...@google.com> > > txstamp_ack in tcp_skb_cb is set iff the SKBTX_ACK_TSTAMP > flag is set for an skb. Thus, it is not required to check > shinfo->tx_flags if the txstamp_ack bit is checked. > > Remove the check on shinfo->tx_flags & SKBTX_ACK_TSTAMP, since > it has already been checked using the txstamp_ack bit. > > Signed-off-by: Soheil Hassas Yeganeh <soh...@google.com> > --- > net/ipv4/tcp_input.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c > index 967520d..2f3fd92 100644 > --- a/net/ipv4/tcp_input.c > +++ b/net/ipv4/tcp_input.c > @@ -3087,8 +3087,7 @@ static void tcp_ack_tstamp(struct sock *sk, struct > sk_buff *skb, > return; > > shinfo = skb_shinfo(skb); > - if ((shinfo->tx_flags & SKBTX_ACK_TSTAMP) && > - !before(shinfo->tskey, prior_snd_una) && > + if (!before(shinfo->tskey, prior_snd_una) && > before(shinfo->tskey, tcp_sk(sk)->snd_una)) > __skb_tstamp_tx(skb, NULL, sk, SCM_TSTAMP_ACK); > } > -- > 2.8.0.rc3.226.g39d4020 > Acked-by: Martin KaFai Lau <ka...@fb.com>
Can it be one step further and completely remove SKBTX_ACK_TSTAMP? like what Willem has also suggested here: http://www.spinics.net/lists/netdev/msg374231.html It seems no one else is using the SKBTX_ACK_TSTAMP except TCP.
