Subject: Re: [quicwg/base-drafts] Unify TLP and RTO into Probe Timeout (#2114)
From: janaiyengar <[email-masked]>
To: quicwg/base-drafts <[email-masked]>
Cc: Subscribed <[email-masked]>
Date: Thu, 13 Dec 2018 23:49:08 -0800
List-Id: Notification list for GitHub issues related to the QUIC WG <quic-issues.ietf.org>

janaiyengar commented on this pull request.

> @@ -1157,6 +1095,9 @@ window.
        congestion_window *= kLossReductionFactor
        congestion_window = max(congestion_window, kMinimumWindow)
        ssthresh = congestion_window
+       // Collapse congestion window if persistent congestion
+       if (pto_count > 2):

@siyengar : In this case, by the time 5 is acked, 3 and 4 should be marked as lost by the time-threshold loss detection mechanism, since they're going to be > 1.125 x SRTT in the past.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/pull/2114#discussion_r241666519