Message ID | 20230426180013.14814-1-farosas@suse.de |
---|---|
Headers | show |
Series | target/arm: Allow CONFIG_TCG=n builds | expand |
On Wed, 26 Apr 2023 at 19:00, Fabiano Rosas <farosas@suse.de> wrote: > > Hi, > > Some minor changes: > > - new patch to move a test under CONFIG_TCG (broken on master); > - new patch to document the unsupported CPU test (Philippe); > - changed the test skip message when no KVM or TCG are present (Igor). Applied to target-arm.next; thanks for your persistence in working through the many versions of this patchset. -- PMM
On Tue, 2 May 2023 at 10:55, Peter Maydell <peter.maydell@linaro.org> wrote: > > On Wed, 26 Apr 2023 at 19:00, Fabiano Rosas <farosas@suse.de> wrote: > > > > Hi, > > > > Some minor changes: > > > > - new patch to move a test under CONFIG_TCG (broken on master); > > - new patch to document the unsupported CPU test (Philippe); > > - changed the test skip message when no KVM or TCG are present (Igor). > > Applied to target-arm.next; thanks for your persistence in > working through the many versions of this patchset. Update: I had to drop "gitlab-ci: Check building KVM-only aarch64 target" because it enables a CI job that fails on our aarch64 runner (because it wants to run tests using KVM but that machine isn't configured to allow the runner to use KVM). thanks -- PMM
On Tue, 2 May 2023 at 15:51, Peter Maydell <peter.maydell@linaro.org> wrote: > > On Tue, 2 May 2023 at 10:55, Peter Maydell <peter.maydell@linaro.org> wrote: > > > > On Wed, 26 Apr 2023 at 19:00, Fabiano Rosas <farosas@suse.de> wrote: > > > > > > Hi, > > > > > > Some minor changes: > > > > > > - new patch to move a test under CONFIG_TCG (broken on master); > > > - new patch to document the unsupported CPU test (Philippe); > > > - changed the test skip message when no KVM or TCG are present (Igor). > > > > Applied to target-arm.next; thanks for your persistence in > > working through the many versions of this patchset. > > Update: I had to drop "gitlab-ci: Check building KVM-only aarch64 target" > because it enables a CI job that fails on our aarch64 runner > (because it wants to run tests using KVM but that machine > isn't configured to allow the runner to use KVM). We fixed the runner config, but the CI still fails on that notcg job because it is trying to run tests that explicitly use '-accel tcg': https://gitlab.com/qemu-project/qemu/-/jobs/4212850809#L3595 Something is weird here, because we built without TCG support on an aarch64 host but we still got qemu-system-i386 and qemu-system-x86_64 binaries, which then don't work and cause the tests to fail... -- PMM
Peter Maydell <peter.maydell@linaro.org> writes: > On Tue, 2 May 2023 at 15:51, Peter Maydell <peter.maydell@linaro.org> wrote: >> >> On Tue, 2 May 2023 at 10:55, Peter Maydell <peter.maydell@linaro.org> wrote: >> > >> > On Wed, 26 Apr 2023 at 19:00, Fabiano Rosas <farosas@suse.de> wrote: >> > > >> > > Hi, >> > > >> > > Some minor changes: >> > > >> > > - new patch to move a test under CONFIG_TCG (broken on master); >> > > - new patch to document the unsupported CPU test (Philippe); >> > > - changed the test skip message when no KVM or TCG are present (Igor). >> > >> > Applied to target-arm.next; thanks for your persistence in >> > working through the many versions of this patchset. >> >> Update: I had to drop "gitlab-ci: Check building KVM-only aarch64 target" >> because it enables a CI job that fails on our aarch64 runner >> (because it wants to run tests using KVM but that machine >> isn't configured to allow the runner to use KVM). > > We fixed the runner config, but the CI still fails on that notcg > job because it is trying to run tests that explicitly use > '-accel tcg': > https://gitlab.com/qemu-project/qemu/-/jobs/4212850809#L3595 > > Something is weird here, because we built without TCG support > on an aarch64 host but we still got qemu-system-i386 > and qemu-system-x86_64 binaries, which then don't work > and cause the tests to fail... > Hmm, that's potentially due to Xen. Looks like we need more (!tcg && !kvm) checks. Let me try to reproduce it.
Fabiano Rosas <farosas@suse.de> writes: > Peter Maydell <peter.maydell@linaro.org> writes: > >> On Tue, 2 May 2023 at 15:51, Peter Maydell <peter.maydell@linaro.org> wrote: >>> >>> On Tue, 2 May 2023 at 10:55, Peter Maydell <peter.maydell@linaro.org> wrote: >>> > >>> > On Wed, 26 Apr 2023 at 19:00, Fabiano Rosas <farosas@suse.de> wrote: >>> > > >>> > > Hi, >>> > > >>> > > Some minor changes: >>> > > >>> > > - new patch to move a test under CONFIG_TCG (broken on master); >>> > > - new patch to document the unsupported CPU test (Philippe); >>> > > - changed the test skip message when no KVM or TCG are present (Igor). >>> > >>> > Applied to target-arm.next; thanks for your persistence in >>> > working through the many versions of this patchset. >>> >>> Update: I had to drop "gitlab-ci: Check building KVM-only aarch64 target" >>> because it enables a CI job that fails on our aarch64 runner >>> (because it wants to run tests using KVM but that machine >>> isn't configured to allow the runner to use KVM). >> >> We fixed the runner config, but the CI still fails on that notcg >> job because it is trying to run tests that explicitly use >> '-accel tcg': >> https://gitlab.com/qemu-project/qemu/-/jobs/4212850809#L3595 >> >> Something is weird here, because we built without TCG support >> on an aarch64 host but we still got qemu-system-i386 >> and qemu-system-x86_64 binaries, which then don't work >> and cause the tests to fail... >> > > Hmm, that's potentially due to Xen. Looks like we need more (!tcg && > !kvm) checks. Let me try to reproduce it. Ah right, the test is skipped on my aarch64 host because I don't have genisomage available. So what we need is this: -- >8 -- From: Fabiano Rosas <farosas@suse.de> Date: Tue, 2 May 2023 13:42:14 -0300 Subject: [PATCH] fixup! tests/qtest: Fix tests when no KVM or TCG are present --- tests/qtest/cdrom-test.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/tests/qtest/cdrom-test.c b/tests/qtest/cdrom-test.c index 26a2400181..09655e6ff0 100644 --- a/tests/qtest/cdrom-test.c +++ b/tests/qtest/cdrom-test.c @@ -205,6 +205,11 @@ int main(int argc, char **argv) g_test_init(&argc, &argv, NULL); + if (!qtest_has_accel("tcg") && !qtest_has_accel("kvm")) { + g_test_skip("No KVM or TCG accelerator available"); + return 0; + } + if (exec_genisoimg(genisocheck)) { /* genisoimage not available - so can't run tests */ return g_test_run();