qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3 00/24] gdbstub: re-organise to for better compilation behaviour
@ 2023-02-21 22:52 Alex Bennée
  2023-02-21 22:52 ` [PATCH v3 01/24] gdbstub/internals.h: clean up include guard Alex Bennée
                   ` (23 more replies)
  0 siblings, 24 replies; 32+ messages in thread
From: Alex Bennée @ 2023-02-21 22:52 UTC (permalink / raw)
  To: qemu-devel
  Cc: Thomas Huth, qemu-arm, qemu-riscv, qemu-s390x, Yoshinori Sato,
	nicolas.eder, Stafford Horne, Daniel Henrique Barboza,
	Mahmoud Mandour, Taylor Simpson, mads, Marek Vasut,
	Artyom Tarasenko, Alistair Francis, qemu-ppc, Yanan Wang,
	Sunil Muthuswamy, Cédric Le Goater, Eduardo Habkost,
	Bin Meng, Bastian Koppelmann, Palmer Dabbelt, Marcel Apfelbaum,
	Laurent Vivier, Philippe Mathieu-Daudé, Michael Rolnik,
	Aleksandar Rikalo, Paolo Bonzini, Alex Bennée,
	Aurelien Jarno, Xiaojuan Yang, Richard Henderson, Peter Maydell,
	Alexandre Iooss, Chris Wulff, Jiaxun Yang, David Gibson,
	Edgar E. Iglesias, Greg Kurz, David Hildenbrand, Song Gao,
	Ilya Leoshkevich, Max Filippov, Mark Cave-Ayland

I was motivated to sort this out while working on my register API
which is target agnostic but ran into the weeds when trying to link up
with the gdbstub. This was due to us building gdbstub for every single
target we support due to a few ABI sensitive bits that require CPU
specific information. This series does a bunch of surgery to break the
monolithic file apart into its constituent parts as well as simplify
the headers to users can avoid bringing in more dependencies than they
need.

I had hoped to go all the way and conditionally compile syscalls only
for the two ABIs (32 and 64 bit) unfortunately I was unable to the
appropriate meson-foo to make that happen.

Other than a minor re-org of the patches and addressing review
comments (see bellow ---) the only new patches are the ones to split
target_ulong into its own header and remove the remaining cpu-def
requirements from syscall handling.

The following patches need review:

gdbstub: split out softmmu/user specifics for syscall handling
include: split target_long definition from cpu-defs
testing: probe gdb for supported architectures ahead of time
gdbstub: only compile gdbstub twice for whole build
gdbstub: rationalise signal mapping in softmmu

Alex Bennée (23):
  gdbstub/internals.h: clean up include guard
  gdbstub: fix-up copyright and license files
  gdbstub: define separate user/system structures
  gdbstub: move GDBState to shared internals header
  includes: move tb_flush into its own header
  gdbstub: move fromhex/tohex routines to internals
  gdbstub: make various helpers visible to the rest of the module
  gdbstub: move chunk of softmmu functionality to own file
  gdbstub: move chunks of user code into own files
  gdbstub: rationalise signal mapping in softmmu
  gdbstub: abstract target specific details from gdb_put_packet_binary
  gdbstub: specialise handle_query_attached
  gdbstub: specialise target_memory_rw_debug
  gdbstub: introduce gdb_get_max_cpus
  gdbstub: specialise stub_can_reverse
  gdbstub: fix address type of gdb_set_cpu_pc
  gdbstub: don't use target_ulong while handling registers
  gdbstub: move register helpers into standalone include
  gdbstub: move syscall handling to new file
  gdbstub: only compile gdbstub twice for whole build
  testing: probe gdb for supported architectures ahead of time
  include: split target_long definition from cpu-defs
  gdbstub: split out softmmu/user specifics for syscall handling

Philippe Mathieu-Daudé (1):
  gdbstub: Make syscall_complete/[gs]et_reg target-agnostic typedefs

 MAINTAINERS                                   |    1 +
 configure                                     |    8 +
 gdbstub/internals.h                           |  214 ++-
 include/exec/cpu-defs.h                       |   19 +-
 include/exec/exec-all.h                       |    1 -
 include/exec/gdbstub.h                        |  208 ---
 include/exec/target_long.h                    |   42 +
 include/exec/tb-flush.h                       |   26 +
 include/gdbstub/helpers.h                     |  103 +
 include/gdbstub/syscalls.h                    |  124 ++
 include/gdbstub/user.h                        |   43 +
 linux-user/user-internals.h                   |    1 +
 accel/stubs/tcg-stub.c                        |    1 +
 accel/tcg/tb-maint.c                          |    1 +
 accel/tcg/translate-all.c                     |    1 +
 cpu.c                                         |    1 +
 gdbstub/gdbstub.c                             | 1654 ++---------------
 gdbstub/softmmu.c                             |  605 +++++-
 gdbstub/syscalls.c                            |  217 +++
 gdbstub/user-target.c                         |  281 +++
 gdbstub/user.c                                |  427 ++++-
 hw/ppc/spapr_hcall.c                          |    1 +
 linux-user/exit.c                             |    2 +-
 linux-user/main.c                             |    1 +
 linux-user/signal.c                           |    2 +-
 plugins/core.c                                |    1 +
 plugins/loader.c                              |    2 +-
 semihosting/arm-compat-semi.c                 |    1 +
 semihosting/guestfd.c                         |    2 +-
 semihosting/syscalls.c                        |    3 +-
 softmmu/runstate.c                            |    2 +-
 target/alpha/gdbstub.c                        |    2 +-
 target/alpha/sys_helper.c                     |    1 +
 target/arm/gdbstub.c                          |    1 +
 target/arm/gdbstub64.c                        |    2 +-
 target/arm/helper-a64.c                       |    2 +-
 target/arm/m_helper.c                         |    1 +
 target/avr/gdbstub.c                          |    2 +-
 target/cris/gdbstub.c                         |    2 +-
 target/hexagon/gdbstub.c                      |    2 +-
 target/hppa/gdbstub.c                         |    2 +-
 target/i386/gdbstub.c                         |    2 +-
 target/i386/whpx/whpx-all.c                   |    2 +-
 target/loongarch/gdbstub.c                    |    1 +
 target/m68k/gdbstub.c                         |    2 +-
 target/m68k/helper.c                          |    1 +
 target/m68k/m68k-semi.c                       |    3 +-
 target/microblaze/gdbstub.c                   |    2 +-
 target/mips/gdbstub.c                         |    2 +-
 target/mips/tcg/sysemu/mips-semi.c            |    3 +-
 target/nios2/cpu.c                            |    2 +-
 target/nios2/nios2-semi.c                     |    3 +-
 target/openrisc/gdbstub.c                     |    2 +-
 target/openrisc/interrupt.c                   |    2 +-
 target/openrisc/mmu.c                         |    2 +-
 target/ppc/cpu_init.c                         |    2 +-
 target/ppc/gdbstub.c                          |    1 +
 target/riscv/csr.c                            |    1 +
 target/riscv/gdbstub.c                        |    1 +
 target/rx/gdbstub.c                           |    2 +-
 target/s390x/gdbstub.c                        |    1 +
 target/s390x/helper.c                         |    2 +-
 target/sh4/gdbstub.c                          |    2 +-
 target/sparc/gdbstub.c                        |    2 +-
 target/tricore/gdbstub.c                      |    2 +-
 target/xtensa/core-dc232b.c                   |    2 +-
 target/xtensa/core-dc233c.c                   |    2 +-
 target/xtensa/core-de212.c                    |    2 +-
 target/xtensa/core-de233_fpu.c                |    2 +-
 target/xtensa/core-dsp3400.c                  |    2 +-
 target/xtensa/core-fsf.c                      |    2 +-
 target/xtensa/core-lx106.c                    |    2 +-
 target/xtensa/core-sample_controller.c        |    2 +-
 target/xtensa/core-test_kc705_be.c            |    2 +-
 target/xtensa/core-test_mmuhifi_c3.c          |    2 +-
 target/xtensa/gdbstub.c                       |    2 +-
 target/xtensa/helper.c                        |    2 +-
 gdbstub/meson.build                           |   33 +-
 gdbstub/trace-events                          |    4 +-
 scripts/probe-gdb-support.sh                  |   36 +
 target/xtensa/import_core.sh                  |    2 +-
 tests/tcg/aarch64/Makefile.target             |    2 +-
 tests/tcg/multiarch/Makefile.target           |    5 +
 .../multiarch/system/Makefile.softmmu-target  |    6 +-
 tests/tcg/s390x/Makefile.target               |    2 +-
 85 files changed, 2375 insertions(+), 1796 deletions(-)
 create mode 100644 include/exec/target_long.h
 create mode 100644 include/exec/tb-flush.h
 create mode 100644 include/gdbstub/helpers.h
 create mode 100644 include/gdbstub/syscalls.h
 create mode 100644 include/gdbstub/user.h
 create mode 100644 gdbstub/syscalls.c
 create mode 100644 gdbstub/user-target.c
 create mode 100755 scripts/probe-gdb-support.sh

