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
linux: Temporary fix for issue #42755 #43115
Conversation
# https://github.com/NixOS/nixpkgs/issues/42755 | ||
xen-netfront_fix_mismatched_rtnl_unlock = { | ||
name = "xen-netfront_fix_mismatched_rtnl_unlock"; | ||
patch = ./xen-netfront_fix_mismatched_rtnl_unlock.patch; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of checking in the patches to nixpkgs you should use fetchpatch
:
patch = fetchpatch {
name = "xen-netfront_fix_mismatched_rtnl_unlock.patch";
url = "https://github.com/torvalds/linux/commit/cb257783c2927b73614b20f915a91ff78aa6f3e8.patch";
sha256 = "1288a5irvq2rhfl0aclb0lvhxwmbqp5blcdpx7d7lwz356cwx8xm";
};
Also the commit message should be more descriptive in itself, not just a reference to an issue. |
Fix a serious issue with the xen-netfront driver introduced in upstream commit f599c64fdf7d ("xen-netfront: Fix race between device setup and open") where the MTU of the device cannot be set properly. This should be removed once it's included in upstream.
@GrahamcOfBorg test kernel-latest kernel-lts |
I'm building these tests manually too in case of @GrahamcOfBorg times out. |
Failure on x86_64-linux (full log) Attempted: tests.kernel-latest, tests.kernel-lts Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: tests.kernel-latest, tests.kernel-lts Partial log (click to expand)
|
Tested on latest and lts kernels. Passes on both. Thanks for your contribution! |
cc @NeQuissimus |
Backporting to 18.03 is underway. Still building. |
Motivation for this change
These patches fix a serious issue with the xen-netfront driver (used on AWS ec2 t2 instances.) They will be included in an upstream release, so this is just a temporary measure. Issue #42755 goes more into depth about this.
This should preferably be merged to stable too, where it should be applied to all versions of the kernel except 4.15, which doesn't contain the patch that introduced this bug.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)