[quicwg/base-drafts] Persistent Congestion Time Threshold (#2365)

[Nick Banks <[e-mail]>]

Updates the persistent congestion logic to be time based instead of PTO count based, to account for the cases where PTOs don't actually occur.

This is the simplest change I could come up with that fixes the main issue, though I think a completely separate timer would still be better than piggy-backing off the `CongestionEvent`. For instance, in the case the app is periodically sending (and therefore no PTOs occur), if the ACK eventually comes much later than when persistent congestion occurs (or should occur) the CC window only decreases when the ACK arrives; not when persistent congestion started. If you have a separate timer for persistent congestion the window could be reduced immediately instead of waiting for the ACK. For now, though, I chose to go with the simpler change/solution that still mostly gets the job done.
You can view, comment on, or merge this pull request online at:

  https://github.com/quicwg/base-drafts/pull/2365

-- Commit Summary --

  * Persistent Congestion Time Threshold

-- File Changes --

    M draft-ietf-quic-recovery.md (40)

-- Patch Links --

https://github.com/quicwg/base-drafts/pull/2365.patch
https://github.com/quicwg/base-drafts/pull/2365.diff

-- 
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/2365