[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v12 01/13] Add MigrationParams structure
From: |
Orit Wasserman |
Subject: |
Re: [Qemu-devel] [PATCH v12 01/13] Add MigrationParams structure |
Date: |
Tue, 19 Jun 2012 19:02:02 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120430 Thunderbird/12.0.1 |
On 06/19/2012 07:00 PM, Eric Blake wrote:
> On 06/19/2012 09:43 AM, Orit Wasserman wrote:
>> From: Isaku Yamahata <address@hidden>
>>
>> Signed-off-by: Isaku Yamahata <address@hidden>
>> ---
>
>>
>> -static void block_set_params(int blk_enable, int shared_base, void *opaque)
>> +static void block_set_params(const MigrationParams *params, void *opaque)
>
> As long as we are changing this signature...
>
>> @@ -394,9 +393,13 @@ void qmp_migrate(const char *uri, bool has_blk, bool
>> blk,
>> Error **errp)
>> {
>> MigrationState *s = migrate_get_current();
>> + MigrationParams params;
>> const char *p;
>> int ret;
>>
>> + params.blk = blk;
>
> ...and since blk is only ever a bool to begin with...
>
>>
>> +struct MigrationParams {
>> + int blk;
>> + int shared;
>
> ...why not use bool instead of int in this struct to represent the fact
> that it really is a binary choice? I think the same can be said for shared.
you right , I will fix it.
Orit
>
- [Qemu-devel] [PATCH v12 00/13] XBZRLE delta for live migration of large memory app, Orit Wasserman, 2012/06/19
- [Qemu-devel] [PATCH v12 01/13] Add MigrationParams structure, Orit Wasserman, 2012/06/19
- [Qemu-devel] [PATCH v12 03/13] Add XBZRLE documentation, Orit Wasserman, 2012/06/19
- [Qemu-devel] [PATCH v12 02/13] Add migration capabilites, Orit Wasserman, 2012/06/19
- [Qemu-devel] [PATCH v12 05/13] Add uleb encoding/decoding functions, Orit Wasserman, 2012/06/19
- [Qemu-devel] [PATCH v12 04/13] Add cache handling functions, Orit Wasserman, 2012/06/19
- [Qemu-devel] [PATCH v12 06/13] Add save_block_hdr function, Orit Wasserman, 2012/06/19
- [Qemu-devel] [PATCH v12 07/13] Add debugging infrastructure, Orit Wasserman, 2012/06/19