1. Patchew
  2. QEMU
  3. View series

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

Akihiko Odaki posted 12 patches 7 months, 2 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230912224107.29669-1-akihiko.odaki@daynix.com
Maintainers: "Alex Bennée" <alex.bennee@linaro.org>, Alexandre Iooss <erdnaxe@crans.org>, Mahmoud Mandour <ma.mandourr@gmail.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Eduardo Habkost <eduardo@habkost.net>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Yanan Wang <wangyanan55@huawei.com>, Paolo Bonzini <pbonzini@redhat.com>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, "Daniel P. Berrangé" <berrange@redhat.com>, Thomas Huth <thuth@redhat.com>, John Snow <jsnow@redhat.com>, Cleber Rosa <crosa@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Song Gao <gaosong@loongson.cn>, Xiaojuan Yang <yangxiaojuan@loongson.cn>, Daniel Henrique Barboza <danielhb413@gmail.com>, "Cédric Le Goater" <clg@kaod.org>, David Gibson <david@gibson.dropbear.id.au>, Greg Kurz <groug@kaod.org>, Nicholas Piggin <npiggin@gmail.com>, Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <alistair.francis@wdc.com>, Bin Meng <bin.meng@windriver.com>, Weiwei Li <liweiwei@iscas.ac.cn>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, Richard Henderson <richard.henderson@linaro.org>, David Hildenbrand <david@redhat.com>, Ilya Leoshkevich <iii@linux.ibm.com>, Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
There is a newer version of this series
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
[PATCH v3 00/12] gdbstub and TCG plugin improvements
Posted by Akihiko Odaki 7 months, 2 weeks ago 
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

-- 
2.42.0
Re: [PATCH v3 00/12] gdbstub and TCG plugin improvements
Posted by Alex Bennée 7 months, 2 weeks ago 
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


-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro
Re: [PATCH v3 00/12] gdbstub and TCG plugin improvements
Posted by Philippe Mathieu-Daudé 7 months, 2 weeks ago 
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.

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.