-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
zfs: 2.1.12 -> 2.1.13 #257755
Conversation
dc9cf23
to
b065b79
Compare
else kernel.kernelOlder "6.2"; | ||
latestCompatibleLinuxPackages = linuxKernel.packages.linux_6_1; | ||
latestCompatibleLinuxPackages = linuxKernel.packages.linux_6_5; |
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.
latestCompatibleLinuxPackages = linuxKernel.packages.linux_6_5; | |
latestCompatibleLinuxPackages = if stdenv'.isx86_64 || removeLinuxDRM | |
then linuxKernel.packages.linux_6_5 | |
else linuxKernel.packages.linux_6_1; |
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.
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?
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.
Yes.
2.1.13 supports Linux 6.5 while 2.2.0-rc only supports Linux 6.4. |
b065b79
to
9414695
Compare
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.
LGTM.
Aside: how does one overlay the kernel modules? I’ve used commit as an overlay for
|
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:
|
For posterity:
should probably also work ;). Documentation is in "what is NIX_PATH" ? |
Is boot broken with this? Pool doesn't seem to be importing on my end. |
I can confirm that using the ZFS kernel module from 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. |
Description of changes
https://github.com/openzfs/zfs/releases/tag/zfs-2.1.13
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)