Skip to content

Commit

Permalink
Disable PLPMTUD
Browse files Browse the repository at this point in the history
Saves a few packets, and avoids a spurious failure in the zerortt test
case due to
quic-interop/quic-interop-runner#398.
  • Loading branch information
Ralith committed Jul 24, 2024
1 parent 4051053 commit d1c836d
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 2 deletions.
7 changes: 6 additions & 1 deletion quinn-interop/src/client.rs
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,12 @@ fn config(alpn: &str, suites: CipherSuite) -> Result<ClientConfig, Box<dyn std::
.max_idle_timeout(Some(Duration::from_millis(9000).try_into()?))
.initial_rtt(Duration::from_millis(100))
// https://github.com/quic-interop/quic-interop-runner/issues/397
.enable_segmentation_offload(false);
.enable_segmentation_offload(false)
// Don't bother probing a known network environment, and avoid
// https://github.com/quic-interop/quic-interop-runner/issues/398
.mtu_discovery_config(None)
// Known interface MTU, minus conservative IPv6 and UDP header sizes
.initial_mtu(1500 - 40 - 8);
let tls_config = quinn::crypto::rustls::QuicClientConfig::with_initial(
Arc::new(tls_config),
TLS13_AES_128_GCM_SHA256
Expand Down
7 changes: 6 additions & 1 deletion quinn-interop/src/server.rs
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,12 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
let crypto = quinn::crypto::rustls::QuicServerConfig::try_from(crypto)?;
let mut transport_config = quinn::TransportConfig::default();
// https://github.com/quic-interop/quic-interop-runner/issues/397
transport_config.enable_segmentation_offload(false);
transport_config
.enable_segmentation_offload(false)
// Don't bother probing a known network environment
.mtu_discovery_config(None)
// Known interface MTU, minus conservative IPv6 and UDP header sizes
.initial_mtu(1500 - 40 - 8);
let mut server_config = h3_quinn::quinn::ServerConfig::with_crypto(Arc::new(crypto));
server_config.transport_config(Arc::new(transport_config));

Expand Down

0 comments on commit d1c836d

Please sign in to comment.