[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2] migration: Convert 'status' of MigrationInfo

From: zhanghailiang
Subject: Re: [Qemu-devel] [PATCH v2] migration: Convert 'status' of MigrationInfo to use an enum type
Date: Tue, 3 Mar 2015 15:15:25 +0800
User-agent: Mozilla/5.0 (Windows NT 6.1; rv:31.0) Gecko/20100101 Thunderbird/31.1.1

On 2015/3/2 23:56, Eric Blake wrote:
On 02/27/2015 07:54 PM, zhanghailiang wrote:
On 2015/2/28 0:48, Eric Blake wrote:
On 02/26/2015 11:19 PM, zhanghailiang wrote:
The original 'status' is an open-coded 'str' type, convert it to use an
enum type.

+# @MigState:

Do we have to abbreviate?  I guess leaving it like this makes the rest
of the existing code base have less churn (since it matches the spelling
of the enum that was previous interanl only), but it might look nicer as

Yes, this is the reason ..., agreed, i don't like the abbreviate,
But there is already a 'MigrationState' type defined:

So, what about MigrationStatus ? ;)

That would be fine with me.

Bad news, this name has also been used :(

In hmp.c:

typedef struct MigrationStatus
    QEMUTimer *timer;
    Monitor *mon;
    bool is_block_migration;
} MigrationStatus

reply via email to

[Prev in Thread] Current Thread [Next in Thread]