Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

congestion window should be reduced after RTO when no pacing #2007

Closed
janaiyengar opened this issue Nov 15, 2018 · 2 comments
Closed

congestion window should be reduced after RTO when no pacing #2007

janaiyengar opened this issue Nov 15, 2018 · 2 comments
Labels
-recovery design An issue that affects the design of the protocol; resolution requires consensus. has-consensus An issue that the Chairs have determined has consensus, by canvassing the mailing list.

Comments

@janaiyengar
Copy link
Contributor

Otherwise, the sender can burst the entire cwnd out on a single ack.
@martinthomson martinthomson added design An issue that affects the design of the protocol; resolution requires consensus. -recovery labels Nov 15, 2018
ianswett added a commit that referenced this issue Nov 20, 2018
@ianswett
Restarting from idle
4149b3c 
Fixes #2007
@ianswett ianswett mentioned this issue Nov 20, 2018
Merged
@siyengar
Copy link

do you mean after RTO verified?

@janaiyengar
Copy link
Contributor Author

From the email discussion with @pravb, just limit always, since a large amount of time has passed and there's probably no ack clock. Again, only when not pacing.

ianswett added a commit that referenced this issue Nov 20, 2018
@ianswett
Restarting from idle (#2023)
13e3796 
* Restarting from idle

Fixes #2007

* Update draft-ietf-quic-recovery.md

* Update draft-ietf-quic-recovery.md

* Update draft-ietf-quic-recovery.md

Co-Authored-By: ianswett <ianswett@users.noreply.github.com>

* Update draft-ietf-quic-recovery.md

Jana's comments
@mnot mnot added the has-consensus An issue that the Chairs have determined has consensus, by canvassing the mailing list. label Mar 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
-recovery design An issue that affects the design of the protocol; resolution requires consensus. has-consensus An issue that the Chairs have determined has consensus, by canvassing the mailing list.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@martinthomson @mnot @siyengar @janaiyengar