[quicwg/base-drafts] draft-ietf-quic-recovery-13 ssthresh never set (#1598)

"Arvid E. Picciani" <notifications@github.com> Tue, 24 July 2018 16:31 UTC

Date: Tue, 24 Jul 2018 16:31:53 +0000
From: "Arvid E. Picciani" <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab958a24219514f73e8370752077e18618d84bb49292cf00000001176f167992a169ce1482c3ac@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/1598@github.com>
Subject: [quicwg/base-drafts] draft-ietf-quic-recovery-13 ssthresh never set (#1598)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5b575479767cc_32a03fe4d20d45c050114"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/-PNQ7hzHMc6qNQsv6KsHGsMvtiU>

in 4.3.  Congestion Avoidance

>  and sets the slow start threshold to the new congestion window.

however, the pseudo code doesnt do that

i'm guessing that CongestionEvent should do that?

```diff
      CongestionEvent(packet_number):
        // Start a new congestion event if packet_number
        // is larger than the end of the previous recovery epoch.
        if (!InRecovery(packet_number)):
          end_of_recovery = largest_sent_packet
          congestion_window *= kMarkReductionFactor
          congestion_window = max(congestion_window, kMinimumWindow)
+         ssthreshold = congestion_window
```

-- 
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/issues/1598

[quicwg/base-drafts] draft-ietf-quic-recovery-13 … Arvid E. Picciani
Re: [quicwg/base-drafts] draft-ietf-quic-recovery… ianswett
Re: [quicwg/base-drafts] draft-ietf-quic-recovery… ianswett