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

improvement: make network device backlog a little bigger #265

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

interduo
Copy link
Collaborator

No description provided.

@dtaht
Copy link
Collaborator

dtaht commented Mar 18, 2023

In general I don't like covering up real latency problems with more buffering. Is there a case you can demonstrate where this actually helps?

@interduo
Copy link
Collaborator Author

That is true what You said - too much buffering is bad, but too less generates packet loss.

Do some tests in some heavy production:
for i in {1..30}; do date +%s; cat /proc/net/softnet_stat; sleep 1; done

This counts how much packets go to hell. Apply this patch then and check again. Ubuntu defaults kernel value is pretty low, this value is used by kernel bridge but I am not pretty sure if its used when using XDP.

@dtaht
Copy link
Collaborator

dtaht commented Mar 20, 2023

thank you for supplying a test like this! We will add that to the upcoming test series.

@interduo
Copy link
Collaborator Author

interduo commented Dec 14, 2023

This helps when not using XDP only. If we use XDP this setting does not matter. Please merge or close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants