From: "Alex Bennée" <alex.bennee@linaro.org>
To: Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>
Cc: qemu-devel@nongnu.org, "Cleber Rosa" <crosa@redhat.com>,
"Helge Deller" <deller@gmx.de>,
"Daniel P. Berrangé" <berrange@redhat.com>,
"Fabiano Rosas" <farosas@suse.de>,
qemu-arm@nongnu.org, "Brian Cain" <brian.cain@oss.qualcomm.com>,
"Thomas Huth" <th.huth+qemu@posteo.eu>,
"Matthew Rosato" <mjrosato@linux.ibm.com>,
"David Hildenbrand" <david@kernel.org>,
"Philippe Mathieu-Daudé" <philmd@mailo.com>,
"Song Gao" <gaosong@loongson.cn>, "Warner Losh" <imp@bsdimp.com>,
"John Snow" <jsnow@redhat.com>,
"Eric Farman" <farman@linux.ibm.com>,
"Cornelia Huck" <cohuck@redhat.com>,
qemu-s390x@nongnu.org, "Jiaxun Yang" <jiaxun.yang@flygoat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Richard Henderson" <richard.henderson@linaro.org>,
"Ilya Leoshkevich" <iii@linux.ibm.com>,
"Laurent Vivier" <laurent@vivier.eu>,
"Brad Smith" <brad@comstyle.com>,
"Kyle Evans" <kevans@freebsd.org>
Subject: Re: [PATCH v5 00/17] testing: various docker clean-ups post meson tcg conversion (pre-PR)
Date: Thu, 17 Sep 2026 18:45:47 +0100 [thread overview]
Message-ID: <87qzir7ovo.fsf@draig.linaro.org> (raw)
In-Reply-To: <c0c5cf55-b4ff-495b-babd-3a75fd991127@oss.qualcomm.com> (Pierrick Bouvier's message of "Thu, 17 Sep 2026 10:20:12 -0700")
Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com> writes:
> On 9/17/2026 9:55 AM, Alex Bennée wrote:
>> This cleans up a few bits and pieces although doesn't totally solve
>> the issue of an apparent stall while meson builds the docker image.
>> However with these changes there should be no difference between the
>> meson build and the equivalent "make docker-image-foo" invocation.
>>
>> We also finally get rid of the ill-advised magic caching support in
>> docker.py.
>>
>> v5
>> - fixup a bunch of rebase damage
>> - swap out my ccache fix for Anders
>> - rename revert to "configure: ensure it also honours container configuration"
>> - use Pierricks solution for avoid too many SKIPs while validating config
>>
>
> It seems that this version removes patch:
> Revert "tests/tcg/meson.build: implement ./configure -cross-cc-* options"
>
> That's fine and correct, but we're missing two things:
> - cross-cc-cflags and prefix should be passed to config.mak also
> (similar to the merge for container options). Not strictly needed as I'm
> not sure anyone needs this "extra" flexibility for roms/vdso, but needed
> if we want to make sure we don't break anything for anyone.
The rom builds are outside the main makefile anyway so they just include
random bits as they want.
> - scripts/meson_buildoptions.sh update that was missing originally for
> those meson options.
We need to do something about it because at various times
"update-buildoptions" does and doesn't change things. Sometimes there
are more options than I've been messing with. It is super confusing.
>
> For the rest, it looks good and ready!
>
> Regards,
> Pierrick
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
next prev parent reply other threads:[~2026-09-17 17:46 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-17 16:55 [PATCH v5 00/17] testing: various docker clean-ups post meson tcg conversion (pre-PR) Alex Bennée
2026-09-17 16:55 ` [PATCH v5 01/17] Revert "configure: remove cross_cc and gdb detection logic" Alex Bennée
2026-09-17 16:55 ` [PATCH v5 02/17] configure: ensure it also honours container configuration Alex Bennée
2026-09-17 16:55 ` [PATCH v5 03/17] configure: correctly honour --disable-containers Alex Bennée
2026-09-17 16:55 ` [PATCH v5 04/17] configure: re-purpose the tcg test compiler configs for vdso building Alex Bennée
2026-09-17 16:55 ` [PATCH v5 05/17] tests/tcg/s390x: Provide the libc bits needed by the system tests Alex Bennée
2026-09-17 16:55 ` [PATCH v5 06/17] tests/tcg: prefix meson built docker tags with qemu/ Alex Bennée
2026-09-17 16:55 ` [PATCH v5 07/17] tests/tcg: use --add-current-user when building containers Alex Bennée
2026-09-17 16:55 ` [PATCH v5 08/17] tests/tcg: don't define tests when missing compilers Alex Bennée
2026-09-17 17:14 ` Pierrick Bouvier
2026-09-17 16:55 ` [PATCH v5 09/17] tests/docker: add container registry to configure and use Alex Bennée
2026-09-17 16:55 ` [PATCH v5 10/17] tests/docker: remove used _read_qemu_dockerfile Alex Bennée
2026-09-17 16:55 ` [PATCH v5 11/17] tests/docker: strip remaining caching logic from docker.py Alex Bennée
2026-09-17 21:19 ` Pierrick Bouvier
2026-09-17 21:49 ` Alex Bennée
2026-09-17 22:00 ` Pierrick Bouvier
2026-09-17 22:18 ` Richard Henderson
2026-09-17 22:25 ` Pierrick Bouvier
2026-09-18 7:30 ` Alex Bennée
2026-09-18 8:05 ` Pierrick Bouvier
2026-09-18 8:29 ` Pierrick Bouvier
2026-09-18 11:57 ` Alex Bennée
2026-09-18 17:07 ` Pierrick Bouvier
2026-09-18 17:58 ` Richard Henderson
2026-09-18 19:22 ` Pierrick Bouvier
2026-09-18 19:53 ` Richard Henderson
2026-09-18 20:21 ` Pierrick Bouvier
2026-09-17 16:55 ` [PATCH v5 12/17] tests/docker: add specific command to parser Alex Bennée
2026-09-17 16:55 ` [PATCH v5 13/17] tests/docker: fix ccache directory variable expansion Alex Bennée
2026-09-17 17:21 ` Pierrick Bouvier
2026-09-17 16:55 ` [PATCH v5 14/17] gitlab: update the @user id mapping file Alex Bennée
2026-09-17 16:55 ` [PATCH v5 15/17] tests/tcg: add console property to docker image custom target Alex Bennée
2026-09-17 16:56 ` [PATCH v5 16/17] tests/tcg: Fix building outside of the source tree Alex Bennée
2026-09-17 16:56 ` [PATCH v5 17/17] gitlab: drop check-tcg from clang-system job Alex Bennée
2026-09-17 17:15 ` Pierrick Bouvier
2026-09-17 17:20 ` [PATCH v5 00/17] testing: various docker clean-ups post meson tcg conversion (pre-PR) Pierrick Bouvier
2026-09-17 17:45 ` Alex Bennée [this message]
2026-09-17 18:29 ` Pierrick Bouvier
2026-09-17 18:53 ` Alex Bennée
2026-09-17 22:13 ` Pierrick Bouvier
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=87qzir7ovo.fsf@draig.linaro.org \
--to=alex.bennee@linaro.org \
--cc=berrange@redhat.com \
--cc=brad@comstyle.com \
--cc=brian.cain@oss.qualcomm.com \
--cc=cohuck@redhat.com \
--cc=crosa@redhat.com \
--cc=david@kernel.org \
--cc=deller@gmx.de \
--cc=farman@linux.ibm.com \
--cc=farosas@suse.de \
--cc=gaosong@loongson.cn \
--cc=iii@linux.ibm.com \
--cc=imp@bsdimp.com \
--cc=jiaxun.yang@flygoat.com \
--cc=jsnow@redhat.com \
--cc=kevans@freebsd.org \
--cc=laurent@vivier.eu \
--cc=marcandre.lureau@redhat.com \
--cc=mjrosato@linux.ibm.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=philmd@mailo.com \
--cc=pierrick.bouvier@oss.qualcomm.com \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=th.huth+qemu@posteo.eu \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.