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

OvmfPkg/PlatformCI: Use Qemu 8 in CI and enable SMP in tests #4711

Closed
wants to merge 4 commits into from

Conversation

osteffenrh
Copy link
Contributor

@osteffenrh osteffenrh commented Aug 2, 2023

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4324

This series switches the OVMF CI jobs on Windows to a recent Qemu
version and re-enables SMP in the Qemu "Run to shell" CI jobs.
The Qemu tests run a little bit slower in SMP mode and sometimes
hit the time limit of 2 minutes, thus this limit is increased to 4
minutes.

Together with 81cb037 ("CI: Use latest image for Linux jobs (Qemu
8, gcc 12)" this series is sufficient to close Bug 4324.

osteffenrh and others added 4 commits August 3, 2023 12:38
Bump the version of the Qemu chocolatey package up to 2023.7.25.
The Linux CI is already using Qemu 8.

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4324

Signed-off-by: Oliver Steffen <osteffen@redhat.com>
This reverts commit 3beb8c9.

Both Windows and Linux CI jobs are now using Qemu 8, this workaround is
no longer needed.

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4324

Signed-off-by: Oliver Steffen <osteffen@redhat.com>
Increase the chance that CI finds bugs in MP changes.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
(cherry picked from commit f92a9dc)

Reapplying, since the CI (Windows and Linux) use Qemu 8 now, which works
with SMP again.

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4324

Signed-off-by: Oliver Steffen <osteffen@redhat.com>
The current job time limit of 2 minutes is just a little bit too short
for running Qemu in SMP mode.

Signed-off-by: Oliver Steffen <osteffen@redhat.com>
@osteffenrh osteffenrh changed the title [TEST] Bug 4324 / Qemu 8 + SMP Use Qemu 8 in CI and enable SMP in tests Aug 3, 2023
@osteffenrh osteffenrh marked this pull request as ready for review August 3, 2023 10:51
@osteffenrh osteffenrh changed the title Use Qemu 8 in CI and enable SMP in tests OvmfPkg/PlatformCI: Use Qemu 8 in CI and enable SMP in tests Aug 3, 2023
@osteffenrh osteffenrh closed this Sep 7, 2023
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

2 participants