Re: [PATCH v4 4/7] qapi: Re-arrange monitor.c functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 22 Jun 2012 14:36:11 -0400
Corey Bryant <coreyb@xxxxxxxxxxxxxxxxxx> wrote:

> Signed-off-by: Corey Bryant <coreyb@xxxxxxxxxxxxxxxxxx>
> ---
> v4:
>  -This patch is new in v4.

Can you explain why it's needed?

> 
>  monitor.c |   40 ++++++++++++++++++++--------------------
>  1 file changed, 20 insertions(+), 20 deletions(-)
> 
> diff --git a/monitor.c b/monitor.c
> index 3433c06..153e949 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -2239,26 +2239,6 @@ static int monitor_put_fd(Monitor *mon, const char *fdname, bool replace,
>      return fd;
>  }
>  
> -void qmp_closefd(const char *fdname, Error **errp)
> -{
> -    int fd = monitor_get_fd(cur_mon, fdname, errp);
> -    if (fd != -1) {
> -        close(fd);
> -    }
> -}
> -
> -static void do_loadvm(Monitor *mon, const QDict *qdict)
> -{
> -    int saved_vm_running  = runstate_is_running();
> -    const char *name = qdict_get_str(qdict, "name");
> -
> -    vm_stop(RUN_STATE_RESTORE_VM);
> -
> -    if (load_vmstate(name) == 0 && saved_vm_running) {
> -        vm_start();
> -    }
> -}
> -
>  int monitor_get_fd(Monitor *mon, const char *fdname, Error **errp)
>  {
>      mon_fd_t *monfd;
> @@ -2299,6 +2279,26 @@ void qmp_getfd(const char *fdname, Error **errp)
>      monitor_put_fd(cur_mon, fdname, replace, copy, errp);
>  }
>  
> +void qmp_closefd(const char *fdname, Error **errp)
> +{
> +    int fd = monitor_get_fd(cur_mon, fdname, errp);
> +    if (fd != -1) {
> +        close(fd);
> +    }
> +}
> +
> +static void do_loadvm(Monitor *mon, const QDict *qdict)
> +{
> +    int saved_vm_running  = runstate_is_running();
> +    const char *name = qdict_get_str(qdict, "name");
> +
> +    vm_stop(RUN_STATE_RESTORE_VM);
> +
> +    if (load_vmstate(name) == 0 && saved_vm_running) {
> +        vm_start();
> +    }
> +}
> +
>  /* mon_cmds and info_cmds would be sorted at runtime */
>  static mon_cmd_t mon_cmds[] = {
>  #include "hmp-commands.h"

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]