[v1] monitor: Reduce headers included in 'monitor/monitor.h'

[PATCH 00/14] monitor: Reduce headers included in 'monitor/monitor.h'

Philippe Mathieu-Daudé posted 14 patches 1 week, 6 days ago

Diff against v2 v3
Download series mbox

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

Maintainers: Brian Cain <brian.cain@oss.qualcomm.com>, Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>, "Dr. David Alan Gilbert" <dave@treblig.org>, Markus Armbruster <armbru@redhat.com>, Michael Roth <michael.roth@amd.com>, Peter Xu <peterx@redhat.com>, Fabiano Rosas <farosas@suse.de>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, "Philippe Mathieu-Daudé" <philmd@mailo.com>, Alistair Francis <alistair.francis@wdc.com>, David Gibson <david@gibson.dropbear.id.au>, Hyman Huang <infra.ai.cloud@bitdeer.com>

There is a newer version of this series

include/hw/hexagon/hexagon_tlb.h |   2 +-
include/monitor/hmp.h            |   1 -
include/monitor/monitor.h        |   3 +-
include/qapi/qmp-registry.h      |   1 -
monitor/monitor-internal.h       |   1 +
target/hexagon/hex_mmu.h         |   3 +-
migration/migration-hmp-cmds.c   |   1 +
monitor/fds.c                    |   1 +
monitor/monitor.c                |   2 +
monitor/qmp.c                    |   3 +-
qapi/qmp-dispatch.c              |   2 +
system/cpus.c                    | 146 ------------------------------
system/device_tree.c             |   1 -
system/dirtylimit-hmp-cmds.c     |  74 ++++++++++++++++
system/dirtylimit.c              |  60 -------------
system/physmem.c                 |   2 -
system/runstate.c                | 148 +++++++++++++++++++++++++++++++
tests/unit/test-util-sockets.c   |   1 +
system/meson.build               |   1 +
19 files changed, 237 insertions(+), 216 deletions(-)
create mode 100644 system/dirtylimit-hmp-cmds.c

Expand all Fold all

[PATCH 00/14] monitor: Reduce headers included in 'monitor/monitor.h'

Posted by Philippe Mathieu-Daudé 1 week, 6 days ago

Another HMP cleanup series to help [1] Marc-André with his
'Make HMP optional' series [2].

We could also remove "exec/hwaddr.h" by moving gpa2hva()
declaration elsewhere, and "qapi/qapi-types-misc.h" by
moving monitor_fdset_add_fd() out to some "monitor/fdset.h"
header.

[1] I expect his work to trivially rebase ;)
[2] 20260626-qemu-no-hmp-v2-0-8af31bc54c61@redhat.com
Based-on: <20260812121232.71958-1-philmd@oss.qualcomm.com>

Philippe Mathieu-Daudé (14):
  hexagon: Remove unnecessary 'monitor/monitor.h' header
  tests/unit: Include 'qemu/main-loop.h' header in test-util-sockets.c
  qapi/qmp-dispatch: Include 'qemu/aio-wait.h' and 'monitor/monitor.h'
  qapi/qmp-registry: Remove unnecessary 'monitor/monitor.h' header
  migration/hmp-cmds: Include 'block/block-global-state.h' header
  monitor: Include missing 'qemu/aio-wait.h' header
  monitor: Include missing 'qemu/lockable.h' header
  monitor: Include missing 'qemu/coroutine-core.h' header
  monitor: Reduce inclusion of 'qapi/qapi-emit-events.h' header
  monitor: Remove unnecessary 'block/block.h' header
  monitor/hmp: Remove unnecessary 'monitor/monitor.h' header
  system: Remove unnecessary 'monitor/monitor.h' header
  system: Move runstate-related code from cpus.c to runstate.c
  system/dirtylimit: Extract HMP code to dirtylimit-hmp-cmds.c

 include/hw/hexagon/hexagon_tlb.h |   2 +-
 include/monitor/hmp.h            |   1 -
 include/monitor/monitor.h        |   3 +-
 include/qapi/qmp-registry.h      |   1 -
 monitor/monitor-internal.h       |   1 +
 target/hexagon/hex_mmu.h         |   3 +-
 migration/migration-hmp-cmds.c   |   1 +
 monitor/fds.c                    |   1 +
 monitor/monitor.c                |   2 +
 monitor/qmp.c                    |   3 +-
 qapi/qmp-dispatch.c              |   2 +
 system/cpus.c                    | 146 ------------------------------
 system/device_tree.c             |   1 -
 system/dirtylimit-hmp-cmds.c     |  74 ++++++++++++++++
 system/dirtylimit.c              |  60 -------------
 system/physmem.c                 |   2 -
 system/runstate.c                | 148 +++++++++++++++++++++++++++++++
 tests/unit/test-util-sockets.c   |   1 +
 system/meson.build               |   1 +
 19 files changed, 237 insertions(+), 216 deletions(-)
 create mode 100644 system/dirtylimit-hmp-cmds.c

-- 
2.53.0