From: "Alex Bennée" <alex.bennee@linaro.org>
To: qemu-devel@nongnu.org
Cc: "Michael Roth" <michael.roth@amd.com>,
"Alexander Bulekov" <alxndr@bu.edu>,
"Qiuhao Li" <Qiuhao.Li@outlook.com>,
"Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Markus Armbruster" <armbru@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
qemu-arm@nongnu.org, "John Snow" <jsnow@redhat.com>,
"Pavel Dovgalyuk" <pavel.dovgaluk@ispras.ru>,
"Darren Kenny" <darren.kenny@oracle.com>,
"Alex Bennée" <alex.bennee@linaro.org>,
"Stefan Hajnoczi" <stefanha@redhat.com>,
"Bandan Das" <bsd@redhat.com>, "Cleber Rosa" <crosa@redhat.com>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Bastian Koppelmann" <kbastian@mail.uni-paderborn.de>,
"Yonggang Luo" <luoyonggang@gmail.com>,
"Li-Wen Hsu" <lwhsu@freebsd.org>,
"Daniel P. Berrangé" <berrange@redhat.com>,
"Thomas Huth" <thuth@redhat.com>,
"Beraldo Leal" <bleal@redhat.com>,
"Ed Maste" <emaste@freebsd.org>,
"Wainer dos Santos Moschetta" <wainersm@redhat.com>
Subject: [PATCH 10/12] tests/docker: Use binaries for debian-tricore-cross
Date: Wed, 15 Feb 2023 19:25:28 +0000 [thread overview]
Message-ID: <20230215192530.299263-11-alex.bennee@linaro.org> (raw)
In-Reply-To: <20230215192530.299263-1-alex.bennee@linaro.org>
From: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
since binutils is pretty old, it fails our CI repeatedly during the
compilation of tricore-binutils. We created a precompiled version using
the debian docker image and download it instead of building it ourself.
We also updated the package to include a newer version of binutils, gcc,
and newlib. The default TriCore ISA version used by tricore-as changed
from the old version, so we have to specify it now. If we don't
'test_fadd' fails with 'unknown opcode'.
The new assembler also picks a new encoding in ld.h which fails the
'test_ld_h' test. We fix that by using the newest TriCore CPU for QEMU.
The old assembler accepted an extra ')' in 'test_imask'. The new one
does not, so lets remove it.
Signed-off-by: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
Message-Id: <20230209145812.46730-1-kbastian@mail.uni-paderborn.de>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/tcg/tricore/macros.h | 2 +-
tests/docker/dockerfiles/debian-tricore-cross.docker | 10 +++-------
tests/tcg/tricore/Makefile.softmmu-target | 6 +++---
3 files changed, 7 insertions(+), 11 deletions(-)
diff --git a/tests/tcg/tricore/macros.h b/tests/tcg/tricore/macros.h
index ec4f5bff52..3df2e0de82 100644
--- a/tests/tcg/tricore/macros.h
+++ b/tests/tcg/tricore/macros.h
@@ -174,7 +174,7 @@ test_ ## num: \
TEST_CASE_E(num, res_lo, res_hi, \
LI(DREG_RS1, rs1); \
rstv; \
- insn EREG_CALC_RESULT, imm1, DREG_RS1, imm2); \
+ insn EREG_CALC_RESULT, imm1, DREG_RS1, imm2; \
)
diff --git a/tests/docker/dockerfiles/debian-tricore-cross.docker b/tests/docker/dockerfiles/debian-tricore-cross.docker
index 5ae58efa09..82e4576485 100644
--- a/tests/docker/dockerfiles/debian-tricore-cross.docker
+++ b/tests/docker/dockerfiles/debian-tricore-cross.docker
@@ -20,6 +20,7 @@ RUN apt update && \
bzip2 \
ca-certificates \
ccache \
+ curl \
flex \
g++ \
gcc \
@@ -34,13 +35,8 @@ RUN apt update && \
python3-setuptools \
python3-wheel
-RUN git clone --single-branch \
- https://github.com/bkoppelmann/tricore-binutils.git \
- /usr/src/binutils && \
- cd /usr/src/binutils && chmod +x missing && \
- CFLAGS=-w ./configure --prefix=/usr/local --disable-nls --target=tricore && \
- make && make install && \
- rm -rf /usr/src/binutils
+RUN curl -#SL https://github.com/bkoppelmann/package_940/releases/download/tricore-toolchain-9.40/tricore-toolchain-9.4.0.tar.gz \
+ | tar -xzC /usr/local/
# This image can only build a very minimal QEMU as well as the tests
ENV DEF_TARGET_LIST tricore-softmmu
diff --git a/tests/tcg/tricore/Makefile.softmmu-target b/tests/tcg/tricore/Makefile.softmmu-target
index d2446af8b4..b3cd56fffc 100644
--- a/tests/tcg/tricore/Makefile.softmmu-target
+++ b/tests/tcg/tricore/Makefile.softmmu-target
@@ -1,7 +1,7 @@
TESTS_PATH = $(SRC_PATH)/tests/tcg/tricore
-LDFLAGS = -T$(TESTS_PATH)/link.ld
-ASFLAGS =
+LDFLAGS = -T$(TESTS_PATH)/link.ld --mcpu=tc162
+ASFLAGS = -mtc162
TESTS += test_abs.tst
TESTS += test_bmerge.tst
@@ -19,7 +19,7 @@ TESTS += test_madd.tst
TESTS += test_msub.tst
TESTS += test_muls.tst
-QEMU_OPTS += -M tricore_testboard -nographic -kernel
+QEMU_OPTS += -M tricore_testboard -cpu tc27x -nographic -kernel
%.pS: $(TESTS_PATH)/%.S
$(HOST_CC) -E -o $@ $<
--
2.39.1
next prev parent reply other threads:[~2023-02-15 19:27 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-15 19:25 [PATCH 00/12] testing/next: docker, avocado, unit, Alex Bennée
2023-02-15 19:25 ` [PATCH 01/12] gitlab: tweak and filter ninja output to reduce build noise Alex Bennée
2023-02-16 7:32 ` Thomas Huth
2023-02-15 19:25 ` [PATCH 02/12] tests/avocado: retire the Aarch64 TCG tests from boot_linux.py Alex Bennée
2023-02-22 15:22 ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 03/12] tests: add socat dependency for tests Alex Bennée
2023-02-15 19:52 ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 04/12] tests: be a bit more strict cleaning up fifos Alex Bennée
2023-02-15 20:44 ` Philippe Mathieu-Daudé
2023-02-15 20:51 ` Richard Henderson
2023-02-16 7:34 ` Thomas Huth
2023-02-15 19:25 ` [PATCH 05/12] gitlab: reduce default verbosity of cirrus run Alex Bennée
2023-02-16 7:37 ` Thomas Huth
2023-02-16 8:02 ` Alex Bennée
2023-02-16 8:15 ` Thomas Huth
2023-02-15 19:25 ` [PATCH 06/12] gitlab: extend custom runners with base_job_template Alex Bennée
2023-02-16 7:39 ` Thomas Huth
2023-02-15 19:25 ` [PATCH 07/12] testing: update ubuntu2004 to ubuntu2204 Alex Bennée
2023-02-15 20:53 ` Richard Henderson
2023-02-16 7:55 ` Thomas Huth
2023-02-16 18:15 ` John Snow
2023-02-16 19:44 ` Daniel P. Berrangé
2023-02-17 16:35 ` John Snow
2023-02-17 17:14 ` Daniel P. Berrangé
2023-02-17 17:20 ` John Snow
2023-02-22 15:11 ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 08/12] tests: skip the nios2 replay_kernel test Alex Bennée
2023-02-15 20:47 ` Philippe Mathieu-Daudé
2023-02-15 20:59 ` Richard Henderson
2023-02-22 15:01 ` Philippe Mathieu-Daudé
2023-02-15 20:54 ` Richard Henderson
2023-02-15 19:25 ` [PATCH 09/12] tests: add tuxrun baseline test to avocado Alex Bennée
2023-02-22 15:06 ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` Alex Bennée [this message]
2023-02-15 20:47 ` [PATCH 10/12] tests/docker: Use binaries for debian-tricore-cross Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 11/12] cirrus.yml: Improve the windows_msys2_task Alex Bennée
2023-02-22 15:32 ` Philippe Mathieu-Daudé
2023-02-15 19:25 ` [PATCH 12/12] tests: ensure we export job results for some cross builds Alex Bennée
2023-02-16 7:59 ` Thomas Huth
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230215192530.299263-11-alex.bennee@linaro.org \
--to=alex.bennee@linaro.org \
--cc=Qiuhao.Li@outlook.com \
--cc=alxndr@bu.edu \
--cc=armbru@redhat.com \
--cc=berrange@redhat.com \
--cc=bleal@redhat.com \
--cc=bsd@redhat.com \
--cc=crosa@redhat.com \
--cc=darren.kenny@oracle.com \
--cc=emaste@freebsd.org \
--cc=jsnow@redhat.com \
--cc=kbastian@mail.uni-paderborn.de \
--cc=luoyonggang@gmail.com \
--cc=lwhsu@freebsd.org \
--cc=marcandre.lureau@redhat.com \
--cc=michael.roth@amd.com \
--cc=pavel.dovgaluk@ispras.ru \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=philmd@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
--cc=thuth@redhat.com \
--cc=wainersm@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).