From: "Alex Bennée" <alex.bennee@linaro.org>
To: "Philippe Mathieu-Daudé" <philmd@linaro.org>
Cc: qemu-devel@nongnu.org, qemu-arm@nongnu.org,
"Nicholas Piggin" <npiggin@gmail.com>,
"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
"Markus Armbruster" <armbru@redhat.com>,
"Akihiko Odaki" <akihiko.odaki@daynix.com>,
qemu-ppc@nongnu.org,
"Richard Henderson" <richard.henderson@linaro.org>,
"Thomas Huth" <thuth@redhat.com>,
"David Gibson" <david@gibson.dropbear.id.au>,
"Daniel Henrique Barboza" <danielhb413@gmail.com>,
"Daniel P. Berrangé" <berrange@redhat.com>,
"Ilya Leoshkevich" <iii@linux.ibm.com>,
"Cédric Le Goater" <clg@kaod.org>,
"Peter Maydell" <peter.maydell@linaro.org>,
qemu-s390x@nongnu.org,
"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
"David Hildenbrand" <david@redhat.com>,
"Pierrick Bouvier" <pierrick.bouvier@linaro.org>
Subject: Re: [PATCH v2 11/11] include/gdbstub: add note to helpers.h
Date: Mon, 24 Mar 2025 17:29:15 +0000 [thread overview]
Message-ID: <87v7ry2v84.fsf@draig.linaro.org> (raw)
In-Reply-To: <729ee2cc-ecaf-41d0-afbd-25f51952dbeb@linaro.org> ("Philippe Mathieu-Daudé"'s message of "Mon, 24 Mar 2025 17:46:31 +0100")
Philippe Mathieu-Daudé <philmd@linaro.org> writes:
> On 24/3/25 11:21, Alex Bennée wrote:
>> We've not yet deprecated but we should steer users away from these
>> helpers if they want to be in a single/heterogeneous binary.
>
> Why not deprecate?
I guess philosophically do we expect to eventually convert all frontends
to the new API or only those that want to be in the single binary?
Should I just be more explicit:
>> *
>> * These are all used by the various frontends and have to be host
>> - * aware to ensure things are store in target order.
>> + * aware to ensure things are store in target order. Consider using
>> + * the endian neutral registers.h if you want the architecture to be
>> + * included in an eventual single QEMU binary.
>> *
>> * Copyright (c) 2022 Linaro Ltd
>> *
These are all used by the various frontends and have to be host aware
to ensure things are store in target order.
New front-ends should not be using these APIs at all. They should be
using the endian neutral registers.h as should any architecture that
intends to be included in an eventual single QEMU binary.
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
next prev parent reply other threads:[~2025-03-24 17:29 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-24 10:21 [PATCH v2 00/11] gdbstub: conversion to runtime endianess helpers Alex Bennée
2025-03-24 10:21 ` [PATCH v2 01/11] include/exec: fix assert in size_memop Alex Bennée
2025-03-24 16:40 ` Philippe Mathieu-Daudé
2025-03-24 17:08 ` Richard Henderson
2025-03-24 19:08 ` Pierrick Bouvier
2025-03-29 7:51 ` Akihiko Odaki
2025-03-30 8:25 ` Philippe Mathieu-Daudé
2025-03-24 10:21 ` [PATCH v2 02/11] include/gdbstub: fix include guard in commands.h Alex Bennée
2025-03-24 19:11 ` Pierrick Bouvier
2025-03-24 10:21 ` [PATCH v2 03/11] gdbstub: assert earlier in handle_read_all_regs Alex Bennée
2025-03-24 19:11 ` Pierrick Bouvier
2025-03-24 10:21 ` [PATCH v2 04/11] gdbstub: introduce target independent gdb register helper Alex Bennée
2025-03-24 10:32 ` Alex Bennée
2025-03-29 8:15 ` Akihiko Odaki
2025-03-24 17:32 ` Richard Henderson
2025-03-24 19:07 ` Pierrick Bouvier
2025-03-24 19:21 ` Pierrick Bouvier
2025-03-24 19:34 ` Pierrick Bouvier
2025-03-29 7:58 ` Akihiko Odaki
2025-03-24 10:21 ` [PATCH v2 05/11] target/arm: convert 32 bit gdbstub to new helpers Alex Bennée
2025-03-24 19:16 ` Pierrick Bouvier
2025-03-24 10:21 ` [PATCH v2 06/11] target/arm: convert 64 " Alex Bennée
2025-03-24 19:18 ` Pierrick Bouvier
2025-03-24 10:21 ` [PATCH v2 07/11] target/ppc: expand comment on FP/VMX/VSX access functions Alex Bennée
2025-03-24 17:35 ` Richard Henderson
2025-03-24 19:24 ` Pierrick Bouvier
2025-03-24 10:21 ` [PATCH v2 08/11] target/ppc: make ppc_maybe_bswap_register static Alex Bennée
2025-03-24 17:34 ` Richard Henderson
2025-03-24 19:24 ` Pierrick Bouvier
2025-03-24 10:21 ` [PATCH v2 09/11] target/ppc: convert gdbstub to new helpers Alex Bennée
2025-03-24 17:39 ` Richard Henderson
2025-03-24 19:29 ` Pierrick Bouvier
2025-03-24 20:04 ` Richard Henderson
2025-03-24 20:49 ` Pierrick Bouvier
2025-03-25 9:22 ` Philippe Mathieu-Daudé
2025-03-25 14:21 ` Pierrick Bouvier
2025-03-24 10:21 ` [PATCH v2 10/11] target/microblaze: convert gdbstub to new helper Alex Bennée
2025-03-24 16:45 ` Philippe Mathieu-Daudé
2025-03-24 19:30 ` Pierrick Bouvier
2025-03-24 10:21 ` [PATCH v2 11/11] include/gdbstub: add note to helpers.h Alex Bennée
2025-03-24 16:46 ` Philippe Mathieu-Daudé
2025-03-24 17:29 ` Alex Bennée [this message]
2025-03-24 19:33 ` Pierrick Bouvier
2025-03-24 19:33 ` Pierrick Bouvier
2025-03-24 19:10 ` [PATCH v2 00/11] gdbstub: conversion to runtime endianess helpers 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=87v7ry2v84.fsf@draig.linaro.org \
--to=alex.bennee@linaro.org \
--cc=akihiko.odaki@daynix.com \
--cc=armbru@redhat.com \
--cc=berrange@redhat.com \
--cc=clg@kaod.org \
--cc=danielhb413@gmail.com \
--cc=david@gibson.dropbear.id.au \
--cc=david@redhat.com \
--cc=edgar.iglesias@gmail.com \
--cc=iii@linux.ibm.com \
--cc=npiggin@gmail.com \
--cc=peter.maydell@linaro.org \
--cc=philmd@linaro.org \
--cc=pierrick.bouvier@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=richard.henderson@linaro.org \
--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 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.