Re: [RFC PATCH] [TCP]: Reworked recovery's TCPCB_LOST marking functions

From: Baruch Even <redacted>
Date: Tue, 6 Mar 2007 21:42:59 +0200

* Ilpo J?rvinen [off-list ref] [070306 14:52]:

quoted

+		newtp->highest_sack = treq->snt_isn + 1;

That's the only initialization that you have for highest_sack, I think
that you should initialize it when a loss is detected to the start_seq
of the first packet that wasn't acked.

He also sets it in tcp_sacktag_write_queue() like this:

+
+				if (after(TCP_SKB_CB(skb)->seq,
+				    tp->highest_sack))
+					tp->highest_sack = TCP_SKB_CB(skb)->seq;