From: Paolo Bonzini <pbonzini@redhat.com>
To: Richard Henderson <richard.henderson@linaro.org>,
Matheus Kowalczuk Ferst <matheus.ferst@eldorado.org.br>
Cc: "Alex Bennée" <alex.bennee@linaro.org>,
"peter.maydell@linaro.org" <peter.maydell@linaro.org>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
"Leandro Lupori" <leandro.lupori@eldorado.org.br>
Subject: Re: [PULL 20/33] configure: handle host compiler in probe_target_compiler
Date: Wed, 22 Jun 2022 10:39:13 +0200 [thread overview]
Message-ID: <a28f5766-2460-f88f-7d5d-2232f113f1cc@redhat.com> (raw)
In-Reply-To: <c11648fe-7bff-41df-f11f-bdb46f8a672c@linaro.org>
On 6/21/22 20:30, Richard Henderson wrote:
> On 6/21/22 09:58, Paolo Bonzini wrote:
>> On 6/20/22 18:41, Matheus Kowalczuk Ferst wrote:
>>> On 17/06/2022 07:12, Paolo Bonzini wrote:
>>>> Hi Matheus,
>>>>
>>>> could you please test the tests-tcg-next branch at
>>>> https://gitlab.com/bonzini/qemu?
>>> At be6090bcac10, it works if no BE toolchain is present. Otherwise, the
>>> script probes powerpc64-linux-gnu-gcc instead of the native tools for
>>> ppc64le-linux-user, and then do_compiler fails because the
>>> $target_cflags contains -mlittle-endian.
>>>
>>
>> So the problem in that case is that powerpc64-linux-gnu-gcc is _not_
>> biarch and thus does not support -mlittle-endian? (I thought PPC
>> compilers were all 32/64-bit and bi-endian).
>
> Nit: this is ppc64le-linux-gnu-gcc, built for gcc135, a power9 ppc64le
> host. It *does* support -mbig-endian, but not -m32.
>
> At least gcc11 as release was not biarch, with no special configure
> arguments. I can try upgrading to the current gcc12 release to see if
> that changed...
Ok, yesterday I had tested with RHEL and there the ppc64le compiler is
bi-arch and bi-endian, but multilibs are disabled. So it cannot build
32-bit hosted binaries like those for tests/tcg, only freestanding
binaries (vof).
On the other hand the powerpc64-linux-gnu-gcc binary from the cross-gcc
package is theoretically multilib-friendly, but it cannot find the CRT
files on the host because they are not in the .../le multilib subdirectory.
The simplest way out is to just test both native and cross. There is
already code to check for a working compiler in the tests/tcg stanza,
and it can be moved to probe_target_compiler.
Paolo
next prev parent reply other threads:[~2022-06-22 8:40 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-01 18:05 [PULL 00/33] testing updates (gitlab, junit, lcitool, x-compile) Alex Bennée
2022-06-01 18:05 ` [PULL 01/33] .gitlab-ci.d/container-cross: Fix RISC-V container dependencies / stages Alex Bennée
2022-06-01 18:05 ` [PULL 02/33] .gitlab-ci.d/crossbuilds: Fix the dependency of the cross-i386-tci job Alex Bennée
2022-06-01 18:05 ` [PULL 03/33] gitlab-ci: add meson JUnit test result into report Alex Bennée
2022-06-01 18:05 ` [PULL 04/33] meson.build: fix summary display of test compilers Alex Bennée
2022-06-01 18:05 ` [PULL 05/33] tests/lcitool: fix up indentation to correct style Alex Bennée
2022-06-01 18:05 ` [PULL 06/33] tests/docker: update debian-armhf-cross with lcitool Alex Bennée
2022-06-01 18:05 ` [PULL 07/33] tests/docker: update debian-armel-cross " Alex Bennée
2022-06-01 18:05 ` [PULL 08/33] tests/docker: update debian-mipsel-cross " Alex Bennée
2022-06-01 18:05 ` [PULL 09/33] tests/docker: update debian-mips64el-cross " Alex Bennée
2022-06-01 18:05 ` [PULL 10/33] tests/docker: update debian-ppc64el-cross " Alex Bennée
2022-06-01 18:05 ` [PULL 11/33] tests/docker: update debian-amd64 " Alex Bennée
2022-06-01 18:05 ` [PULL 12/33] configure: do not define or use the CPP variable Alex Bennée
2022-06-01 18:05 ` [PULL 13/33] build: clean up ninja invocation Alex Bennée
2022-06-01 18:05 ` [PULL 14/33] build: add a more generic way to specify make->ninja dependencies Alex Bennée
2022-06-01 18:05 ` [PULL 15/33] build: do a full build before running TCG tests Alex Bennée
2022-06-01 18:05 ` [PULL 16/33] configure, meson: move symlinking of ROMs to meson Alex Bennée
2022-06-01 18:05 ` [PULL 17/33] tests/tcg: correct target CPU for sparc32 Alex Bennée
2022-06-01 18:05 ` [PULL 18/33] tests/tcg: merge configure.sh back into main configure script Alex Bennée
2022-06-01 18:05 ` [PULL 19/33] configure: add missing cross compiler fallbacks Alex Bennée
2022-06-01 18:05 ` [PULL 20/33] configure: handle host compiler in probe_target_compiler Alex Bennée
2022-06-15 11:57 ` Matheus Kowalczuk Ferst
2022-06-16 1:20 ` Alex Bennée
2022-06-17 10:12 ` Paolo Bonzini
2022-06-20 16:41 ` Matheus Kowalczuk Ferst
2022-06-21 1:38 ` Richard Henderson
2022-06-21 16:58 ` Paolo Bonzini
2022-06-21 18:30 ` Richard Henderson
2022-06-22 8:39 ` Paolo Bonzini [this message]
2022-06-01 18:05 ` [PULL 21/33] configure: introduce --cross-prefix-*= Alex Bennée
2022-06-01 18:05 ` [PULL 22/33] configure: include more binutils in tests/tcg makefile Alex Bennée
2022-06-01 18:05 ` [PULL 23/33] configure: move symlink configuration earlier Alex Bennée
2022-06-01 18:05 ` [PULL 24/33] configure: enable cross-compilation of s390-ccw Alex Bennée
2022-06-01 18:05 ` [PULL 25/33] configure: enable cross-compilation of optionrom Alex Bennée
2022-06-01 18:05 ` [PULL 26/33] configure: enable cross compilation of vof Alex Bennée
2022-06-01 18:05 ` [PULL 27/33] configure: remove unused variables from config-host.mak Alex Bennée
2022-06-01 18:05 ` [PULL 28/33] gitlab: introduce a common base job template Alex Bennée
2022-06-01 18:05 ` [PULL 29/33] gitlab: convert Cirrus jobs to .base_job_template Alex Bennée
2022-06-01 18:05 ` [PULL 30/33] gitlab: convert static checks " Alex Bennée
2022-06-01 18:05 ` [PULL 31/33] gitlab: convert build/container jobs " Alex Bennée
2022-06-01 18:05 ` [PULL 32/33] gitlab: don't run CI jobs in forks by default Alex Bennée
2022-06-02 10:50 ` Philippe Mathieu-Daudé via
2022-06-06 11:01 ` Daniel P. Berrangé
2022-06-01 18:05 ` [PULL 33/33] docs/devel: clean-up the CI links in the docs Alex Bennée
2022-06-01 21:21 ` [PULL 00/33] testing updates (gitlab, junit, lcitool, x-compile) Richard Henderson
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=a28f5766-2460-f88f-7d5d-2232f113f1cc@redhat.com \
--to=pbonzini@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=leandro.lupori@eldorado.org.br \
--cc=matheus.ferst@eldorado.org.br \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.org \
/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).