Re: [quicwg/base-drafts] Make language in ECN validation clearer (#3777)

[Jana Iyengar <[e-mail]>]

@janaiyengar approved this pull request.

A few suggestions, but LGTM.

> +An endpoint that sets ECT(0) or ECT(1) codepoints on packets records the ECN
+counts it has received in ACK frames. When processing an ACK frame, the
+endpoint MUST validate the increase in ECN counts based on the markings that
+were applied to packets that are newly acknowledged.

```suggestion
An endpoint that sets ECT(0) or ECT(1) codepoints on packets records the ECN
counts it receives in ACK frames. When processing an ACK frame, the endpoint
MUST validate any increases from previously received ECN counts, as follows:
```
... and drop the line below about ECN validation.

>  
-* For validation to succeed, the total increase in ECT(0), ECT(1), and CE counts
-  MUST be no smaller than the total number of QUIC packets sent with an ECT
-  codepoint that are newly acknowledged in this ACK frame.  This step detects
-  any network remarking from ECT(0), ECT(1), or CE codepoints to Not-ECT.
+* ECN validation fails if the sum of the increases to ECT(0), ECT(1), and CE
+  counts in the ACK frame are less than the number of newly acknowledged
+  packets that were sent with an ECT codepoint.  This step detects any
+  network remarking from ECT(0), ECT(1), or CE codepoints to Not-ECT.

```suggestion
  erroneous network overwriting from ECT(0), ECT(1), or CE codepoints to
  Not-ECT.
```

>    This step detects any erroneous network remarking from ECT(0) to ECT(1) (or
   vice versa).

```suggestion
  This step detects any erroneous network overwriting from ECT(0) to ECT(1) (or
  vice versa).
```

> +
+ECN validation involves the following checks:

```suggestion
```

-- 
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/3777#pullrequestreview-433642378