Conversation
This reverts commit 66f78a9.
|
I think at most we should revert the specific patch that’s at fault. It’s far too close to release to do an entire |
|
Yea, we absolutely are not going to rollback the entire systemd version when it's only been observed as an issue on one platform. I'd be fine with shipping a revert of the offending patch (we already do that for one other issue), but rolling back systemd entirely is way too extreme of a solution. |
|
Besides my Macbook (which cannot run stock NixOS) I only have a common x86 machine available to test, where the release works fine. Besides the page size problem, from the discussion in the issue the release should be broken on any platform without physical memory available below the 4G physical address--this is uncommon, but is allowed by the x86-64 spec. @mzyngier mentioned that his Opteron A1100 from 2014 would be broken, but I do not have one on hand to verify (I replied asking him to confirm whether he can reproduce the issue). The specific problematic commit would be this one; reverting it has been verified by @andre4ik3 to fix the problem. |
Fixes boot for me. Closes: NixOS#355284
|
Ultimately whatever we do is going to have to land within the first few weeks post‐release, unless we decide to significantly delay the release for this, which seems unlikely when the main issue is on unsupported platforms and supported hardware that may be susceptible to the issue is very rare. |
Agreed, if this was earlier then I wouldn't reject this but at this point, we probably should only revert the specific commit in systemd. There's already a revert patch from the issue thread (systemd/systemd#35026 (comment)). We probably could get that into staging-next and revert it after 24.11 branch off. Edit: didn't see Alyssa's comment appear until after I made this comment. |
|
We cannot put it into |
|
Oh ok, I didn't know that. Also didn't see that comment you posted after Alyssa made the PR until I posted my comment. |
|
Closing in favor of #355290 |
Fixes boot for me. Closes: NixOS#355284 (cherry picked from commit 63ce466) (cherry picked from commit 91ea63f)
This reverts commit 66f78a9.
systemd 256.7 introduces a breaking change to
systemd-bootcausing the system to fail to boot on some architectures (including generation rollback) and necessitating manual recovery. From the discussion, it looks like there are some pretty serious fundamentally mistaken assumptions in their code which may take some time to rectify. In order to avoid bricking our users by default, we should revert to the last working version until this issue is worked out upstream.Pinging maintainers:
@flokli
@kloenk
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.