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
ECN counters and Coalescing of QUIC packets #1838
Comments
Sounds good. I'm sure that we'd appreciate getting a PR for this. |
So how do we deal with the different packet number spaces here. I see two options: |
I'm thinking that you count separately, in the interests of keeping ACK accounting separate (and consistent). But no special handling, or disabling ECN for different phases. You can still use Initial/Handshake to verify that the path supports ECN. |
Agreed, each PN space has its own set of counters, and an ACK frame sent in a PN space conveys the counters of that PN space. |
This deals with the distinction between IP and QUIC packets better, clarifies the interaction with packet coalescing, and makes it clear that counts are ACK state and therefore separated by packet number space. Closes #1838.
I realize there exist a need to clarify the text regarding ECN on the relation between IP packets and QUIC packets. As QUIC tracks QUIC packets, and not IP packets, this relation when coaleing occurs needs to be clarified. What I think is needed are:
The text was updated successfully, but these errors were encountered: