mbox series

[v3,00/12] gdbstub and TCG plugin improvements

Message ID 20230912224107.29669-1-akihiko.odaki@daynix.com
Headers show
Series gdbstub and TCG plugin improvements | expand

Message

Akihiko Odaki Sept. 12, 2023, 10:40 p.m. UTC
This series extracts fixes and refactorings that can be applied
independently from "[PATCH RESEND v5 00/26] plugins: Allow to read
registers" as suggested by Nicholas Piggin.

Patch "target/ppc: Remove references to gdb_has_xml" is also updated to
remove some dead code I missed earlier and thus the Reviewed-by tag is
dropped.

V2 -> V3:
  Added patch "plugins: Check if vCPU is realized".

V1 -> V2:
  Rebased.
  Added patch "gdbstub: Fix target_xml initialization".
  Added patch "gdbstub: Fix target.xml response".
  Added patch "gdbstub: Replace gdb_regs with an array".

Akihiko Odaki (12):
  gdbstub: Fix target_xml initialization
  gdbstub: Fix target.xml response
  plugins: Check if vCPU is realized
  contrib/plugins: Use GRWLock in execlog
  gdbstub: Introduce GDBFeature structure
  target/arm: Move the reference to arm-core.xml
  hw/core/cpu: Return static value with gdb_arch_name()
  gdbstub: Use g_markup_printf_escaped()
  target/arm: Remove references to gdb_has_xml
  target/ppc: Remove references to gdb_has_xml
  gdbstub: Remove gdb_has_xml variable
  gdbstub: Replace gdb_regs with an array

 MAINTAINERS               |  2 +-
 meson.build               |  2 +-
 gdbstub/internals.h       |  2 -
 include/exec/gdbstub.h    | 17 +++----
 include/hw/core/cpu.h     |  4 +-
 target/ppc/internal.h     |  2 +-
 contrib/plugins/execlog.c | 16 ++++---
 gdbstub/gdbstub.c         | 94 +++++++++++++++++++--------------------
 gdbstub/softmmu.c         |  2 +-
 plugins/core.c            |  2 +-
 stubs/gdbstub.c           |  6 +--
 target/arm/cpu.c          |  9 ++--
 target/arm/cpu64.c        |  4 +-
 target/arm/gdbstub.c      | 32 +------------
 target/i386/cpu.c         |  6 +--
 target/loongarch/cpu.c    |  8 ++--
 target/ppc/gdbstub.c      | 24 ++--------
 target/riscv/cpu.c        |  6 +--
 target/s390x/cpu.c        |  4 +-
 target/tricore/cpu.c      |  4 +-
 scripts/feature_to_c.py   | 48 ++++++++++++++++++++
 scripts/feature_to_c.sh   | 69 ----------------------------
 22 files changed, 146 insertions(+), 217 deletions(-)
 create mode 100755 scripts/feature_to_c.py
 delete mode 100644 scripts/feature_to_c.sh

Comments

Alex Bennée Sept. 14, 2023, 3:56 p.m. UTC | #1
Akihiko Odaki <akihiko.odaki@daynix.com> writes:

> This series extracts fixes and refactorings that can be applied
> independently from "[PATCH RESEND v5 00/26] plugins: Allow to read
> registers" as suggested by Nicholas Piggin.
>
> Patch "target/ppc: Remove references to gdb_has_xml" is also updated to
> remove some dead code I missed earlier and thus the Reviewed-by tag is
> dropped.

Queued to gdbstub/next, thanks.

>
> V2 -> V3:
>   Added patch "plugins: Check if vCPU is realized".
>
> V1 -> V2:
>   Rebased.
>   Added patch "gdbstub: Fix target_xml initialization".
>   Added patch "gdbstub: Fix target.xml response".
>   Added patch "gdbstub: Replace gdb_regs with an array".
>
> Akihiko Odaki (12):
>   gdbstub: Fix target_xml initialization
>   gdbstub: Fix target.xml response
>   plugins: Check if vCPU is realized
>   contrib/plugins: Use GRWLock in execlog
>   gdbstub: Introduce GDBFeature structure
>   target/arm: Move the reference to arm-core.xml
>   hw/core/cpu: Return static value with gdb_arch_name()
>   gdbstub: Use g_markup_printf_escaped()
>   target/arm: Remove references to gdb_has_xml
>   target/ppc: Remove references to gdb_has_xml
>   gdbstub: Remove gdb_has_xml variable
>   gdbstub: Replace gdb_regs with an array
>
>  MAINTAINERS               |  2 +-
>  meson.build               |  2 +-
>  gdbstub/internals.h       |  2 -
>  include/exec/gdbstub.h    | 17 +++----
>  include/hw/core/cpu.h     |  4 +-
>  target/ppc/internal.h     |  2 +-
>  contrib/plugins/execlog.c | 16 ++++---
>  gdbstub/gdbstub.c         | 94 +++++++++++++++++++--------------------
>  gdbstub/softmmu.c         |  2 +-
>  plugins/core.c            |  2 +-
>  stubs/gdbstub.c           |  6 +--
>  target/arm/cpu.c          |  9 ++--
>  target/arm/cpu64.c        |  4 +-
>  target/arm/gdbstub.c      | 32 +------------
>  target/i386/cpu.c         |  6 +--
>  target/loongarch/cpu.c    |  8 ++--
>  target/ppc/gdbstub.c      | 24 ++--------
>  target/riscv/cpu.c        |  6 +--
>  target/s390x/cpu.c        |  4 +-
>  target/tricore/cpu.c      |  4 +-
>  scripts/feature_to_c.py   | 48 ++++++++++++++++++++
>  scripts/feature_to_c.sh   | 69 ----------------------------
>  22 files changed, 146 insertions(+), 217 deletions(-)
>  create mode 100755 scripts/feature_to_c.py
>  delete mode 100644 scripts/feature_to_c.sh
Philippe Mathieu-Daudé Sept. 14, 2023, 5:18 p.m. UTC | #2
On 14/9/23 17:56, Alex Bennée wrote:
> 
> Akihiko Odaki <akihiko.odaki@daynix.com> writes:
> 
>> This series extracts fixes and refactorings that can be applied
>> independently from "[PATCH RESEND v5 00/26] plugins: Allow to read
>> registers" as suggested by Nicholas Piggin.
>>
>> Patch "target/ppc: Remove references to gdb_has_xml" is also updated to
>> remove some dead code I missed earlier and thus the Reviewed-by tag is
>> dropped.
> 
> Queued to gdbstub/next, thanks.

I left a comment in patch #3:
https://lore.kernel.org/qemu-devel/bf33447c-119f-c4b9-5f80-d4ad6169c708@linaro.org/

If you agree with the comment I can send a patch to replace so
you can keep this series queued.

Regards,

Phil.