Re: [quicwg/base-drafts] Editorial fixes to ECN text (#2189)

[Martin Thomson <[e-mail]>]

martinthomson commented on this pull request.



> @@ -3031,33 +3031,28 @@ an ACK frame without ECN feedback, the endpoint stops setting ECT codepoints in
 subsequent IP packets, with the expectation that either the network path or the
 peer no longer supports ECN.
 
-To reduce the risk of non-standard compliant ECN markings affecting the
-operation of an endpoint, an endpoint verifies the counts it receives when it
-receives new acknowledgements:
+Network devices that corrupt or apply non-standard ECN markings might result in
+reduced throughput or other undesirable side-effects.  To reduce this risk, an
+endpoint uses the following steps to verify the counts it receives in an ACK
+frame.  Note that the counts MUST NOT be verified if the ACK frame does not

Creating any coupling between frames and packets will be annoying.  We've resisted attempts to do that in the past.

There are two conditions necessary: first that counters never decrease when put in packets with higher numbers, second: that no check is performed if the packet number is lower.  I don't think that we have the first condition either (though it is strongly implied, we don't insist upon it for other frame types).  

-- 
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/2189#discussion_r242400838