Re: [quicwg/base-drafts] Gorry's ECN rewrite (#4059)

[Gorry Fairhurst <[e-mail]>]

@gorryfair commented on this pull request.



>  
 
 ### ECN Counts
 
-On receiving a QUIC packet with an ECT or CE codepoint, an ECN-enabled endpoint
-that can access the ECN codepoints from the enclosing IP packet increases the
-corresponding ECT(0), ECT(1), or CE count, and includes these counts in
-subsequent ACK frames; see {{generating-acks}} and {{frame-ack}}.
-
-An IP packet that results in no QUIC packets being processed does not increase
-ECN counts.  A QUIC packet detected by a receiver as a duplicate does not
-affect the receiver's local ECN codepoint counts; see {{security-ecn}} for
-relevant security concerns.
+Use of ECN requires the receiving endpoint to read the ECN codepoint from an IP

Can I suggest: /To enable the congestion controller to react to CE-marked packets, the sender needs to be able to set an ECT codepoint in the IP header and the receiver needs to be able to read the ECN field from an IP packet./
...i.e.,  removing /in order/ ... /ECN code point/ is the /ECN-field/ or name the code point value(s).

-- 
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/4059#discussion_r486124740