[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] correct migrate_set_speed's args_type
From: |
Luiz Capitulino |
Subject: |
[Qemu-devel] Re: [PATCH] correct migrate_set_speed's args_type |
Date: |
Wed, 24 Nov 2010 15:48:57 -0200 |
On Wed, 24 Nov 2010 09:28:54 +0800
Wen Congyang <address@hidden> wrote:
> At 2010年11月23日 21:31, Luiz Capitulino Write:
> > On Tue, 23 Nov 2010 10:43:48 -0200
> > Luiz Capitulino <address@hidden> wrote:
> >
> >> On Tue, 23 Nov 2010 13:41:26 +0800
> >> Wen Congyang <address@hidden> wrote:
> >>
> >>> The args_type of migrate_set_speed in qmp-commands.hx is wrong.
> >>> When we set migrate speed by json, qemu will be core dumped.
> >>>
> >>> Signed-off-by: Wen Congyang
> >>
> >> Nice catch.
> >>
> >> Was caused by 07de3e60b05 and hence affects master only. Could you please
> >> mention that in the commit log? Also, your email address is missing
> >> in the signed-off-by line.
> >
> > There's another problem there: we used to accept a json number but now we
> > accept only a json integer.
>
> Do you mean the parameter passed to migrate_set_speed by json should be
> float or integer?
We used to accept both until the aforementioned commit.
> But in the function do_migrate_set_speed(), we only accept a integer.
Yes, that's a behavior change and I'd like to check with Markus that
he's aware of it. In any case, I'll apply your fix as soon as you send
me a new version with the changes I asked.
>
> >
> > Markus, are you aware of this change?
> >
> >>
> >>>
> >>> ---
> >>> qmp-commands.hx | 2 +-
> >>> 1 files changed, 1 insertions(+), 1 deletions(-)
> >>>
> >>> diff --git a/qmp-commands.hx b/qmp-commands.hx
> >>> index 793cf1c..16bdb08 100644
> >>> --- a/qmp-commands.hx
> >>> +++ b/qmp-commands.hx
> >>> @@ -495,7 +495,7 @@ EQMP
> >>>
> >>> {
> >>> .name = "migrate_set_speed",
> >>> - .args_type = "value:f",
> >>> + .args_type = "value:o",
> >>> .params = "value",
> >>> .help = "set maximum speed (in bytes) for migrations",
> >>> .user_print = monitor_user_noop,
> >>
> >
> >
>