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

zfs: 2.1.12 -> 2.1.13 #257755

Merged
merged 1 commit into from
Sep 29, 2023
Merged

zfs: 2.1.12 -> 2.1.13 #257755

merged 1 commit into from
Sep 29, 2023

Conversation

adamcstephens
Copy link
Contributor

@adamcstephens adamcstephens commented Sep 28, 2023

Description of changes

https://github.com/openzfs/zfs/releases/tag/zfs-2.1.13

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.11 Release Notes (or backporting 23.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

else kernel.kernelOlder "6.2";
latestCompatibleLinuxPackages = linuxKernel.packages.linux_6_1;
latestCompatibleLinuxPackages = linuxKernel.packages.linux_6_5;
Copy link
Member

@Mic92 Mic92 Sep 28, 2023

Choose a reason for hiding this comment

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

Suggested change
latestCompatibleLinuxPackages = linuxKernel.packages.linux_6_5;
latestCompatibleLinuxPackages = if stdenv'.isx86_64 || removeLinuxDRM
then linuxKernel.packages.linux_6_5
else linuxKernel.packages.linux_6_1;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I understand this now that I looked up the module option for removeLinuxDRM. Interestingly #236459 didn't have this logic. Should we maybe just leave it in here moving forward even if 6.5 gets dropped before the next zfs version?

Copy link
Member

Choose a reason for hiding this comment

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

Yes.

@toastal
Copy link
Contributor

toastal commented Sep 28, 2023

2.1.13 supports Linux 6.5 while 2.2.0-rc only supports Linux 6.4.

Copy link
Member

@RaitoBezarius RaitoBezarius left a comment

Choose a reason for hiding this comment

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

LGTM.

@delroth delroth added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Sep 28, 2023
@toastal
Copy link
Contributor

toastal commented Sep 28, 2023

Aside: how does one overlay the kernel modules? I’ve used commit as an overlay for zfs in Nixpkgs, but ran into issues compiling the kernel which is pulling in a version from the original Nixpkgs. I tried pkgs.linuxPackages_x.extend (final: prev: { zfs = pkgs.zfs; }) in configuration.nix. This built, but that followed up with extraModulePackages missing zfs. Added, zfs but then got this error:

error: builder for '/nix/store/bhifa6fz4m2j7sgcifhzjb4dw2d1bsf5-linux-6.5.5-modules-shrunk.drv' failed with exit code 1;
       last 10 log lines:
       >   copying dependency: /nix/store/q244lms4jggjab57mlvsgncdqmg5lgrh-linux-6.5.5-modules/lib/modules/6.5.5/kernel/drivers/gpu/drm/amd/amdgpu/amdgpu.ko.xz
       > root module: kvm-amd
       >   copying dependency: /nix/store/q244lms4jggjab57mlvsgncdqmg5lgrh-linux-6.5.5-modules/lib/modules/6.5.5/kernel/virt/lib/irqbypass.ko.xz
       >   copying dependency: /nix/store/q244lms4jggjab57mlvsgncdqmg5lgrh-linux-6.5.5-modules/lib/modules/6.5.5/kernel/arch/x86/kvm/kvm.ko.xz
       >   dependency already copied: /nix/store/q244lms4jggjab57mlvsgncdqmg5lgrh-linux-6.5.5-modules/lib/modules/6.5.5/kernel/drivers/base/firmware_loader/firmware_class.ko.xz
       >   dependency already copied: /nix/store/q244lms4jggjab57mlvsgncdqmg5lgrh-linux-6.5.5-modules/lib/modules/6.5.5/kernel/drivers/char/hw_random/rng-core.ko.xz
       >   copying dependency: /nix/store/q244lms4jggjab57mlvsgncdqmg5lgrh-linux-6.5.5-modules/lib/modules/6.5.5/kernel/drivers/crypto/ccp/ccp.ko.xz
       >   copying dependency: /nix/store/q244lms4jggjab57mlvsgncdqmg5lgrh-linux-6.5.5-modules/lib/modules/6.5.5/kernel/arch/x86/kvm/kvm-amd.ko.xz
       > root module: zfs
       > modprobe: FATAL: Module zfs not found in directory /nix/store/q244lms4jggjab57mlvsgncdqmg5lgrh-linux-6.5.5-modules/lib/modules/6.5.5
       For full logs, run 'nix log /nix/store/bhifa6fz4m2j7sgcifhzjb4dw2d1bsf5-linux-6.5.5-modules-shrunk.drv'.

@victorklos
Copy link

Thanks! I can confirm that this works, rocking it now. A new AMD GPU on a system with ZFS is now possible 😄

Had to search a bit for what was needed to run this. For posterity:

git clone -b the-pull-request-branch --depth 1 url-of-nixpkgs
sudo nixos-rebuild -I nixpkgs=/path/to/cloned/nixpkgs switch

@RaitoBezarius
Copy link
Member

RaitoBezarius commented Sep 29, 2023

Thanks! I can confirm that this works, rocking it now. A new AMD GPU on a system with ZFS is now possible 😄

Had to search a bit for what was needed to run this. For posterity:

git clone -b the-pull-request-branch --depth 1 url-of-nixpkgs
sudo nixos-rebuild -I nixpkgs=/path/to/cloned/nixpkgs switch

For posterity:

sudo nixos-rebuild -I nixpkgs="https://github.com/adamcstephens/nixpkgs/archive/94146957746e894891d3932423449c0d9d7577e0.tar.gz" switch

should probably also work ;).

Documentation is in "what is NIX_PATH" ?

@RaitoBezarius RaitoBezarius merged commit f427e60 into NixOS:master Sep 29, 2023
@eadwu
Copy link
Member

eadwu commented Sep 30, 2023

Is boot broken with this? Pool doesn't seem to be importing on my end.

@shymega
Copy link

shymega commented Oct 1, 2023

I can confirm that using the ZFS kernel module from master works on linuxPackages_xanmod_stable. However, I'm finding you have to compile the whole Xanmod kernel, manually.

I may switch to stock Linux whilst I wait for this bump to either hit unstable or 23.05. It wasn't working in 23.05 or unstable (Xanmod and ZFS) last time I tried, which was this evening.

Have to admit, I don't really like using Git/unstable for kernels, but with the lack of a similar construct like this for Xanmod, I think I'll have to stick with it. Maybe lock Nixpkgs in my Flake.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 1-10 10.rebuild-linux: 101-500 12.approvals: 1 This PR was reviewed and approved by one reputable person
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants