Re: [PATCH net] tcp: prevent bogus FRTO undos with non-SACK flows

2018-07-02 Thread Ilpo Järvinen
On Sat, 30 Jun 2018, Neal Cardwell wrote: > As I mentioned, I ran your patch through all our team's TCP > packetdrill tests, and it passes all of the tests. One of our tests > needed updating, because if there is a non-SACK connection with a > spurious RTO due to a delayed flight of ACKs then the

Re: [PATCH net] tcp: prevent bogus FRTO undos with non-SACK flows

2018-07-01 Thread David Miller
From: Ilpo Järvinen Date: Fri, 29 Jun 2018 13:07:53 +0300 (EEST) > If SACK is not enabled and the first cumulative ACK after the RTO > retransmission covers more than the retransmitted skb, a spurious > FRTO undo will trigger (assuming FRTO is enabled for that RTO). > The reason is that any non-r

Re: [PATCH net] tcp: prevent bogus FRTO undos with non-SACK flows

2018-06-30 Thread Neal Cardwell
On Fri, Jun 29, 2018 at 3:52 PM Ilpo Järvinen wrote: > > +.005 < . 1:1(0) ack 2001 win 257 > > Why did the receiver send a cumulative ACK only for 2001? Sorry, you are right Ilpo. Upon further reflection, the packetdrill scenario I posted is not a realistic one, and I agree we should not worry ab

Re: [PATCH net] tcp: prevent bogus FRTO undos with non-SACK flows

2018-06-29 Thread Ilpo Järvinen
On Fri, 29 Jun 2018, Neal Cardwell wrote: > On Fri, Jun 29, 2018 at 6:07 AM Ilpo Järvinen > wrote: > > > > If SACK is not enabled and the first cumulative ACK after the RTO > > retransmission covers more than the retransmitted skb, a spurious > > FRTO undo will trigger (assuming FRTO is enabled

Re: [PATCH net] tcp: prevent bogus FRTO undos with non-SACK flows

2018-06-29 Thread Neal Cardwell
On Fri, Jun 29, 2018 at 6:07 AM Ilpo Järvinen wrote: > > If SACK is not enabled and the first cumulative ACK after the RTO > retransmission covers more than the retransmitted skb, a spurious > FRTO undo will trigger (assuming FRTO is enabled for that RTO). > The reason is that any non-retransmitte

[PATCH net] tcp: prevent bogus FRTO undos with non-SACK flows

2018-06-29 Thread Ilpo Järvinen
If SACK is not enabled and the first cumulative ACK after the RTO retransmission covers more than the retransmitted skb, a spurious FRTO undo will trigger (assuming FRTO is enabled for that RTO). The reason is that any non-retransmitted segment acknowledged will set FLAG_ORIG_SACK_ACKED in tcp_clea