[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 4/7] qapi: Re-arrange monitor.c functions
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH v4 4/7] qapi: Re-arrange monitor.c functions |
Date: |
Tue, 26 Jun 2012 18:28:50 -0300 |
On Tue, 26 Jun 2012 17:16:28 -0400
Corey Bryant <address@hidden> wrote:
>
>
> On 06/26/2012 04:45 PM, Luiz Capitulino wrote:
> > On Fri, 22 Jun 2012 14:36:11 -0400
> > Corey Bryant <address@hidden> wrote:
> >
> >> Signed-off-by: Corey Bryant <address@hidden>
> >> ---
> >> v4:
> >> -This patch is new in v4.
> >
> > Can you explain why it's needed?
> >
>
> This just made patch 3/7 easier to review.
Better to explain it then and move it to before that patch.
>
> >>
> >> 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"
> >
> >
>
[Qemu-devel] [PATCH v4 5/7] block: Prevent /dev/fd/X filename from being detected as floppy, Corey Bryant, 2012/06/22
[Qemu-devel] [PATCH v4 4/7] qapi: Re-arrange monitor.c functions, Corey Bryant, 2012/06/22
[Qemu-devel] [PATCH v4 6/7] block: Convert open calls to qemu_open, Corey Bryant, 2012/06/22
[Qemu-devel] [PATCH v4 3/7] qapi: Add pass-fd QMP command, Corey Bryant, 2012/06/22
Re: [Qemu-devel] [PATCH v4 0/7] file descriptor passing using pass-fd, Daniel P. Berrange, 2012/06/26