Re: [quicwg/base-drafts] Fix to ECN section regarding validation (#2113)

[Benjamin Saunders <[e-mail]>]

Ralith commented on this pull request.



>  
 * The total increase in ECT(0), ECT(1), and CE counters reported in the ACK
   frame MUST be at least the total number of QUIC packets newly acknowledged in
-  this ACK frame.
+  this ACK frame. Detects if the network remark ECT(0), ECT(1) or CE to
+  Not-ECT.
+
+If a sender receives an ACK that contains no new acknowledgments, for example
+due to reordering of the ACKs, then ECN counter comparison SHOULD NOT be
+performed. Also if sender do not have state to determine if a particular PSN
+is newly acknowledge or not, then the comparison SHOULD NOT be performed.

This reads to me like the comparison of the new ECN block to the local reference (i.e. the verification procedure) should not be performed if the ACK acknowledges any packets which are outside the window and hence are not guaranteed not to be new acknowledgements. Should it instead indicate that such an acknowledgement should not be counted towards new acknowledgements?

-- 
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/2113#discussion_r241110556