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

openafs: fix OpenAFS 1.6 on Linux 4.20 #56862

Merged
merged 4 commits into from Mar 6, 2019

Conversation

artemist
Copy link
Member

@artemist artemist commented Mar 5, 2019

Motivation for this change

I have been requested to add linuxPackages_latest support to OpenAFS 1.6. I found working patches, although not enough for 5.0

Things done

This patch cherry-picks commits from the OpenAFS git repository using
git show to add support for Linux 4.20 for OpenAFS 1.6.

There do not seem to be patches in the Git repository for full Linux 5.0
support, although I have added the ones which exist so far.

This patch has been tested on Linux 4.4, 4.14, 4.20, and is confirmed to
fail on Linux 5.0. It does not change the drv of the openafs package,
only linuxPackages_*.openafs.

  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

This patch cherry-picks commits from the OpenAFS git repository using
`git show` to add support for Linux 4.20 for OpenAFS 1.6.

There do not seem to be patches in the Git repository for full Linux 5.0
support, although I have added the ones which exist so far.

This patch has been tested on Linux 4.4, 4.14, 4.20, and is confirmed to
fail on Linux 5.0. It does not change the drv of the openafs package,
only linuxPackages_*.openafs.
@veprbl
Copy link
Member

veprbl commented Mar 5, 2019

cc @spacefrogg

@artemist
Copy link
Member Author

artemist commented Mar 5, 2019

@GrahamcOfBorg build linuxPackages_4_20.openafs

Edit, oops, I can't do this. Can someone else run this for me?

@veprbl
Copy link
Member

veprbl commented Mar 5, 2019

@GrahamcOfBorg build linuxPackages_4_20.openafs

@veprbl
Copy link
Member

veprbl commented Mar 5, 2019

@GrahamcOfBorg build linuxPackages_4_20.openafs

Copy link
Member

@veprbl veprbl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Builds against most kernels (except hardened). Works.

@spacefrogg
Copy link
Contributor

@artemist Thanks for the effort! Looks good from my side.

@artemist
Copy link
Member Author

artemist commented Mar 6, 2019

@veprbl That's an annoying issue with struct layout randomization. i'm writing a patch I'll submit upstream after some testing, but it's out of scope for this PR (and quite possibly nixpkgs)

@veprbl veprbl merged commit 8a8628f into NixOS:master Mar 6, 2019
@artemist artemist deleted the openafs-1.6-linux_4_20 branch March 13, 2019 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants