[PULL,v2,16/57] tests/tcg: enable building for i386

Message ID	20180621062605.941-17-alex.bennee@linaro.org
State	Accepted
Commit	8b17219e7d01ad3e863656e6e7d26c2185392cd3
Headers	show Delivered-To: patch@linaro.org Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 2001:4830:134:3::11 as permitted sender) client-ip=2001:4830:134:3::11; From: =?utf-8?q?Alex_Benn=C3=A9e?= <alex.bennee@linaro.org> To: peter.maydell@linaro.org Date: Thu, 21 Jun 2018 07:25:24 +0100 Message-Id: <20180621062605.941-17-alex.bennee@linaro.org> In-Reply-To: <20180621062605.941-1-alex.bennee@linaro.org> References: <20180621062605.941-1-alex.bennee@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [Qemu-devel] [PULL v2 16/57] tests/tcg: enable building for i386 Precedence: list Cc: agraf@suse.de, Eduardo Habkost <ehabkost@redhat.com>, =?utf-8?q?Alex_B?= =?utf-8?b?ZW5uw6ll?= <alex.bennee@linaro.org>, richard.henderson@linaro.org, qemu-devel@nongnu.org, f4bug@amsat.org, cota@braap.org, Paolo Bonzini <pbonzini@redhat.com>, famz@redhat.com, aurelien@aurel32.net, Richard Henderson <rth@twiddle.net> Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: "Qemu-devel" <qemu-devel-bounces+patch=linaro.org@nongnu.org>
Series	add check-tcg and associated machinery \| expand [PULL,v2,00/57] add check-tcg and associated machinery [PULL,v2,01/57] docker: docker.py wrap StringIO import for python3 [PULL,v2,02/57] configure: add support for --cross-cc-FOO [PULL,v2,03/57] configure: move i386_cc to cross_cc_i386 [PULL,v2,04/57] configure: allow user to specify --cross-cc-cflags-foo= [PULL,v2,05/57] configure: set cross_cc_FOO for host compiler [PULL,v2,06/57] docker: Add "cc" subcommand [PULL,v2,07/57] docker: extend "cc" command to accept compiler [PULL,v2,08/57] docker: allow "cc" command to run in user context [PULL,v2,09/57] docker: Makefile.include introduce DOCKER_SCRIPT [PULL,v2,10/57] tests/tcg: move architecture independent tests into subdir [PULL,v2,11/57] tests/tcg/multiarch: Build fix for linux-test [PULL,v2,12/57] tests/tcg/multiarch: enable additional linux-test tests [PULL,v2,13/57] tests/tcg/multiarch: don't hard code paths/ports for linux-test [PULL,v2,14/57] tests/tcg/multiarch: move most output to stdout [PULL,v2,15/57] tests/tcg: move i386 specific tests into subdir [PULL,v2,16/57] tests/tcg: enable building for i386 [PULL,v2,17/57] tests/tcg/i386: Build fix for hello-i386 [PULL,v2,18/57] tests/tcg/i386: fix test-i386 [PULL,v2,19/57] tests/tcg/i386: add runner for test-i386-fprem [PULL,v2,20/57] tests/tcg/x86_64: add Makefile.target [PULL,v2,21/57] tests/tcg/i386/test-i386: use modern vector_size attributes [PULL,v2,22/57] tests/tcg/i386/test-i386: fix printf format [PULL,v2,23/57] tests/tcg: move ARM specific tests into subdir [PULL,v2,24/57] tests/tcg: enable building for ARM [PULL,v2,25/57] tests/tcg/arm: fix up test-arm-iwmmxt test [PULL,v2,26/57] tests/tcg: enable building for AArch64 [PULL,v2,27/57] tests/tcg/arm: add fcvt test cases for AArch32/64 [PULL,v2,28/57] tests/tcg: move MIPS specific tests into subdir [PULL,v2,29/57] tests/tcg: enable building for MIPS [PULL,v2,30/57] tests/docker/Makefile.include: fix mipsel-cross dependancy [PULL,v2,31/57] tests/tcg/mips: include common mips hello-mips [PULL,v2,32/57] tests/tcg: enable building for s390x [PULL,v2,33/57] tests/tcg: enable building for ppc64 [PULL,v2,34/57] tests/tcg: enable building for Alpha [PULL,v2,35/57] tests/tcg/alpha: add Alpha specific tests [PULL,v2,36/57] tests/tcg: enable building for HPPA [PULL,v2,37/57] tests/tcg: enable building for m68k [PULL,v2,38/57] tests/tcg: enable building for sh4 [PULL,v2,39/57] tests/tcg: enable building for sparc64 [PULL,v2,40/57] tests/tcg: enable building for mips64 [PULL,v2,41/57] tests/tcg: enable building for RISCV64 [PULL,v2,42/57] docker: move debian-powerpc-cross to sid based build [PULL,v2,43/57] tests/tcg: enable building for PowerPC [PULL,v2,44/57] tests/tcg/Makefile: update to be called from Makefile.target [PULL,v2,45/57] Makefile.target: add (clean-/build-)guest-tests targets [PULL,v2,46/57] tests/Makefile.include: add [build\|clean\|check]-tcg targets [PULL,v2,47/57] tests/tcg: add run, diff, and skip helper macros [PULL,v2,48/57] tests/tcg: override runners for broken tests [PULL,v2,49/57] tests/tcg/i386: extend timeout for runcom test [PULL,v2,50/57] tests: add top-level make dependency for docker builds [PULL,v2,51/57] docker: docker.py use "version" to probe usage [PULL,v2,52/57] docker: docker.py don't conflate checksums for extra_files [PULL,v2,53/57] docker: docker.py add check sub-command [PULL,v2,54/57] tests/Makefile: call sub-makes with SKIP_DOCKER_BUILD=1 [PULL,v2,55/57] docker: docker.py adding age check command [PULL,v2,56/57] tests/docker/Makefile.include: only force SID to NOCACHE if old [PULL,v2,57/57] .travis.yml: add check-tcg test

