[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: |
Wen Congyang |
Subject: |
[Qemu-devel] Re: [PATCH] correct migrate_set_speed's args_type |
Date: |
Wed, 24 Nov 2010 09:28:54 +0800 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100413 Fedora/3.0.4-2.fc13 Thunderbird/3.0.4 |
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?
But in the function do_migrate_set_speed(), we only accept a integer.
>
> 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,
>>
>
>