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

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

Philippe Mathieu-Daudé posted 2 patches 1 month ago

Diff against v2 v3 v4 v5
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260107182019.51769-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>, Zhao Liu <zhao1.liu@intel.com>, Marcelo Tosatti <mtosatti@redhat.com>, Laurent Vivier <laurent@vivier.eu>, 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>, Max Filippov <jcmvbkbc@gmail.com>

There is a newer version of this series

include/monitor/hmp-target.h  | 28 +++++-----------------------
include/monitor/hmp.h         | 23 +++++++++++++++++++++++
target/i386/sev.h             |  2 ++
hw/i386/sgx-stub.c            |  2 +-
hw/i386/sgx.c                 |  1 -
monitor/hmp-cmds.c            |  1 -
stubs/target-monitor-defs.c   |  2 +-
target/i386/cpu-apic.c        |  2 +-
target/i386/sev-system-stub.c |  2 +-
target/i386/sev.c             |  1 -
target/m68k/monitor.c         |  1 +
target/riscv/monitor.c        |  1 +
target/sh4/monitor.c          |  1 -
target/xtensa/monitor.c       |  1 -
14 files changed, 36 insertions(+), 32 deletions(-)

Expand all Fold all

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

Posted by Philippe Mathieu-Daudé 1 month ago

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

Philippe Mathieu-Daudé (2):
  target/i386: Include missing 'svm.h' header in 'sev.h'
  monitor/hmp: Reduce target-specific definitions

 include/monitor/hmp-target.h  | 28 +++++-----------------------
 include/monitor/hmp.h         | 23 +++++++++++++++++++++++
 target/i386/sev.h             |  2 ++
 hw/i386/sgx-stub.c            |  2 +-
 hw/i386/sgx.c                 |  1 -
 monitor/hmp-cmds.c            |  1 -
 stubs/target-monitor-defs.c   |  2 +-
 target/i386/cpu-apic.c        |  2 +-
 target/i386/sev-system-stub.c |  2 +-
 target/i386/sev.c             |  1 -
 target/m68k/monitor.c         |  1 +
 target/riscv/monitor.c        |  1 +
 target/sh4/monitor.c          |  1 -
 target/xtensa/monitor.c       |  1 -
 14 files changed, 36 insertions(+), 32 deletions(-)

-- 
2.52.0

Re: [PATCH 0/2] monitor/hmp: Reduce target-specific definitions

Posted by Pierrick Bouvier 1 month ago

On 1/7/26 10:20 AM, Philippe Mathieu-Daudé wrote:
> Move most of target-agnostic definitions from
> "monitor/hmp-target.h" to "monitor/hmp.h" to
> reduce files target-poisoning.
> 
> Philippe Mathieu-Daudé (2):
>    target/i386: Include missing 'svm.h' header in 'sev.h'
>    monitor/hmp: Reduce target-specific definitions
> 
>   include/monitor/hmp-target.h  | 28 +++++-----------------------
>   include/monitor/hmp.h         | 23 +++++++++++++++++++++++
>   target/i386/sev.h             |  2 ++
>   hw/i386/sgx-stub.c            |  2 +-
>   hw/i386/sgx.c                 |  1 -
>   monitor/hmp-cmds.c            |  1 -
>   stubs/target-monitor-defs.c   |  2 +-
>   target/i386/cpu-apic.c        |  2 +-
>   target/i386/sev-system-stub.c |  2 +-
>   target/i386/sev.c             |  1 -
>   target/m68k/monitor.c         |  1 +
>   target/riscv/monitor.c        |  1 +
>   target/sh4/monitor.c          |  1 -
>   target/xtensa/monitor.c       |  1 -
>   14 files changed, 36 insertions(+), 32 deletions(-)
> 

(in private),
build issues:
https://github.com/pbo-linaro/qemu-ci/actions/runs/20792272184/job/59717050408