From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
To: qemu-devel@nongnu.org
Cc: qemu-arm@nongnu.org, "Alex Bennée" <alex.bennee@linaro.org>,
"Alistair Francis" <alistair.francis@wdc.com>,
"Weiwei Li" <liwei1518@gmail.com>,
"Palmer Dabbelt" <palmer@dabbelt.com>,
qemu-riscv@nongnu.org, richard.henderson@linaro.org,
"Daniel Henrique Barboza" <dbarboza@ventanamicro.com>,
philmd@linaro.org, "Liu Zhiwei" <zhiwei_liu@linux.alibaba.com>,
"Peter Maydell" <peter.maydell@linaro.org>
Subject: Re: [PATCH v4 00/12] single-binary: compile once semihosting
Date: Fri, 22 Aug 2025 08:05:15 -0700 [thread overview]
Message-ID: <40cf55e8-b7d5-49ee-8c0e-10c32a1df00f@linaro.org> (raw)
In-Reply-To: <20250822150058.18692-1-pierrick.bouvier@linaro.org>
On 2025-08-22 08:00, Pierrick Bouvier wrote:
> This series compiles once semihosting files in system mode.
> The most complicated file was semihosting/arm-compat-semi.c, which was carefully
> cleaned in easy to understand steps.
>
> v2
> --
>
> - use vaddr for syscalls.c/.h
> - static qualifier for console_{in,out}_gf
> - use vaddr for arm-compat-semi.c
> - semihosting/arm-compat-semi: fix cast for common_semi_set_ret
>
> v3
> --
>
> - keep common_semi_sys_exit_extended (Peter)
>
> v4
> --
>
> - fix patch 9 commit message to s/uint64_t/vaddr (Phil)
>
> Pierrick Bouvier (12):
> semihosting/syscalls: compile once in system and per target for user
> mode
> semihosting/syscalls: replace uint64_t with vaddr where appropriate
> semihosting/guestfd: compile once for system/user
> semihosting/arm-compat-semi: change common_semi_sys_exit_extended
> target/riscv/common-semi-target: remove sizeof(target_ulong)
> target/{arm,riscv}/common-semi-target: eradicate target_ulong
> include/semihosting/common-semi: extract common_semi API
> semihosting/arm-compat-semi: eradicate sizeof(target_ulong)
> semihosting/arm-compat-semi: replace target_ulong with vaddr
> semihosting/arm-compat-semi: eradicate target_long
> semihosting/arm-compat-semi: remove dependency on cpu.h
> semihosting/arm-compat-semi: compile once in system and per target for
> user mode
>
> include/semihosting/common-semi.h | 6 +
> include/semihosting/guestfd.h | 7 --
> include/semihosting/semihost.h | 2 +
> include/semihosting/syscalls.h | 30 ++---
> semihosting/arm-compat-semi-stub.c | 19 +++
> semihosting/arm-compat-semi.c | 65 ++++++++---
> semihosting/guestfd.c | 26 +----
> semihosting/syscalls.c | 109 +++++++++---------
> ...mon-semi-target.h => common-semi-target.c} | 22 ++--
> ...mon-semi-target.h => common-semi-target.c} | 27 +++--
> semihosting/meson.build | 18 +--
> target/arm/meson.build | 4 +
> target/riscv/meson.build | 4 +
> 13 files changed, 190 insertions(+), 149 deletions(-)
> create mode 100644 semihosting/arm-compat-semi-stub.c
> rename target/arm/{common-semi-target.h => common-semi-target.c} (59%)
> rename target/riscv/{common-semi-target.h => common-semi-target.c} (53%)
>
Ping on this series.
I'll be out next week, but it should be ready to be pulled once the
trunk reopens.
Regards,
Pierrick
prev parent reply other threads:[~2025-08-22 15:09 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-22 15:00 [PATCH v4 00/12] single-binary: compile once semihosting Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 01/12] semihosting/syscalls: compile once in system and per target for user mode Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 02/12] semihosting/syscalls: replace uint64_t with vaddr where appropriate Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 03/12] semihosting/guestfd: compile once for system/user Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 04/12] semihosting/arm-compat-semi: change common_semi_sys_exit_extended Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 05/12] target/riscv/common-semi-target: remove sizeof(target_ulong) Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 06/12] target/{arm, riscv}/common-semi-target: eradicate target_ulong Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 07/12] include/semihosting/common-semi: extract common_semi API Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 08/12] semihosting/arm-compat-semi: eradicate sizeof(target_ulong) Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 09/12] semihosting/arm-compat-semi: replace target_ulong with vaddr Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 10/12] semihosting/arm-compat-semi: eradicate target_long Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 11/12] semihosting/arm-compat-semi: remove dependency on cpu.h Pierrick Bouvier
2025-08-22 15:00 ` [PATCH v4 12/12] semihosting/arm-compat-semi: compile once in system and per target for user mode Pierrick Bouvier
2025-08-22 15:05 ` Pierrick Bouvier [this message]
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=40cf55e8-b7d5-49ee-8c0e-10c32a1df00f@linaro.org \
--to=pierrick.bouvier@linaro.org \
--cc=alex.bennee@linaro.org \
--cc=alistair.francis@wdc.com \
--cc=dbarboza@ventanamicro.com \
--cc=liwei1518@gmail.com \
--cc=palmer@dabbelt.com \
--cc=peter.maydell@linaro.org \
--cc=philmd@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-riscv@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=zhiwei_liu@linux.alibaba.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).