From: "Alex Bennée" <alex.bennee@linaro.org>
To: qemu-devel@nongnu.org
Cc: qemu-ppc@nongnu.org,
"Yoshinori Sato" <ysato@users.sourceforge.jp>,
qemu-arm@nongnu.org, "Liu Zhiwei" <zhiwei_liu@linux.alibaba.com>,
"Michael Rolnik" <mrolnik@gmail.com>,
"Yanan Wang" <wangyanan55@huawei.com>,
"Eduardo Habkost" <eduardo@habkost.net>,
"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
"Beraldo Leal" <bleal@redhat.com>,
"Cleber Rosa" <crosa@redhat.com>,
"David Hildenbrand" <david@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Song Gao" <gaosong@loongson.cn>,
"Daniel Henrique Barboza" <danielhb413@gmail.com>,
"Laurent Vivier" <laurent@vivier.eu>,
"Warner Losh" <imp@bsdimp.com>,
"Alex Bennée" <alex.bennee@linaro.org>,
"Alexandre Iooss" <erdnaxe@crans.org>,
"Ilya Leoshkevich" <iii@linux.ibm.com>,
"Cédric Le Goater" <clg@kaod.org>,
qemu-s390x@nongnu.org, "Kyle Evans" <kevans@freebsd.org>,
"Brad Smith" <brad@comstyle.com>,
"Nicholas Piggin" <npiggin@gmail.com>,
"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
"Pierrick Bouvier" <pierrick.bouvier@linaro.org>,
"John Snow" <jsnow@redhat.com>,
"Alistair Francis" <alistair.francis@wdc.com>,
"Richard Henderson" <richard.henderson@linaro.org>,
"Brian Cain" <bcain@quicinc.com>,
"Thomas Huth" <thuth@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
qemu-riscv@nongnu.org, "Bin Meng" <bin.meng@windriver.com>,
"Daniel Henrique Barboza" <dbarboza@ventanamicro.com>,
"Weiwei Li" <liwei1518@gmail.com>,
"Marcel Apfelbaum" <marcel.apfelbaum@gmail.com>,
"Palmer Dabbelt" <palmer@dabbelt.com>,
"Mahmoud Mandour" <ma.mandourr@gmail.com>
Subject: [PATCH v4 00/29] maintainer updates for 9.0 pre-PR (tests, plugin register support)
Date: Tue, 27 Feb 2024 14:43:06 +0000 [thread overview]
Message-ID: <20240227144335.1196131-1-alex.bennee@linaro.org> (raw)
The test patch is a simple house keeping one to clean up some
inadvertent GPLv3 tagging to GPLv2-or-later. I've also increased the
timeout for check-tcg due to TCI timeouts.
The main bulk of this series is register reading support for TCG
plugins. The main change to the API is that the get/read_register
functions are now implicitly vCPU context only so have dropped the
vcpu index parameter and use current_cpu instead. This avoids the need
for locking the gdbstub structures in case there is a cross-vCPU
access.
I've pulled in some patches from Pierrick's inline enhancement series
that allow this to happen (making sure the init callback is in the
vCPU context).
Changes for v4
Addressed final comments from Akihiko
Added BSD changes
Changes for v3:
Now we just use GINT_TO_POINTER to wrap the gdb_regnum with a comment
this might become and opaque structure later. Either way the plugin
shouldn't treat it like anything other than an anonymous handle.
The following still need review:
tests/tcg: expand insn test case to exercise register API
gdbstub: expose api to find registers
tests/vm: update openbsd image to 7.4 (0 acks, 1 sobs, 1 tbs)
tests/vm: avoid re-building the VM images all the time
But I'd like to get this merged soon so its not a mad rush before soft
freeze and so we can get the new inline support stabilised and merged.
Alex.
Akihiko Odaki (11):
target/arm: Use GDBFeature for dynamic XML
target/ppc: Use GDBFeature for dynamic XML
target/riscv: Use GDBFeature for dynamic XML
gdbstub: Use GDBFeature for gdb_register_coprocessor
gdbstub: Use GDBFeature for GDBRegisterState
gdbstub: Change gdb_get_reg_cb and gdb_set_reg_cb
gdbstub: Simplify XML lookup
gdbstub: Infer number of core registers from XML
hw/core/cpu: Remove gdb_get_dynamic_xml member
gdbstub: Add members to identify registers to GDBFeature
plugins: Use different helpers when reading registers
Alex Bennée (13):
tests/tcg: update licenses to GPLv2 as intended
tests/tcg: bump TCG test timeout to 120s
tests/vm: avoid re-building the VM images all the time
tests/vm: update openbsd image to 7.4
linux-user: ensure nios2 processes queued work
gdbstub: expose api to find registers
plugins: create CPUPluginState and migrate plugin_mask
plugins: add an API to read registers
tests/tcg: expand insn test case to exercise register API
contrib/plugins: fix imatch
contrib/plugins: extend execlog to track register changes
docs/devel: lift example and plugin API sections up
docs/devel: document some plugin assumptions
Pierrick Bouvier (5):
plugins: remove previous n_vcpus functions from API
plugins: add qemu_plugin_num_vcpus function
plugins: fix order of init/idle/resume callback
cpu: call plugin init hook asynchronously
docs/devel: plugins can trigger a tb flush
docs/devel/multi-thread-tcg.rst | 1 +
docs/devel/tcg-plugins.rst | 72 +++-
accel/tcg/plugin-helpers.h | 3 +-
include/exec/gdbstub.h | 43 ++-
include/hw/core/cpu.h | 18 +-
include/qemu/plugin.h | 14 +
include/qemu/qemu-plugin.h | 59 +++-
include/qemu/typedefs.h | 1 +
plugins/plugin.h | 6 +-
target/arm/cpu.h | 27 +-
target/arm/internals.h | 14 +-
target/hexagon/internal.h | 4 +-
target/microblaze/cpu.h | 4 +-
target/ppc/cpu-qom.h | 1 +
target/ppc/cpu.h | 5 +-
target/riscv/cpu.h | 5 +-
target/s390x/cpu.h | 2 -
tests/tcg/aarch64/semicall.h | 4 +-
tests/tcg/arm/semicall.h | 4 +-
tests/tcg/multiarch/float_helpers.h | 4 +-
tests/tcg/riscv64/semicall.h | 4 +-
accel/tcg/plugin-gen.c | 46 ++-
contrib/plugins/cache.c | 2 +-
contrib/plugins/execlog.c | 318 +++++++++++++++---
gdbstub/gdbstub.c | 169 ++++++----
hw/core/cpu-common.c | 19 +-
linux-user/nios2/cpu_loop.c | 1 +
plugins/api.c | 96 ++++--
plugins/core.c | 28 +-
target/arm/cpu.c | 2 -
target/arm/cpu64.c | 1 -
target/arm/gdbstub.c | 230 ++++++-------
target/arm/gdbstub64.c | 122 +++----
target/avr/cpu.c | 1 -
target/hexagon/cpu.c | 4 +-
target/hexagon/gdbstub.c | 10 +-
target/i386/cpu.c | 2 -
target/loongarch/cpu.c | 2 -
target/loongarch/gdbstub.c | 13 +-
target/m68k/cpu.c | 1 -
target/m68k/helper.c | 26 +-
target/microblaze/cpu.c | 6 +-
target/microblaze/gdbstub.c | 14 +-
target/ppc/cpu_init.c | 7 -
target/ppc/gdbstub.c | 114 ++++---
target/riscv/cpu.c | 15 -
target/riscv/gdbstub.c | 145 ++++----
target/rx/cpu.c | 1 -
target/s390x/cpu.c | 1 -
target/s390x/gdbstub.c | 105 +++---
tests/plugin/insn.c | 21 ++
.../multiarch/arm-compat-semi/semiconsole.c | 4 +-
.../multiarch/arm-compat-semi/semihosting.c | 4 +-
tests/tcg/multiarch/float_convd.c | 4 +-
tests/tcg/multiarch/float_convs.c | 4 +-
tests/tcg/multiarch/float_madds.c | 4 +-
tests/tcg/multiarch/libs/float_helpers.c | 4 +-
plugins/qemu-plugins.symbols | 5 +-
scripts/feature_to_c.py | 14 +-
tests/tcg/Makefile.target | 9 +-
tests/tcg/i386/system/boot.S | 6 +-
tests/tcg/x86_64/system/boot.S | 6 +-
tests/vm/Makefile.include | 2 +-
tests/vm/basevm.py | 4 +-
tests/vm/openbsd | 9 +-
65 files changed, 1235 insertions(+), 661 deletions(-)
--
2.39.2
next reply other threads:[~2024-02-27 14:44 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-27 14:43 Alex Bennée [this message]
2024-02-27 14:43 ` [PATCH v4 01/29] tests/tcg: update licenses to GPLv2 as intended Alex Bennée
2024-02-27 14:43 ` [PATCH v4 02/29] tests/tcg: bump TCG test timeout to 120s Alex Bennée
2024-02-27 14:43 ` [PATCH v4 03/29] tests/vm: avoid re-building the VM images all the time Alex Bennée
2024-02-27 17:14 ` Thomas Huth
2024-02-27 14:43 ` [PATCH v4 04/29] tests/vm: update openbsd image to 7.4 Alex Bennée
2024-02-27 17:35 ` Thomas Huth
2024-02-27 14:43 ` [PATCH v4 05/29] target/arm: Use GDBFeature for dynamic XML Alex Bennée
2024-02-27 14:43 ` [PATCH v4 06/29] target/ppc: " Alex Bennée
2024-02-27 14:43 ` [PATCH v4 07/29] target/riscv: " Alex Bennée
2024-02-27 14:43 ` [PATCH v4 08/29] gdbstub: Use GDBFeature for gdb_register_coprocessor Alex Bennée
2024-02-27 14:43 ` [PATCH v4 09/29] gdbstub: Use GDBFeature for GDBRegisterState Alex Bennée
2024-02-27 14:43 ` [PATCH v4 10/29] gdbstub: Change gdb_get_reg_cb and gdb_set_reg_cb Alex Bennée
2024-02-27 14:43 ` [PATCH v4 11/29] gdbstub: Simplify XML lookup Alex Bennée
2024-02-27 14:43 ` [PATCH v4 12/29] gdbstub: Infer number of core registers from XML Alex Bennée
2024-02-27 14:43 ` [PATCH v4 13/29] hw/core/cpu: Remove gdb_get_dynamic_xml member Alex Bennée
2024-02-27 14:43 ` [PATCH v4 14/29] gdbstub: Add members to identify registers to GDBFeature Alex Bennée
2024-02-27 14:43 ` [PATCH v4 15/29] plugins: remove previous n_vcpus functions from API Alex Bennée
2024-02-27 14:43 ` [PATCH v4 16/29] plugins: add qemu_plugin_num_vcpus function Alex Bennée
2024-02-27 14:43 ` [PATCH v4 17/29] plugins: fix order of init/idle/resume callback Alex Bennée
2024-02-27 14:43 ` [PATCH v4 18/29] linux-user: ensure nios2 processes queued work Alex Bennée
2024-02-27 14:43 ` [PATCH v4 19/29] cpu: call plugin init hook asynchronously Alex Bennée
2024-02-27 14:43 ` [PATCH v4 20/29] plugins: Use different helpers when reading registers Alex Bennée
2024-02-27 14:43 ` [PATCH v4 21/29] gdbstub: expose api to find registers Alex Bennée
2024-02-28 5:08 ` Akihiko Odaki
2024-02-27 14:43 ` [PATCH v4 22/29] plugins: create CPUPluginState and migrate plugin_mask Alex Bennée
2024-02-27 14:43 ` [PATCH v4 23/29] plugins: add an API to read registers Alex Bennée
2024-02-28 5:08 ` Akihiko Odaki
2024-02-27 14:43 ` [PATCH v4 24/29] tests/tcg: expand insn test case to exercise register API Alex Bennée
2024-02-27 14:43 ` [PATCH v4 25/29] contrib/plugins: fix imatch Alex Bennée
2024-02-27 14:43 ` [PATCH v4 26/29] contrib/plugins: extend execlog to track register changes Alex Bennée
2024-02-27 14:43 ` [PATCH v4 27/29] docs/devel: lift example and plugin API sections up Alex Bennée
2024-02-27 14:43 ` [PATCH v4 28/29] docs/devel: document some plugin assumptions Alex Bennée
2024-02-27 14:43 ` [PATCH v4 29/29] docs/devel: plugins can trigger a tb flush Alex Bennée
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=20240227144335.1196131-1-alex.bennee@linaro.org \
--to=alex.bennee@linaro.org \
--cc=alistair.francis@wdc.com \
--cc=bcain@quicinc.com \
--cc=bin.meng@windriver.com \
--cc=bleal@redhat.com \
--cc=brad@comstyle.com \
--cc=clg@kaod.org \
--cc=crosa@redhat.com \
--cc=danielhb413@gmail.com \
--cc=david@redhat.com \
--cc=dbarboza@ventanamicro.com \
--cc=edgar.iglesias@gmail.com \
--cc=eduardo@habkost.net \
--cc=erdnaxe@crans.org \
--cc=gaosong@loongson.cn \
--cc=iii@linux.ibm.com \
--cc=imp@bsdimp.com \
--cc=jsnow@redhat.com \
--cc=kevans@freebsd.org \
--cc=laurent@vivier.eu \
--cc=liwei1518@gmail.com \
--cc=ma.mandourr@gmail.com \
--cc=marcel.apfelbaum@gmail.com \
--cc=mrolnik@gmail.com \
--cc=npiggin@gmail.com \
--cc=palmer@dabbelt.com \
--cc=pbonzini@redhat.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-riscv@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=thuth@redhat.com \
--cc=wainersm@redhat.com \
--cc=wangyanan55@huawei.com \
--cc=ysato@users.sourceforge.jp \
--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 read-only IMAP folder(s) and NNTP newsgroup(s).