Skip to content
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

router: enforce configured MTU #3738

Open
matzf opened this issue Apr 29, 2020 · 3 comments
Open

router: enforce configured MTU #3738

matzf opened this issue Apr 29, 2020 · 3 comments
Labels
c/router SCION Router i/learning op i/proposal accepted i/proposal A new idea requiring additional input and discussion

Comments

@matzf
Copy link
Contributor

matzf commented Apr 29, 2020

The border router should respect and enforce the configured inter and intra-AS MTU.

Assuming the MTU is configured correctly, IP fragmentation should not happen. So perhaps, the router should also set IP_DONTFRAG to ensure that packets are not fragmented.

Note: this used to be blocked by exceedingly large control plane messages, but this is no longer an issue as this now all uses QUIC.

@matzf
Copy link
Contributor Author

matzf commented Apr 29, 2020

Compare also

// FIXME(kormat): this should be reduced as soon as we respect the actual link MTU.
const pktBufSize = 9 * 1024

@matzf
Copy link
Contributor Author

matzf commented May 4, 2020

Related: #1328

@matzf matzf changed the title BR: enforce configured MTU router: enforce configured MTU Oct 5, 2022
@matzf matzf added i/proposal A new idea requiring additional input and discussion c/router SCION Router labels Oct 5, 2022
@matzf
Copy link
Contributor Author

matzf commented Mar 14, 2023

Discussion in the proposal review meeting; consensus to accept this . However, we need to be careful as doing so may unearth issues with MTUs, e.g. on long SCION paths, the effective MTU may fall below the minimum MTU required for QUIC.

This proposal will be accepted unless objections are raised in the 10 day final comment period starting now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/router SCION Router i/learning op i/proposal accepted i/proposal A new idea requiring additional input and discussion
Projects
None yet
Development

No branches or pull requests

2 participants