Message ID

20180621062605.941-17-alex.bennee@linaro.org

State

Accepted

Commit

8b17219e7d01ad3e863656e6e7d26c2185392cd3

Headers

Received-SPF: pass (google.com: domain of
	qemu-devel-bounces+patch=linaro.org@nongnu.org designates
	2001:4830:134:3::11 as permitted sender)
	client-ip=2001:4830:134:3::11; 
From: =?utf-8?q?Alex_Benn=C3=A9e?= <alex.bennee@linaro.org>
To: peter.maydell@linaro.org
Date: Thu, 21 Jun 2018 07:25:24 +0100
Message-Id: <20180621062605.941-17-alex.bennee@linaro.org>
In-Reply-To: <20180621062605.941-1-alex.bennee@linaro.org>
References: <20180621062605.941-1-alex.bennee@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Subject: [Qemu-devel] [PULL v2 16/57] tests/tcg: enable building for i386
Precedence: list
Cc: agraf@suse.de, Eduardo Habkost <ehabkost@redhat.com>, =?utf-8?q?Alex_B?=
	=?utf-8?b?ZW5uw6ll?= <alex.bennee@linaro.org>,
	richard.henderson@linaro.org, qemu-devel@nongnu.org, 
	f4bug@amsat.org, cota@braap.org, Paolo Bonzini <pbonzini@redhat.com>,
	famz@redhat.com, aurelien@aurel32.net,
	Richard Henderson <rth@twiddle.net>
Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org
Sender: "Qemu-devel" <qemu-devel-bounces+patch=linaro.org@nongnu.org>

Series

add check-tcg and associated machinery | expand

Commit Message

Alex Bennée June 21, 2018, 6:25 a.m. UTC

While you can construct a compile command that does work using the
x86_64 host compiler that most people use this is flakey. Different
distros handle this is different ways so we default to using a known
good i386 compiler via docker.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>


-- 
2.17.1

diff --git a/tests/tcg/i386/Makefile.include b/tests/tcg/i386/Makefile.include
new file mode 100644
index 0000000000..be1c3008dd
--- /dev/null
+++ b/tests/tcg/i386/Makefile.include
@@ -0,0 +1,9 @@ 
+#
+# Makefile.include for all i386
+#
+# There is enough brokeness in x86_64 compilers that we don't default
+# to using the x86_64 system compiler for i386 binaries.
+#
+
+DOCKER_IMAGE=fedora-i386-cross
+DOCKER_CROSS_COMPILER=gcc
diff --git a/tests/tcg/i386/Makefile.target b/tests/tcg/i386/Makefile.target
index 2f27b65e2d..8b46cc021e 100644
--- a/tests/tcg/i386/Makefile.target
+++ b/tests/tcg/i386/Makefile.target
@@ -28,3 +28,10 @@  hello-i386: LDFLAGS+=-nostdlib
 test-i386: test-i386.c test-i386-code16.S test-i386-vm86.S test-i386.h test-i386-shift.h test-i386-muldiv.h
 	$(CC) $(CFLAGS) $(LDFLAGS) -o $@ \
 	   $(<D)/test-i386.c $(<D)/test-i386-code16.S $(<D)/test-i386-vm86.S -lm
+
+# Specialist test runners
+run-runcom: runcom pi_10.com
+	$(call quiet-command, $(QEMU) ./runcom $(I386_SRC)/pi_10.com > runcom.out, "TEST", "$< on $(TARGET_NAME)")
+
+# On i386 and x86_64 Linux only supports 4k pages (large pages are a different hack)
+EXTRA_RUNS+=run-test-mmap-4096

[PULL,v2,16/57] tests/tcg: enable building for i386

Commit Message

Patch