On 17/04/2017 22:00, Juan Quintela wrote:
> hmp_loadvm is already there, so be consistent.
Can you move both to hmp.c instead?
Paolo
> Signed-off-by: Juan Quintela <quint...@redhat.com>
> ---
> include/sysemu/sysemu.h | 1 -
> migration/savevm.c | 5 -----
> monitor.c | 5 +++++
> 3 files changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
> index 5f2f21d..146a0dc 100644
> --- a/include/sysemu/sysemu.h
> +++ b/include/sysemu/sysemu.h
> @@ -75,7 +75,6 @@ void qemu_remove_exit_notifier(Notifier *notify);
> void qemu_add_machine_init_done_notifier(Notifier *notify);
> void qemu_remove_machine_init_done_notifier(Notifier *notify);
>
> -void hmp_savevm(Monitor *mon, const QDict *qdict);
> int save_vmstate(Monitor *mon, const char *name);
> int load_vmstate(const char *name);
> void hmp_delvm(Monitor *mon, const QDict *qdict);
> diff --git a/migration/savevm.c b/migration/savevm.c
> index f628d01..cbd7e0d 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -2156,11 +2156,6 @@ int save_vmstate(Monitor *mon, const char *name)
> return ret;
> }
>
> -void hmp_savevm(Monitor *mon, const QDict *qdict)
> -{
> - save_vmstate(mon, qdict_get_try_str(qdict, "name"));
> -}
> -
> void qmp_xen_save_devices_state(const char *filename, Error **errp)
> {
> QEMUFile *f;
> diff --git a/monitor.c b/monitor.c
> index ceb0489..2fca4fb 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -1854,6 +1854,11 @@ static void hmp_loadvm(Monitor *mon, const QDict
> *qdict)
> }
> }
>
> +static void hmp_savevm(Monitor *mon, const QDict *qdict)
> +{
> + save_vmstate(mon, qdict_get_try_str(qdict, "name"));
> +}
> +
> int monitor_get_fd(Monitor *mon, const char *fdname, Error **errp)
> {
> mon_fd_t *monfd;
>
