* [PULL 1/3] Revert "Makefile: include tests/Makefile.include before ninja calculation"
2026-06-01 16:18 [PULL 0/3] testing hotfixes (Makefile, gitlab, docker) Alex Bennée
@ 2026-06-01 16:18 ` Alex Bennée
2026-06-01 16:18 ` [PULL 2/3] gitlab: work around the inability to build targets for MacOS Alex Bennée
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Alex Bennée @ 2026-06-01 16:18 UTC (permalink / raw)
To: qemu-devel
Cc: Alex Bennée, Pierrick Bouvier, Cornelia Huck, Paolo Bonzini
This reverts commit fd63125b9089610d63a53c11ff2fda6b53a2265d which
broke the ability to run the check-functional-FOO series of tests.
We will need to try something else for the MacOS builds.
Reviewed-by: Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>
Tested-by: Cornelia Huck <cohuck@redhat.com> # running tests on an s390x
Message-ID: <20260601143129.144786-2-alex.bennee@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
diff --git a/Makefile b/Makefile
index 829c51e364c..54547a37b1a 100644
--- a/Makefile
+++ b/Makefile
@@ -45,8 +45,6 @@ include config-host.mak
include Makefile.prereqs
Makefile.prereqs: config-host.mak
-include $(SRC_PATH)/tests/Makefile.include
-
# 0. ensure the build tree is okay
# Check that we're not trying to do an out-of-tree build from
@@ -182,6 +180,8 @@ endif # config-host.mak does not exist
SUBDIR_MAKEFLAGS=$(if $(V),,--no-print-directory --quiet)
+include $(SRC_PATH)/tests/Makefile.include
+
all: recurse-all
SUBDIR_RULES=$(foreach t, all clean distclean, $(addsuffix /$(t), $(SUBDIRS)))
--
2.47.3
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PULL 2/3] gitlab: work around the inability to build targets for MacOS
2026-06-01 16:18 [PULL 0/3] testing hotfixes (Makefile, gitlab, docker) Alex Bennée
2026-06-01 16:18 ` [PULL 1/3] Revert "Makefile: include tests/Makefile.include before ninja calculation" Alex Bennée
@ 2026-06-01 16:18 ` Alex Bennée
2026-06-01 16:18 ` [PULL 3/3] configure: use debian-all-test-cross for mipsel tcg tests Alex Bennée
2026-06-02 17:23 ` [PULL 0/3] testing hotfixes (Makefile, gitlab, docker) Stefan Hajnoczi
3 siblings, 0 replies; 5+ messages in thread
From: Alex Bennée @ 2026-06-01 16:18 UTC (permalink / raw)
To: qemu-devel; +Cc: Alex Bennée, Pierrick Bouvier, Cornelia Huck
Unfortunately a previous fix to ensure .ninja-goals was set for the
TCG tests broken the ability to run check-functional-FOO. As we have
now reverted we need a solution for the MacOS gitlab run. The simplest
is to add an explicit make invocation to build the signed binaries
before we run the tests.
Reviewed-by: Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>
Tested-by: Cornelia Huck <cohuck@redhat.com> # running tests on an s390x
Message-ID: <20260601143129.144786-3-alex.bennee@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
diff --git a/.gitlab-ci.d/macos.yml b/.gitlab-ci.d/macos.yml
index f61750a3f4a..8366b4ad564 100644
--- a/.gitlab-ci.d/macos.yml
+++ b/.gitlab-ci.d/macos.yml
@@ -29,6 +29,7 @@
- cd build
- ../configure --enable-werror $CONFIGURE_ARGS || { cat config.log meson-logs/meson-log.txt; exit 1; }
- $MAKE -j$(sysctl -n hw.ncpu)
+ - for TARGET in $TEST_BINARIES ; do $MAKE $TARGET ; done
- for TARGET in $TEST_TARGETS ; do $MAKE $TARGET ; done
aarch64-macos-15-build:
@@ -44,6 +45,7 @@ aarch64-macos-15-build:
--cross-prefix-i386=i686-elf-
--cross-prefix-x86_64=x86_64-elf-
--disable-plugins
+ TEST_BINARIES: qemu-system-aarch64 qemu-system-i386 qemu-system-x86_64
TEST_TARGETS: check-unit run-tcg-tests-aarch64-softmmu run-tcg-tests-i386-softmmu run-tcg-tests-x86_64-softmmu
aarch64-macos-26-build:
@@ -60,4 +62,5 @@ aarch64-macos-26-build:
--cross-prefix-i386=i686-elf-
--cross-prefix-x86_64=x86_64-elf-
--disable-plugins
+ TEST_BINARIES: qemu-system-aarch64 qemu-system-i386 qemu-system-x86_64
TEST_TARGETS: check-unit run-tcg-tests-aarch64-softmmu run-tcg-tests-i386-softmmu run-tcg-tests-x86_64-softmmu
--
2.47.3
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PULL 3/3] configure: use debian-all-test-cross for mipsel tcg tests
2026-06-01 16:18 [PULL 0/3] testing hotfixes (Makefile, gitlab, docker) Alex Bennée
2026-06-01 16:18 ` [PULL 1/3] Revert "Makefile: include tests/Makefile.include before ninja calculation" Alex Bennée
2026-06-01 16:18 ` [PULL 2/3] gitlab: work around the inability to build targets for MacOS Alex Bennée
@ 2026-06-01 16:18 ` Alex Bennée
2026-06-02 17:23 ` [PULL 0/3] testing hotfixes (Makefile, gitlab, docker) Stefan Hajnoczi
3 siblings, 0 replies; 5+ messages in thread
From: Alex Bennée @ 2026-06-01 16:18 UTC (permalink / raw)
To: qemu-devel
Cc: Alex Bennée, Philippe Mathieu-Daudé, Pierrick Bouvier,
Cornelia Huck, Paolo Bonzini, Philippe Mathieu-Daudé,
Jiaxun Yang
Although we have had the mips compilers in all-test-cross for a while
we had been surviving using the mipsel cross compiler image. However
when that was removed we missed updating the container to use.
Fixes: 366bb88e785 (buildsys: Remove MIPS cross containers)
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>
Tested-by: Cornelia Huck <cohuck@redhat.com> # running tests on an s390x
Message-ID: <20260601143129.144786-4-alex.bennee@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
diff --git a/configure b/configure
index e138d83bc9f..1ceb4d7ecc4 100755
--- a/configure
+++ b/configure
@@ -1430,7 +1430,7 @@ probe_target_compiler() {
container_image=debian-all-test-cross
container_cross_prefix=aarch64-linux-gnu-
;;
- alpha|hppa|m68k|mips|riscv64|sh4|sparc64)
+ alpha|hppa|m68k|mips|mipsel|riscv64|sh4|sparc64)
container_image=debian-all-test-cross
;;
mips64)
--
2.47.3
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PULL 0/3] testing hotfixes (Makefile, gitlab, docker)
2026-06-01 16:18 [PULL 0/3] testing hotfixes (Makefile, gitlab, docker) Alex Bennée
` (2 preceding siblings ...)
2026-06-01 16:18 ` [PULL 3/3] configure: use debian-all-test-cross for mipsel tcg tests Alex Bennée
@ 2026-06-02 17:23 ` Stefan Hajnoczi
3 siblings, 0 replies; 5+ messages in thread
From: Stefan Hajnoczi @ 2026-06-02 17:23 UTC (permalink / raw)
To: Alex Bennée; +Cc: qemu-devel, Alex Bennée
[-- Attachment #1: Type: text/plain, Size: 116 bytes --]
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/11.1 for any user-visible changes.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread