-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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: stable supports up to Linux 6.4 #245561
Conversation
ZFS stable supports up to Linux 6.4 in fact, this is not clearly communicated in the META file but it does not matter, I have been running 6.4 for a while with ZFS (and even bcachefs).
The former maintainers are not active anymore.
First of all: really appreciate the effort of trying to avoid stepping zfs compat backwards. Thanks! That said, I am slightly concerned about deviating from upstreams' stability guarantees. According to this comment from a ZFS developer:
I don't think we can match the same level of testing, or can we? Filesystem bugs are danger zone™ Personally, I'd prefer keeping EOL kernels around until zfs formally catches up, but that touches on another (controversial) debate.. |
I'd let you ask what is the "same level of testing" as the one ZFS is doing.
Anyway, it's out of question of keeping EOL kernels in nixpkgs IMHO,
especially for ZFS.
Now, what is only left is introducing an extra layer of distinction between
latestTestedByZFSTeam and latestPossible I guess?
Filesystem bugs are definitely danger zone, but I don't think that ZFS 2.2
is going to make this easier for anyone given the state of upstream.
Anyone who really needs a extremely high level of guarantee should probably
stay on NixOS LTS kernels (or better : NixOS stable releases). We cannot
guarantee that nixos-unstable will guarantee bump less rides for that.
Though I am open to alternative solutions, but if we cannot reach consensus
in the next days, I am inclined to regress latest to 6.1 and 6.3 drop
should proceed. EOL kernels should be dropped as soon as possible.
Le jeu. 27 juil. 2023 à 00:58, ius ***@***.***> a écrit :
… First of all: really appreciate the effort of trying to avoid stepping zfs
compat backwards. Thanks!
That said, I *am* slightly concerned about deviating from upstreams'
stability guarantees. According to this comment
<openzfs/zfs#14024 (comment)>
from a ZFS developer:
The version in the META file represents latest kernel which we have
tested. Fast moving downstream distributions, and end users building from
source, are always welcome to perform their own testing and bump that max
version as they see fit. What we want to avoid is misleading anyone about
what has, and has not, been tested by the developers.
I don't think we can match the same level of testing, or can we?
Filesystem bugs are danger zone™
Personally, I'd prefer keeping EOL kernels around until zfs formally
catches up, but that touches on another (controversial) debate..
—
Reply to this email directly, view it on GitHub
<#245561 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACMZRDLHMHL6KMQD5MXQUTXSGOIVANCNFSM6AAAAAA2Y3KIXE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
I think this is a fine approach. |
Note that ZFS' META was recently updated to indicate Linux 6.4 compat: Provided, this is their master, but I'd expect recent-ish releases (maybe 2.1.13 soon?) to include it. |
I don't think I'm comfortable marking zfs for support higher than what upstream does. While 2.1.12 has fixes for 6.4, for whatever reason upstream did not mark it as supported. Even the first two RC's of 2.2 are only 6.3, but I hope that to change for the next release. Given the unpredictability on when a kernel will be downgraded, what if instead we remove As @RaitoBezarius points out, users who are more concerned about stability likely should stay on LTS if at all possible. |
I'm honestly not convinced by this argument, as I said in my first post, does anyone what bumping the META file entails in terms of testing? If what happens is that they idly check it passes OpenZFS test suite and they bump it, does OpenZFS developers owe you something if eat your data? Anyway, I will not force my way towards this PR in any case, I am already using this patch on my production systems for > 3 weeks now. In all cases, I will now ask to proceed with the dropping PR. |
Also, not in favor of that because I use It's a tough call because I feel like we are going into circles regarding this problem. |
Yeah, I understand this sentiment completely. My personal opinion would be to leave the EOL kernel in tree, but mark it as insecure. I don't maintain the kernel package though and I know that suggestion is not favored by many people. Whatever the solution, I'd rather fail evaluating the config rather than silently downgrading. The status quo doesn't impact me though, given that I pin my kernel when not using LTS. |
I am extremely against this solution personally.
Silent downgrading is definitely not supposed to break anything by virtue of downgrading will always send you to the LTS kernel at least… Of course, reality is much more annoying than that, but if you are a user who needs a non-LTS kernel with ZFS, you are out of luck except if you buy into insecure kernels. |
Anyway, I also pinged ZFS upstream to ask them about 6.4 META for 2.1.(12|13), we will see how it goes. Let's keep this PR as a draft for now. |
Thanks. It seems like they got a handful of 6.4 compat fixes in for 2.1.12, though I don't know if those are exhaustive. |
2.2.0-rc3 does indeed officially list 6.4 support. |
@@ -13,10 +13,10 @@ callPackage ./generic.nix args { | |||
# check the release notes for compatible kernels | |||
kernelCompatible = | |||
if stdenv'.isx86_64 || removeLinuxDRM | |||
then kernel.kernelOlder "6.4" |
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.
Please also update unstable.
I bump zfs unstable here: #246163 |
fwiw https://nixos.org/manual/nixos/stable/index.html#sec-linux-zfs I know that this is a debate we had in the past already (see also https://discourse.nixos.org/t/aggressive-kernel-removal-on-eol-in-nixos/23097 for more context) and I can understand the pain considering that I'm also using ZFS pretty heavily, but as a package maintainer of the linux kernels I'm pretty strictly against keeping EOLed kernels in here. |
Sorry if this feels like a rehash of previous discussions. To be clear, I completely respect your stance. Unfortunately, I suspect this will keep coming up without some change to the status quo. This PR seems to be yet another example of the discord caused by the available |
Closing, in favor of #246300. |
Yes, this is nice in theory, untrue in reality. For example: the The preposition that using a recent, but EOL kernel is automatically insecure is a bit of an oversimplification. Whether or not there's any real risk in that depends on the user's threat model and the specific set of security patches that are lacking. It does seem clear that using an EOL kernel merits at least a warning, and probably explicit opt-in (via the existing As another option, distributions backporting security patches to EOL kernels isn't unheard of. But that would obviously present a maintenance burden to the kernel package maintainers, so understandable if that route isn't taken. |
I think this is still very true in reality for a lot of folks. As you are using a RDNA3 card, and you seem to be aware about the requirement for running latest stable kernels, I would argue this is probably on us on not saying enough to people to not use ZFS in those situations if they cannot accommodate with this silent downgrade, which would have probably avoided the issue. Introducing a warning is delicate because we have no way to talk to unstable users in any meaningful way yet except a Discourse post (in which I would have been curious if you had seen the notice). We could have a state mechanism to inform about silent downgrade at activation time, I would also recommend strongly to people to read Overall, there's a lot to do, but I don't feel like it's a good usage of maintainer energy as we don't offer any form of strong promise on ZFS and latest stable kernels, which, I believe, is something that almost no community Linux distribution offers neither. Anyone is free to drop a solution for their use case which is general enough for this class of problem, of course.
It has been discussed in the post mentioned above.
Yeah, I will say politely, this is a non-starter given our human resources in terms of maintenance, except if some people wants to start an Open Collective to fund a full-time kernel maintainer in nixpkgs for that. Anyway, I apologize for your bad experience, I guess I will try to open a PR on documenting better when you should not use ZFS on NixOS and include those cases for now, because I don't see the silent downgrade situation changing soon except if folks want to start paying Nixpkgs ZFS maintainers for doing the dance between: nixpkgs, kernel and upstream OpenZFS, which is highly painful honestly. |
FWIW we should probably stop recommending |
The problem of this is that we will still get complaints about folks not understanding why linux_6_3 disappeared, IMHO. Anyway, in those situations, I feel like no matter what we do, there will be problems and issues, so I feel inclined to minimize the energy spent until there's someone who care enough sending a PR to change things. |
Description of changes
ZFS stable supports up to 6.4 in fact, this is not clearly communicated in the META file
but it does not matter, I have been running 6.4 for a while with ZFS (and even bcachefs).
Unblocks #244883.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/
)