-- 
2.39.1



^ permalink raw reply	[flat|nested] 32+ messages in thread

end of thread, other threads:[~2023-02-22  7:19 UTC | newest]

Thread overview: 32+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-02-21 22:52 [PATCH v3 00/24] gdbstub: re-organise to for better compilation behaviour Alex Bennée
2023-02-21 22:52 ` [PATCH v3 01/24] gdbstub/internals.h: clean up include guard Alex Bennée
2023-02-21 22:52 ` [PATCH v3 02/24] gdbstub: fix-up copyright and license files Alex Bennée
2023-02-21 22:52 ` [PATCH v3 03/24] gdbstub: Make syscall_complete/[gs]et_reg target-agnostic typedefs Alex Bennée
2023-02-21 22:52 ` [PATCH v3 04/24] gdbstub: define separate user/system structures Alex Bennée
2023-02-21 22:52 ` [PATCH v3 05/24] gdbstub: move GDBState to shared internals header Alex Bennée
2023-02-21 22:52 ` [PATCH v3 06/24] includes: move tb_flush into its own header Alex Bennée
2023-02-21 22:52 ` [PATCH v3 07/24] gdbstub: move fromhex/tohex routines to internals Alex Bennée
2023-02-21 22:52 ` [PATCH v3 08/24] gdbstub: make various helpers visible to the rest of the module Alex Bennée
2023-02-21 22:52 ` [PATCH v3 09/24] gdbstub: move chunk of softmmu functionality to own file Alex Bennée
2023-02-22  0:26   ` Richard Henderson
2023-02-21 22:52 ` [PATCH v3 10/24] gdbstub: move chunks of user code into own files Alex Bennée
2023-02-22  7:16   ` Richard Henderson
2023-02-21 22:52 ` [PATCH v3 11/24] gdbstub: rationalise signal mapping in softmmu Alex Bennée
2023-02-22  0:32   ` Richard Henderson
2023-02-21 22:52 ` [PATCH v3 12/24] gdbstub: abstract target specific details from gdb_put_packet_binary Alex Bennée
2023-02-21 22:52 ` [PATCH v3 13/24] gdbstub: specialise handle_query_attached Alex Bennée
2023-02-21 22:52 ` [PATCH v3 14/24] gdbstub: specialise target_memory_rw_debug Alex Bennée
2023-02-21 22:52 ` [PATCH v3 15/24] gdbstub: introduce gdb_get_max_cpus Alex Bennée
2023-02-21 22:52 ` [PATCH v3 16/24] gdbstub: specialise stub_can_reverse Alex Bennée
2023-02-21 22:52 ` [PATCH v3 17/24] gdbstub: fix address type of gdb_set_cpu_pc Alex Bennée
2023-02-21 22:52 ` [PATCH v3 18/24] gdbstub: don't use target_ulong while handling registers Alex Bennée
2023-02-22  0:51   ` Richard Henderson
2023-02-21 22:52 ` [PATCH v3 19/24] gdbstub: move register helpers into standalone include Alex Bennée
2023-02-21 22:52 ` [PATCH v3 20/24] gdbstub: move syscall handling to new file Alex Bennée
2023-02-21 22:52 ` [PATCH v3 21/24] gdbstub: only compile gdbstub twice for whole build Alex Bennée
2023-02-22  7:18   ` Richard Henderson
2023-02-21 22:52 ` [PATCH v3 22/24] testing: probe gdb for supported architectures ahead of time Alex Bennée
2023-02-22  1:10   ` Richard Henderson
2023-02-21 22:52 ` [PATCH v3 23/24] include: split target_long definition from cpu-defs Alex Bennée
2023-02-22  1:17   ` Richard Henderson
2023-02-21 22:52 ` [PATCH v3 24/24] gdbstub: split out softmmu/user specifics for syscall handling Alex Bennée

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).