[v3] monitor/hmp: Reduce target-specific definitions

[PATCH v3 0/8] monitor/hmp: Reduce target-specific definitions

Philippe Mathieu-Daudé posted 8 patches 2 weeks, 4 days ago

Diff against v1 v2 v4 v5
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260119110358.66821-1-philmd@linaro.org

Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, Eduardo Habkost <eduardo@habkost.net>, "Dr. David Alan Gilbert" <dave@treblig.org>, Markus Armbruster <armbru@redhat.com>, Zhao Liu <zhao1.liu@intel.com>, Marcelo Tosatti <mtosatti@redhat.com>, Laurent Vivier <laurent@vivier.eu>, Nicholas Piggin <npiggin@gmail.com>, Chinmay Rath <rathc@linux.ibm.com>, Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <alistair.francis@wdc.com>, Weiwei Li <liwei1518@gmail.com>, Daniel Henrique Barboza <dbarboza@ventanamicro.com>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, Yoshinori Sato <yoshinori.sato@nifty.com>, Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Artyom Tarasenko <atar4qemu@gmail.com>, Max Filippov <jcmvbkbc@gmail.com>

There is a newer version of this series

MAINTAINERS                   |   2 +-
include/monitor/hmp-target.h  |  64 -------------------
include/monitor/hmp.h         |  31 +++++++++
monitor/monitor-internal.h    |  10 ++-
target/i386/sev.h             |   2 +
hw/i386/sgx-stub.c            |   2 +-
hw/i386/sgx.c                 |   1 -
monitor/hmp-cmds.c            |   1 -
monitor/hmp-target.c          | 108 +------------------------------
monitor/hmp.c                 | 116 ++++++++++++++++++++++++++++++++--
stubs/target-monitor-defs.c   |   2 +-
target/i386/cpu-apic.c        |   2 +-
target/i386/monitor.c         |   9 ++-
target/i386/sev-system-stub.c |   2 +-
target/i386/sev.c             |   1 -
target/m68k/monitor.c         |  62 +++++++++---------
target/ppc/ppc-qmp-cmds.c     |  26 ++++----
target/riscv/monitor.c        |   2 +-
target/riscv/riscv-qmp-cmds.c |   1 -
target/sh4/monitor.c          |   1 -
target/sparc/monitor.c        |  11 ++--
target/xtensa/monitor.c       |   1 -
22 files changed, 212 insertions(+), 245 deletions(-)
delete mode 100644 include/monitor/hmp-target.h

Expand all Fold all

[PATCH v3 0/8] monitor/hmp: Reduce target-specific definitions

Posted by Philippe Mathieu-Daudé 2 weeks, 4 days ago

Missing review: #6

v3: Rename MD_I32 -> MD_U32 (Dave)
v2: Change get_value() prototype signature, use ldn_he_p()

Move most of target-agnostic definitions from
"monitor/hmp-target.h" to "monitor/hmp.h" to
reduce files target-poisoning.

Philippe Mathieu-Daudé (8):
  target/i386: Include missing 'svm.h' header in 'sev.h'
  monitor: Add hmp_cmds_for_target() helper
  monitor: Reduce target-specific methods
  monitor: Have MonitorDef::get_value() return an unsigned type
  monitor: Have *get_monitor_def() fill an unsigned value
  monitor: Truncate target register using ldn_he_p() API
  monitor: Reduce target-specific methods further
  monitor: Remove 'monitor/hmp-target.h' header

 MAINTAINERS                   |   2 +-
 include/monitor/hmp-target.h  |  64 -------------------
 include/monitor/hmp.h         |  31 +++++++++
 monitor/monitor-internal.h    |  10 ++-
 target/i386/sev.h             |   2 +
 hw/i386/sgx-stub.c            |   2 +-
 hw/i386/sgx.c                 |   1 -
 monitor/hmp-cmds.c            |   1 -
 monitor/hmp-target.c          | 108 +------------------------------
 monitor/hmp.c                 | 116 ++++++++++++++++++++++++++++++++--
 stubs/target-monitor-defs.c   |   2 +-
 target/i386/cpu-apic.c        |   2 +-
 target/i386/monitor.c         |   9 ++-
 target/i386/sev-system-stub.c |   2 +-
 target/i386/sev.c             |   1 -
 target/m68k/monitor.c         |  62 +++++++++---------
 target/ppc/ppc-qmp-cmds.c     |  26 ++++----
 target/riscv/monitor.c        |   2 +-
 target/riscv/riscv-qmp-cmds.c |   1 -
 target/sh4/monitor.c          |   1 -
 target/sparc/monitor.c        |  11 ++--
 target/xtensa/monitor.c       |   1 -
 22 files changed, 212 insertions(+), 245 deletions(-)
 delete mode 100644 include/monitor/hmp-target.h

-- 
2.52.0