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 23, 2024
1 parent c332c9d commit 6e6a235
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
6 changes: 5 additions & 1 deletion quinn-interop/src/client.rs
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,11 @@ 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)
.initial_mtu(1500 - 40 - 8);
let tls_config = quinn::crypto::rustls::QuicClientConfig::try_from(tls_config)?;
let mut client_config = quinn::ClientConfig::new(Arc::new(tls_config));
client_config
Expand Down
6 changes: 5 additions & 1 deletion quinn-interop/src/server.rs
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,11 @@ 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)
.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 6e6a235

Please sign in to comment.