Lower default MTU to 900 #4985
Merged
Lower default MTU to 900 #4985
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some players continue to have "stuck at connected" issues connecting to most servers, but apparently this issue has become more prominent in the last two weeks or so? It is almost certainly an MTU problem because there are at least two servers on the hub that run a lower default MTU, and these players had no problem connecting to them. For one of these reporters, I actually increased the MTU to 1000 and they could no longer connect, and could connect again once it was lowered to 900. It's not clear what recent changes, either to the codebase or to the public Internet that have been exercising this MTU issue more. For those experiencing MTU issues, it seems that connecting to a less full server results in higher probability of success. Nevertheless, bringing down the default MTU and then possibly enabling MTU expansion in the future would make this game playable for a small but not insignificant bit of players.
|
I've been doing some work on Lidgren to make this more practical, as I don't want to enable this without Auto Expand MTU. But Auto Expand MTU is currently broken on the Lidgren version we're using (already fixed on fork master). |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some players continue to have "stuck at connected" issues connecting to most servers, but apparently this issue has become more prominent in the last two weeks or so?
It is almost certainly an MTU problem because there are at least two servers on the hub that run a lower default MTU, and these players had no problem connecting to them. For one of these reporters, I actually increased the MTU to 1000 and they could no longer connect, and could connect again once it was lowered to 900.
It's not clear what recent changes, either to the codebase or to the public Internet that have been exercising this MTU issue more. For those experiencing MTU issues, it seems that connecting to a less full server results in higher probability of success.
Nevertheless, bringing down the default MTU and then possibly enabling MTU expansion in the future would make this game playable for a small but not insignificant bit of players.
Implicitly approved by @PJB3005