-
-
Notifications
You must be signed in to change notification settings - Fork 109
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
Turn on scatter-gather and checksum offload #367
Conversation
I'd rather remove those ethtool calls completely and rely on default (chosen by the kernel, or other network configuration tools in given distribution) - which are both "on" here. Also, when rolling back some non-standard settings, it's helpful to do research why they were put there in the first place, to verify if really "There is no reason to not have [...]".
Finally, please add reference to QubesOS/qubes-issues#3510 in the commit message |
Checksum offload was turned off in QubesOS/qubes-issues#700 due to a bug that is unlikely to still be relevant. Scatter-gather was turned off for unclear reasons that are also unlikely to be relevant nowadays. This should significantly improve networking performance. Fixes QubesOS/qubes-issues#3510.
ff46f1c
to
045e85e
Compare
This is even more important because one of my physical network does not like to turn on SG. So don't enforce it at all is required. |
Is it a problem if SG is enabled on the Xen interface? |
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.1&build=2022032618-4.1&flavor=pull-requests New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.1&build=2022031706-4.1&flavor=update
Failed tests26 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/36922#dependencies 3 fixed
Unstable tests
|
There is no reason to not have scatter-gather and checksum offload
turned on.