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

More strongly recommend ECN marking #3373

Closed
martinthomson opened this issue Jan 21, 2020 · 6 comments
Closed

More strongly recommend ECN marking #3373

martinthomson opened this issue Jan 21, 2020 · 6 comments
Labels
-transport design has-consensus

Comments

@martinthomson
Copy link
Member

@martinthomson martinthomson commented Jan 21, 2020

In #3320, @mirjak suggested that a different algorithm might be better to recommend enabling ECN and only disabling it if a failure is detected.

That algorithm adopts the same posture as the draft: that ECN is optional and that endpoints are permitted to be cautious in enabling it. If we want to adopt the view that ECN works and endpoints are required to use it unless it breaks, that's a bigger change.

I open this issue, while being aware that this is re-opening something we discussed and agreed previously, without there being new information to present. Chairs, if you do think we should discuss this, it's a design issue.

@ekr
Copy link
Collaborator

@ekr ekr commented Feb 4, 2020

I would not be in favor of reopening this.

@mirjak
Copy link
Contributor

@mirjak mirjak commented Feb 4, 2020

Just to clarify the proposal was that if you decide to enable ECN, you should just do it and only disable if you notice a failure, instead of having the more complicated probing logic that we have right now (in the appendix I think)

@larseggert larseggert added the design label Feb 4, 2020
@project-bot project-bot bot moved this from Triage to Design Issues in Late Stage Processing Feb 4, 2020
@huitema
Copy link
Contributor

@huitema huitema commented Feb 5, 2020

The Interop Matrix shows that out of 14 implementations doing active tests, only 3 demonstrate ECN support. No ill will, but major places like AWS or the Windows OS do not support ECN, so it is not clear what more text in the spec would change.

@martinthomson
Copy link
Member Author

@martinthomson martinthomson commented Feb 5, 2020

This is just about this one paragraph:

To reduce the chances of misinterpreting congestive loss as packets dropped by a faulty network element, an endpoint could set the ECT(0) codepoint in the first ten outgoing packets on a path, or for a period of three RTTs, whichever occurs first.

The concrete proposal would be to remove this paragraph.

@larseggert
Copy link
Member

@larseggert larseggert commented Feb 5, 2020

Discussed in ZRH. Proposed resolution is to close with no action.

@larseggert larseggert added the proposal-ready label Feb 5, 2020
@project-bot project-bot bot moved this from Design Issues to Consensus Emerging in Late Stage Processing Feb 5, 2020
@mirjak
Copy link
Contributor

@mirjak mirjak commented Feb 5, 2020

I will make a proposal for an editorial PR for the paragraph mention above to make it sounds less scary that this risk exists.

@LPardue LPardue moved this from Consensus Emerging to Consensus Call issued in Late Stage Processing Feb 19, 2020
@LPardue LPardue removed the proposal-ready label Feb 19, 2020
@LPardue LPardue added the call-issued label Feb 26, 2020
@LPardue LPardue added has-consensus and removed call-issued labels Mar 4, 2020
@project-bot project-bot bot moved this from Consensus Call issued to Consensus Declared in Late Stage Processing Mar 4, 2020
Late Stage Processing automation moved this from Consensus Declared to Text Incorporated Mar 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-transport design has-consensus
Projects
Late Stage Processing
  
Issue Handled
Development

No branches or pull requests

6 participants