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
qemu: 2.11.1 -> 2.12.0 #39513
qemu: 2.11.1 -> 2.12.0 #39513
Conversation
@dtzWill I didn't check the musl patches, feel free to look at them now or fix whatever breaks later 😄 |
Failure on aarch64-linux (full log) Attempted: qemu Partial log (click to expand)
|
Thanks for the heads-up, I'll take a look and let you know / file PR to fix it. I gotta call it for the night, but will take a look in the morning. If there's anyone waiting to see this merged please go ahead, don't block it on my account :). |
LGTM re:musl --there's an issue but it's not QEMU's fault and I'll fix it separately. With that addressed it builds and works in tests and such. |
FWIW dtzWill@91b5f5a adds fix w/musl, I was planning to submit it separately but can probably be added along with if you don't have any objections. Not sure who else should look at this... @shlevy ? :) |
Does this version include the riscv support? |
I thought so, and looking at the changelog it looks like "yes": https://wiki.qemu.org/ChangeLog/2.12#RISC-V |
I'll include your fix here. |
Yes, and builds by default: |
Most everyone using musl patches the linux headers instead, but various software uses a local workaround like the one added in this commit (psutils, for example). It's not obvious to me which project has the "bug", and I'm reluctant to even propose modifying our headers without clear answer on the issue. Also, modifying those headers triggers rebuild-all-the-things. Hopefully upstream projects sort this out, in the meantime adding this define is a bit of a kludge but does the job. ------- For the curious, the patch usually is something like this: https://patchwork.kernel.org/patch/3833241/ Here's an updated version that also ensures kernel users get the sysinfo struct as expected too: https://raw.githubusercontent.com/openwrt/openwrt/e3c43ade0bae9491aeea50fa361e846bb5002dc0/target/linux/generic/pending-4.14/270-uapi-kernel.h-glibc-specific-inclusion-of-sysinfo.h.patch (cherry picked from commit dtzWill/nixpkgs@91b5f5a)
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.
Should we remove the separate |
Please do! |
upstream qemu 2.12 includes riscv support
Failure on aarch64-linux (full log) Attempted: qemu Partial log (click to expand)
|
that ofborg timeout is normal for a full qemu build. |
Failure on aarch64-linux (full log) Attempted: qemu Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: qemu Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: qemu Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: qemu Partial log (click to expand)
|
Motivation for this change
Bump and remove obsolete patches. Closes #39423 .
Edit: also include a fix by @dtzWill for musl and remove the now obsolete
qemu-riscv
package.Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)