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.

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

Mixture of transport and congestion control functions #2684

Closed
gorryfair opened this issue May 10, 2019 · 3 comments
Closed

Mixture of transport and congestion control functions #2684

gorryfair opened this issue May 10, 2019 · 3 comments
Assignees
Labels
-transport editorial An issue that does not affect the design of the protocol; does not require consensus.

Comments

@gorryfair
Copy link
Contributor

Section 9.4 contains a mixture of transport (handling migration, setup, etc) and congestion control functions (choice of path parameters - RTO, window, etc).

The abstract suggests that these are a part of QUIC Recovery ID, which seems correct to me. So, I would like to discuss if these are intentionally a part of the transport spec, rather than the Loss Detection & Congestion Control Spec.?

Specifically, para 1,2 : "... SHOULD NOT contribute to congestion control or RTT estimation for the new path." and "... SHOULD immediately reset ...." - Where the ID could note whatever transport implementors need to be aware of, but point normatively to the QUIC Recovery ID for RFC2119 language.

Para 3: "An endpoint MUST NOT return to the send rate..." seems a reasonable thing to note, but also, I think, should point to QUIC Recovery ID.

@larseggert larseggert changed the title Section 9.4: Contains a mixture of transport and congestion control functions Mixture of transport and congestion control functions May 10, 2019
@mnot mnot added this to Triage in Late Stage Processing May 22, 2019
@larseggert larseggert added the editorial An issue that does not affect the design of the protocol; does not require consensus. label May 22, 2019
@mnot mnot moved this from Triage to Editorial Issues in Late Stage Processing May 22, 2019
@larseggert
Copy link
Member

Discussed in London: @gorryfair to work with @janaiyengar to propose an editorial PR

@ianswett
Copy link
Contributor

The target is to fix this issue with the same PR as #2909

@martinthomson
Copy link
Member

Discussed today with editors and we believe that these issues have been rectified in the numerous edits that section has received since this issue was opened.

Late Stage Processing automation moved this from Editorial Issues to Issue Handled Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-transport editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
Late Stage Processing
  
Issue Handled
Development

No branches or pull requests

4 participants