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

Update QEMU to 7.0.0 #4076

Closed
wants to merge 1 commit into from

Conversation

alexcrichton
Copy link
Member

Seems good for us to stay updated as we're relying on it for tests, so
this updates the version of QEMU used on CI to run cross-compiled tests
for s390x and AArch64

Seems good for us to stay updated as we're relying on it for tests, so
this updates the version of QEMU used on CI to run cross-compiled tests
for s390x and AArch64
@cfallin
Copy link
Member

cfallin commented Apr 27, 2022

cc @uweigand for s390x issues (qemu is segfaulting) -- are there any known problems in qemu 7.0.0 wrt s390x support?

@alexcrichton
Copy link
Member Author

Alas perhaps an update another day!

@alexcrichton alexcrichton deleted the update-qemu branch April 28, 2022 14:41
@uweigand
Copy link
Member

cc @uweigand for s390x issues (qemu is segfaulting) -- are there any known problems in qemu 7.0.0 wrt s390x support?

I don't know of any known problems, but some regression may of course have crept in. I'll try to reproduce a run with qemu 7.0.0 locally.

@uweigand
Copy link
Member

It seems this patch: https://lists.gnu.org/archive/html/qemu-devel/2021-09/msg07466.html is causing the problem. This changes the layout of a data structure on the signal stack that user-space code relies upon. Reverting the data structure change using the attached patch makes the problem go away for me.
qemu-fix-s390-sigframe.txt

@alexcrichton
Copy link
Member Author

Oh nice! I don't know of any pressing need to land this so I think it's ok to wait for this to be resolved upstream in a release before we update.

@uweigand
Copy link
Member

Oh nice! I don't know of any pressing need to land this so I think it's ok to wait for this to be resolved upstream in a release before we update.

Just FYI the patch to fix this regression has now been merged; QEMU 7.1.0 should have the fix.

@iii-i
Copy link
Contributor

iii-i commented Jun 23, 2022

I brought ci/qemu-madvise.patch upstream: https://lists.gnu.org/archive/html/qemu-devel/2022-06/msg03752.html.
You should be able to drop it once you switch to 7.1.0.

@alexcrichton
Copy link
Member Author

Oh awesome, thanks @iii-i!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants