|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH v3 30/49] monitor: isolate HMP declarations in hmp.h
* Marc-André Lureau (marcandre.lureau@xxxxxxxxxx) wrote:
> Also rename password & commands with hmp in the name, while at it.
> Other functions need larger changes which we will take care of next.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
Reviewed-by: Dr. David Alan Gilbert <dave@xxxxxxxxxxx>
> ---
> accel/accel-system.c | 1 +
> accel/tcg/monitor.c | 1 +
> chardev/char.c | 2 +-
> disas/disas-mon.c | 1 +
> gdbstub/system.c | 2 +-
> hw/char/virtio-serial-bus.c | 1 +
> hw/core/machine-hmp-cmds.c | 1 -
> hw/core/sysbus.c | 1 +
> hw/hexagon/hexagon_tlb.c | 1 +
> hw/misc/auxbus.c | 1 +
> hw/usb/bus.c | 1 +
> hw/usb/host-libusb.c | 1 +
> hw/xen/xen-bus.c | 1 +
> include/monitor/hmp.h | 21 +++++++++++++++++++++
> include/monitor/monitor.h | 18 ------------------
> monitor/hmp.c | 8 ++++----
> monitor/monitor-internal.h | 1 +
> net/slirp.c | 1 +
> stubs/monitor-core.c | 1 +
> stubs/monitor-internal.c | 2 +-
> target/rx/disas.c | 1 +
> tests/unit/test-util-sockets.c | 1 +
> tools/qemu-vnc/stubs.c | 1 +
> trace/trace-hmp-cmds.c | 1 -
> ui/ui-hmp-cmds.c | 4 ++--
> util/error-report.c | 2 +-
> util/qemu-print.c | 1 +
> 27 files changed, 48 insertions(+), 30 deletions(-)
>
> diff --git a/accel/accel-system.c b/accel/accel-system.c
> index 9176665202d2..977804c4048a 100644
> --- a/accel/accel-system.c
> +++ b/accel/accel-system.c
> @@ -28,6 +28,7 @@
> #include "qom/compat-properties.h"
> #include "qapi/qapi-commands-accelerator.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "hw/core/boards.h"
> #include "hw/core/cpu.h"
> #include "accel/accel-ops.h"
> diff --git a/accel/tcg/monitor.c b/accel/tcg/monitor.c
> index be5c1950177c..74170ddef708 100644
> --- a/accel/tcg/monitor.c
> +++ b/accel/tcg/monitor.c
> @@ -11,6 +11,7 @@
> #include "qapi/type-helpers.h"
> #include "qapi/qapi-commands-machine.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "system/tcg.h"
> #include "tcg/tcg.h"
> #include "internal-common.h"
> diff --git a/chardev/char.c b/chardev/char.c
> index c6c8133f5c1d..9da0911e503c 100644
> --- a/chardev/char.c
> +++ b/chardev/char.c
> @@ -24,7 +24,7 @@
>
> #include "qemu/osdep.h"
> #include "qemu/cutils.h"
> -#include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "monitor/qmp-helpers.h"
> #include "qemu/config-file.h"
> #include "qemu/error-report.h"
> diff --git a/disas/disas-mon.c b/disas/disas-mon.c
> index 9c693618c277..bc9dec3a7761 100644
> --- a/disas/disas-mon.c
> +++ b/disas/disas-mon.c
> @@ -10,6 +10,7 @@
> #include "system/memory.h"
> #include "hw/core/cpu.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
>
> /*
> * Get LENGTH bytes from info's buffer, at target address memaddr.
> diff --git a/gdbstub/system.c b/gdbstub/system.c
> index 070bc26f416c..8a1cdb11db36 100644
> --- a/gdbstub/system.c
> +++ b/gdbstub/system.c
> @@ -29,7 +29,7 @@
> #include "hw/core/boards.h"
> #include "chardev/char.h"
> #include "chardev/char-fe.h"
> -#include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "trace.h"
> #include "internals.h"
>
> diff --git a/hw/char/virtio-serial-bus.c b/hw/char/virtio-serial-bus.c
> index c1973f0248fc..02604740f86a 100644
> --- a/hw/char/virtio-serial-bus.c
> +++ b/hw/char/virtio-serial-bus.c
> @@ -25,6 +25,7 @@
> #include "qemu/module.h"
> #include "migration/qemu-file-types.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qemu/error-report.h"
> #include "qemu/queue.h"
> #include "hw/core/qdev-properties.h"
> diff --git a/hw/core/machine-hmp-cmds.c b/hw/core/machine-hmp-cmds.c
> index 686304bafab5..1c700aad3587 100644
> --- a/hw/core/machine-hmp-cmds.c
> +++ b/hw/core/machine-hmp-cmds.c
> @@ -15,7 +15,6 @@
>
> #include "qemu/osdep.h"
> #include "monitor/hmp.h"
> -#include "monitor/monitor.h"
> #include "qapi/error.h"
> #include "qapi/qapi-builtin-visit.h"
> #include "qapi/qapi-commands-accelerator.h"
> diff --git a/hw/core/sysbus.c b/hw/core/sysbus.c
> index 3e1160ee921d..13df7cbafe10 100644
> --- a/hw/core/sysbus.c
> +++ b/hw/core/sysbus.c
> @@ -21,6 +21,7 @@
> #include "qapi/error.h"
> #include "hw/core/sysbus.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "system/address-spaces.h"
>
> static void sysbus_dev_print(Monitor *mon, DeviceState *dev, int indent);
> diff --git a/hw/hexagon/hexagon_tlb.c b/hw/hexagon/hexagon_tlb.c
> index b6d4aff389e5..2d878cee736d 100644
> --- a/hw/hexagon/hexagon_tlb.c
> +++ b/hw/hexagon/hexagon_tlb.c
> @@ -12,6 +12,7 @@
> #include "hw/core/resettable.h"
> #include "migration/vmstate.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qapi/error.h"
> #include "exec/page-protection.h"
> #include "exec/target_page.h"
> diff --git a/hw/misc/auxbus.c b/hw/misc/auxbus.c
> index 877f34560626..ac2525b90fec 100644
> --- a/hw/misc/auxbus.c
> +++ b/hw/misc/auxbus.c
> @@ -33,6 +33,7 @@
> #include "hw/misc/auxbus.h"
> #include "hw/i2c/i2c.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qapi/error.h"
>
> #ifndef DEBUG_AUX
> diff --git a/hw/usb/bus.c b/hw/usb/bus.c
> index 3b6fbd46ac3f..9b9b2e7c2f8f 100644
> --- a/hw/usb/bus.c
> +++ b/hw/usb/bus.c
> @@ -9,6 +9,7 @@
> #include "system/system.h"
> #include "migration/vmstate.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "trace.h"
> #include "qemu/cutils.h"
>
> diff --git a/hw/usb/host-libusb.c b/hw/usb/host-libusb.c
> index b9f3ad3f66dd..af67d5dfeb10 100644
> --- a/hw/usb/host-libusb.c
> +++ b/hw/usb/host-libusb.c
> @@ -48,6 +48,7 @@
> #include "qapi/error.h"
> #include "migration/vmstate.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qemu/error-report.h"
> #include "qemu/main-loop.h"
> #include "qemu/module.h"
> diff --git a/hw/xen/xen-bus.c b/hw/xen/xen-bus.c
> index dfad2bc5085f..a563f6066bb4 100644
> --- a/hw/xen/xen-bus.c
> +++ b/hw/xen/xen-bus.c
> @@ -17,6 +17,7 @@
> #include "hw/xen/xen-bus.h"
> #include "hw/xen/xen-bus-helper.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qapi/error.h"
> #include "qobject/qdict.h"
> #include "system/system.h"
> diff --git a/include/monitor/hmp.h b/include/monitor/hmp.h
> index 9258a049bffb..166cd4100c63 100644
> --- a/include/monitor/hmp.h
> +++ b/include/monitor/hmp.h
> @@ -18,6 +18,9 @@
> #include "qapi/qapi-types-common.h"
> #include "monitor/monitor.h"
>
> +#define TYPE_MONITOR_HMP "monitor-hmp"
> +OBJECT_DECLARE_TYPE(MonitorHMP, MonitorHMPClass, MONITOR_HMP);
> +
> #define HMP_STUB(cmd) \
> void hmp_##cmd(Monitor *mon, const QDict *qdict) \
> { \
> @@ -30,6 +33,24 @@ struct MonitorDef {
> int64_t (*get_value)(Monitor *mon, const MonitorDef *md, int offset);
> };
>
> +void monitor_new_hmp(const char *id, const char *chardev_id,
> + bool use_readline, Error **errp);
> +
> +int monitor_vprintf(Monitor *mon, const char *fmt, va_list ap)
> + G_GNUC_PRINTF(2, 0);
> +int monitor_printf(Monitor *mon, const char *fmt, ...) G_GNUC_PRINTF(2, 3);
> +void monitor_printc(Monitor *mon, int ch);
> +
> +void monitor_hmp_read_command(MonitorHMP *hmp, int show_prompt);
> +int monitor_hmp_read_password(MonitorHMP *hmp, ReadLineFunc *readline_func,
> + void *opaque);
> +
> +void monitor_register_hmp(const char *name, bool info,
> + void (*cmd)(Monitor *mon, const QDict *qdict));
> +void monitor_register_hmp_info_hrt(const char *name,
> + HumanReadableText *(*handler)(Error
> **errp));
> +
> +
> CPUArchState *mon_get_cpu_env(Monitor *mon);
> CPUState *mon_get_cpu(Monitor *mon);
>
> diff --git a/include/monitor/monitor.h b/include/monitor/monitor.h
> index 9f048ba103b5..72a8f6ea5b4f 100644
> --- a/include/monitor/monitor.h
> +++ b/include/monitor/monitor.h
> @@ -10,9 +10,6 @@
> #define TYPE_MONITOR "monitor"
> OBJECT_DECLARE_TYPE(Monitor, MonitorClass, MONITOR);
>
> -#define TYPE_MONITOR_HMP "monitor-hmp"
> -OBJECT_DECLARE_TYPE(MonitorHMP, MonitorHMPClass, MONITOR_HMP);
> -
> #define TYPE_MONITOR_QMP "monitor-qmp"
> OBJECT_DECLARE_TYPE(MonitorQMP, MonitorQMPClass, MONITOR_QMP);
>
> @@ -30,8 +27,6 @@ void monitor_init_globals_core(void);
> char *monitor_compat_id(void);
> void monitor_new_qmp(const char *id, const char *chardev_id,
> bool pretty, Error **errp);
> -void monitor_new_hmp(const char *id, const char *chardev_id,
> - bool use_readline, Error **errp);
> int monitor_new(MonitorOptions *opts, bool allow_hmp, Error **errp);
> int monitor_new_opts(QemuOpts *opts, Error **errp);
> void monitor_cleanup(void);
> @@ -43,28 +38,15 @@ int monitor_get_fd(Monitor *mon, const char *fdname,
> Error **errp);
> int monitor_fd_param(Monitor *mon, const char *fdname, Error **errp);
>
> int monitor_puts(Monitor *mon, const char *str);
> -int monitor_vprintf(Monitor *mon, const char *fmt, va_list ap)
> - G_GNUC_PRINTF(2, 0);
> -int monitor_printf(Monitor *mon, const char *fmt, ...) G_GNUC_PRINTF(2, 3);
> -void monitor_printc(Monitor *mon, int ch);
> void monitor_flush(Monitor *mon);
> int monitor_get_cpu_index(Monitor *mon);
>
> int monitor_puts_locked(Monitor *mon, const char *str);
> void monitor_flush_locked(Monitor *mon);
>
> -void monitor_read_command(MonitorHMP *hmp, int show_prompt);
> -int monitor_read_password(MonitorHMP *hmp, ReadLineFunc *readline_func,
> - void *opaque);
> -
> AddfdInfo *monitor_fdset_add_fd(int fd, bool has_fdset_id, int64_t fdset_id,
> const char *opaque, Error **errp);
> int monitor_fdset_dup_fd_add(int64_t fdset_id, int flags, Error **errp);
> void monitor_fdset_dup_fd_remove(int dup_fd);
>
> -void monitor_register_hmp(const char *name, bool info,
> - void (*cmd)(Monitor *mon, const QDict *qdict));
> -void monitor_register_hmp_info_hrt(const char *name,
> - HumanReadableText *(*handler)(Error
> **errp));
> -
> #endif /* MONITOR_H */
> diff --git a/monitor/hmp.c b/monitor/hmp.c
> index 8134dfaad4bb..b4d05d47c4bf 100644
> --- a/monitor/hmp.c
> +++ b/monitor/hmp.c
> @@ -136,7 +136,7 @@ static void monitor_command_cb(void *opaque, const char
> *cmdline,
> monitor_resume(&hmp->parent_obj);
> }
>
> -void monitor_read_command(MonitorHMP *hmp, int show_prompt)
> +void monitor_hmp_read_command(MonitorHMP *hmp, int show_prompt)
> {
> if (!hmp->rs) {
> return;
> @@ -148,8 +148,8 @@ void monitor_read_command(MonitorHMP *hmp, int
> show_prompt)
> }
> }
>
> -int monitor_read_password(MonitorHMP *hmp, ReadLineFunc *readline_func,
> - void *opaque)
> +int monitor_hmp_read_password(MonitorHMP *hmp, ReadLineFunc *readline_func,
> + void *opaque)
> {
> if (hmp->rs) {
> readline_start(hmp->rs, "Password: ", 1, readline_func, opaque);
> @@ -1647,7 +1647,7 @@ static void monitor_hmp_complete(UserCreatable *uc,
> Error **errp)
> monitor_readline_flush,
> hmp,
> monitor_find_completion);
> - monitor_read_command(hmp, 0);
> + monitor_hmp_read_command(hmp, 0);
> }
>
> qemu_chr_fe_set_handlers(&hmp->parent_obj.chr,
> diff --git a/monitor/monitor-internal.h b/monitor/monitor-internal.h
> index fdeeeb853636..ee9ba0c8231e 100644
> --- a/monitor/monitor-internal.h
> +++ b/monitor/monitor-internal.h
> @@ -27,6 +27,7 @@
>
> #include "chardev/char-fe.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qapi/qapi-emit-events.h"
> #include "qapi/qapi-types-control.h"
> #include "qapi/qapi-types-qom.h"
> diff --git a/net/slirp.c b/net/slirp.c
> index 517dd23be14b..9bf09a2c8bc9 100644
> --- a/net/slirp.c
> +++ b/net/slirp.c
> @@ -36,6 +36,7 @@
> #include "clients.h"
> #include "hub.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qemu/error-report.h"
> #include "qemu/sockets.h"
> #include <libslirp.h>
> diff --git a/stubs/monitor-core.c b/stubs/monitor-core.c
> index a7c32297c90a..b0c7002bd406 100644
> --- a/stubs/monitor-core.c
> +++ b/stubs/monitor-core.c
> @@ -1,5 +1,6 @@
> #include "qemu/osdep.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qapi/qapi-emit-events.h"
>
> Monitor *monitor_cur(void)
> diff --git a/stubs/monitor-internal.c b/stubs/monitor-internal.c
> index 731fad221ecc..6f69f1f14ae4 100644
> --- a/stubs/monitor-internal.c
> +++ b/stubs/monitor-internal.c
> @@ -1,6 +1,6 @@
> #include "qemu/osdep.h"
> #include "qapi/error.h"
> -#include "monitor/monitor.h"
> +#include "monitor/hmp.h"
>
> int monitor_get_fd(Monitor *mon, const char *name, Error **errp)
> {
> diff --git a/target/rx/disas.c b/target/rx/disas.c
> index 67b932882914..0eb2ee6f4507 100644
> --- a/target/rx/disas.c
> +++ b/target/rx/disas.c
> @@ -19,6 +19,7 @@
> #include "qemu/osdep.h"
> #include "disas/dis-asm.h"
> #include "qemu/bitops.h"
> +#include "monitor/hmp.h"
> #include "cpu.h"
>
> typedef struct DisasContext {
> diff --git a/tests/unit/test-util-sockets.c b/tests/unit/test-util-sockets.c
> index ab3f39c3efb5..b2a884529598 100644
> --- a/tests/unit/test-util-sockets.c
> +++ b/tests/unit/test-util-sockets.c
> @@ -24,6 +24,7 @@
> #include "qapi/error.h"
> #include "socket-helpers.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
>
> static void test_fd_is_socket_bad(void)
> {
> diff --git a/tools/qemu-vnc/stubs.c b/tools/qemu-vnc/stubs.c
> index 1c82d8cff430..26597fefaa99 100644
> --- a/tools/qemu-vnc/stubs.c
> +++ b/tools/qemu-vnc/stubs.c
> @@ -9,6 +9,7 @@
> #include "system/runstate.h"
> #include "hw/core/qdev.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "migration/vmstate.h"
>
> bool runstate_is_running(void)
> diff --git a/trace/trace-hmp-cmds.c b/trace/trace-hmp-cmds.c
> index 390173095cff..c8f0133abecf 100644
> --- a/trace/trace-hmp-cmds.c
> +++ b/trace/trace-hmp-cmds.c
> @@ -25,7 +25,6 @@
> #include "qemu/osdep.h"
> #include "monitor/hmp.h"
> #include "monitor/hmp-completion.h"
> -#include "monitor/monitor.h"
> #include "qapi/error.h"
> #include "qapi/qapi-commands-trace.h"
> #include "qobject/qdict.h"
> diff --git a/ui/ui-hmp-cmds.c b/ui/ui-hmp-cmds.c
> index 806a7bece7cb..4ef459490ba2 100644
> --- a/ui/ui-hmp-cmds.c
> +++ b/ui/ui-hmp-cmds.c
> @@ -327,7 +327,7 @@ static void hmp_change_read_arg(void *opaque, const char
> *password,
> void *readline_opaque)
> {
> qmp_change_vnc_password(password, NULL);
> - monitor_read_command(opaque, 1);
> + monitor_hmp_read_command(opaque, 1);
> }
>
> void hmp_change_vnc(Monitor *mon, const char *device, const char *target,
> @@ -344,7 +344,7 @@ void hmp_change_vnc(Monitor *mon, const char *device,
> const char *target,
> }
> if (!arg) {
> MonitorHMP *hmp = MONITOR_HMP(mon);
> - monitor_read_password(hmp, hmp_change_read_arg, NULL);
> + monitor_hmp_read_password(hmp, hmp_change_read_arg, NULL);
> } else {
> qmp_change_vnc_password(arg, errp);
> }
> diff --git a/util/error-report.c b/util/error-report.c
> index f333af9249b9..aaa15bc79827 100644
> --- a/util/error-report.c
> +++ b/util/error-report.c
> @@ -11,7 +11,7 @@
> */
>
> #include "qemu/osdep.h"
> -#include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qemu/error-report.h"
>
> /*
> diff --git a/util/qemu-print.c b/util/qemu-print.c
> index 7b9591035e57..a2d1f0244168 100644
> --- a/util/qemu-print.c
> +++ b/util/qemu-print.c
> @@ -12,6 +12,7 @@
>
> #include "qemu/osdep.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qemu/qemu-print.h"
>
> /*
>
> --
> 2.55.0.543.g5ebe2ebe4ea8
>
--
-----Open up your eyes, open up your mind, open up your code -------
/ Dr. David Alan Gilbert | Running GNU/Linux | Happy \
\ dave @ treblig.org | | In Hex /
\ _________________________|_____ http://www.treblig.org |_______/
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |