Skip to content

Use QEMU 9 #23

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

Merged
merged 6 commits into from
Apr 9, 2025
Merged

Use QEMU 9 #23

merged 6 commits into from
Apr 9, 2025

Conversation

jonathanpallant
Copy link
Contributor

@jonathanpallant jonathanpallant commented Apr 4, 2025

Runs QEMU tests on Linux and Windows. We do this because QEMU 9 is required to run the MPS3-AN536 tests (a virtual Cortex-R52) and QEMU 9 isn't available in Ubuntu 24.04.

It is however available on Windows using chocolatey.

Edit:

People didn't like that I was using Windows, so now I pull QEMU 9 for Ubuntu 24.04 from https://github.com/jonathanpallant/qemu9-for-ubuntu-2404.

There was a small variation in the counter output from run to run, so I removed the output.
QEMU 9 is available for Windows, but not for Ubuntu 24.04 (unless you compile it from source, and I don't want to do that).
Pollutes the QEMU output in CI because there's no audio device available.
@adamgreig
Copy link
Member

LGTM. I think if it turns out you can easily get qemu9 on Ubuntu then so much the better, but if not this doesn't seem like a big problem. Right now won't Windows run all the tests and Ubuntu run all but the qemu9-only ones -- is there any point having both Ubuntu and Windows runners? Could it just be purely Windows for now, and maybe revert to purely Ubuntu once we can get qemu9 there?

@jonathanpallant
Copy link
Contributor Author

I left the Ubuntu one because it installs and runs quicker but, yes, it's only running a subset of the tests. Happy to take it out if you prefer.

@jonathanpallant jonathanpallant changed the title Use windows for qemu Use QEMU 9 Apr 9, 2025
Copy link

@9names 9names left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jonathanpallant jonathanpallant merged commit d20623c into main Apr 9, 2025
57 checks passed
@jonathanpallant jonathanpallant deleted the use-windows-for-qemu branch April 9, 2025 12:28
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.

4 participants