diff options
| author | Ilya Leoshkevich <iii@linux.ibm.com> | 2024-12-16 14:37:56 +0100 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2025-01-07 14:47:56 +0100 |
| commit | 3fa010d5317e64276c67ed468b994a2aaa570475 (patch) | |
| tree | 6d72102b7ccbbb657e2bbc10a28dd2f85b834077 | |
| parent | 6528013b5f5ba6bb3934b7f5fe57a3110680530f (diff) | |
| download | qemu-3fa010d5317e64276c67ed468b994a2aaa570475.zip qemu-3fa010d5317e64276c67ed468b994a2aaa570475.tar.gz qemu-3fa010d5317e64276c67ed468b994a2aaa570475.tar.bz2 | |
tests/tcg/s390x: Use the SLOF libc headers for the multiarch tests
Compiling the s390x tests on Fedora, which has the s390x cross-compiler
installed, fails with:
In file included from qemu/tests/tcg/s390x/console.c:8:
qemu/tests/tcg/s390x/../../../pc-bios/s390-ccw/sclp.c:11:10: fatal error: string.h: No such file or directory
11 | #include <string.h>
This is because Fedora does not have a cross-libc. Since console.c
already uses the SLOF libc implementation, add the respective headers
to the include path.
Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20241216133819.78583-1-iii@linux.ibm.com>
Tested-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
| -rw-r--r-- | tests/tcg/s390x/Makefile.softmmu-target | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/tcg/s390x/Makefile.softmmu-target b/tests/tcg/s390x/Makefile.softmmu-target index 969bc57..7adde2f 100644 --- a/tests/tcg/s390x/Makefile.softmmu-target +++ b/tests/tcg/s390x/Makefile.softmmu-target @@ -45,7 +45,8 @@ TESTS += $(ASM_TESTS) S390X_MULTIARCH_RUNTIME_OBJS = head64.o console.o $(MINILIB_OBJS) $(MULTIARCH_TESTS): $(S390X_MULTIARCH_RUNTIME_OBJS) $(MULTIARCH_TESTS): LDFLAGS += $(S390X_MULTIARCH_RUNTIME_OBJS) -$(MULTIARCH_TESTS): CFLAGS += $(MINILIB_INC) +$(MULTIARCH_TESTS): CFLAGS += $(MINILIB_INC) \ + -I$(SRC_PATH)/roms/SLOF/lib/libc/include/ memory: CFLAGS += -DCHECK_UNALIGNED=0 # s390x clears the BSS section so we need to account for that |