[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: |
Markus Armbruster |
Subject: |
[Qemu-devel] Re: [PATCH] correct migrate_set_speed's args_type |
Date: |
Fri, 10 Dec 2010 16:20:34 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
[Note cc: Dan, Avi]
Luiz Capitulino <address@hidden> writes:
> 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.
>
> Markus, are you aware of this change?
I pointed out the incompatible change in my review[*] of the offending
patch:
This isn't backwards bug-compatible.
Before, a client could send any number. Any fractional part was
ignored.
Now, the number must be an integer. Other numbers are rejected.
I don't care myself, but others have argued most forcefully for keeping
QMP fully backward compatible from 0.13 on, so they might object.
"Others" did not object, so this went in. Not released yet, so it's not
too late to object.
[*] http://lists.gnu.org/archive/html/qemu-devel/2010-09/msg01905.html
- [Qemu-devel] Re: [PATCH] correct migrate_set_speed's args_type,
Markus Armbruster <=