Skip to content
This repository has been archived by the owner on Aug 28, 2023. It is now read-only.

Section 8 #99

Closed
larseggert opened this issue Aug 31, 2021 · 1 comment · Fixed by #104 or #105
Closed

Section 8 #99

larseggert opened this issue Aug 31, 2021 · 1 comment · Fixed by #104 or #105
Assignees

Comments

@larseggert
Copy link
Contributor

Markku Kojo said:

The draft says in the intro that CUBIC is to be regarded as current standard for TCP congestion control. It sounds a bit like it would obsolete RFC 5681 which is not the intent. RFC 5681 still has its specific role as the document that gives the baseline and generic guidelines for TCP (and other) congestion control.
Instead, I think this document should articulate very carefully its role among the congestion control algorithms. How, I am not sure. Maybe simply as an alternative for RFC5681 congestion avoidance and multiplicative decrease.

Please note also that when specifying these algorithms this document is
in direct conflict with a MUST in RFC5681 which says: "however, a TCP MUST NOT be more aggressive than the following algorithms allow (that is, MUST NOT send data when the value of cwnd computed by the following algorithms would not allow the data to be sent)."
Therefore, the draft should make this differentation very clear maybe already in the abstract and justify the deviations much better than it currently does (accompanied with evidence = data). This is very important i order to make a convincing case why it is ok for this doc to deviate from the current Standards Track TCP normative statements.

@larseggert larseggert self-assigned this Aug 31, 2021
@larseggert
Copy link
Contributor Author

larseggert commented Aug 31, 2021

The first part is hopefully addressed by #104.

The second part sounds like we would need to formally update 5681, to allow for CUBIC's behavior? I have a proposal for this in #105.

@larseggert larseggert linked a pull request Aug 31, 2021 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
1 participant