Re: [quicwg/base-drafts] Rewrite Packet Size section (#2036)

[Martin Thomson <[e-mail]>]

martinthomson commented on this pull request.



> +* An endpoint could store additional information from the IP or UDP headers to
+  use for validation (for example, the IP ID or UDP checksum).
+
+The endpoint SHOULD ignore all ICMP messages that are not validated or do not
+carry sufficient quoted packet payload to perform validation.  Any reduction in
+the QUIC MPS MAY be provisional until QUIC's loss detection algorithm
+determines that the quoted packet has actually been lost.
+
+## Considerations for Datagram Packetization Layer PMTU Discovery
+
+Section 6.4 of {{!DPLPMTUD}} provides considerations for
+implementing Datagram Packetization Layer PMTUD (DPLPMTUD) with QUIC.
+
+When implementing the algorithm in Section 5.3 of
+{{!DPLPMTUD}}, the initial value of BASE_PMTU SHOULD be
+consistent with the minimum QUIC packet size.

The reason I point this out is that the minimum isn't a minimum, it's just the minimum for an Initial packet.  The minimum packet size is something like 21 bytes absent context.  I think that the minimum in this context is probably 1200, which is the lower limit on what servers accept, but then clients don't enforce any such limit.  And of course, probing at 1200 on IPv6 is pointless.

In other words, you do need to be a little more precise.

-- 
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/2036#discussion_r235824683