Re: [Qemu-devel] [PATCH] migration: remove unused field bytes_xfer

[Juan Quintela]

Wei Yang <address@hidden> wrote:
> On Tue, Apr 02, 2019 at 08:31:06AM +0800, Wei Yang wrote:
>>MigrationState->bytes_xfer is only set to 0 in migrate_init().
>>
>>Remove this unnecessary field.
>>
>>Signed-off-by: Wei Yang <address@hidden>
>
> Hi, David

Hi

I am on duty this week, will get it.

>
> Are you willing to pick up this one?
>
>>---
>> migration/migration.c | 1 -
>> migration/migration.h | 1 -
>> 2 files changed, 2 deletions(-)
>>
>>diff --git a/migration/migration.c b/migration/migration.c
>>index dea7078bf4..c929cf8d0f 100644
>>--- a/migration/migration.c
>>+++ b/migration/migration.c
>>@@ -1681,7 +1681,6 @@ void migrate_init(MigrationState *s)
>>      * parameters/capabilities that the user set, and
>>      * locks.
>>      */
>>-    s->bytes_xfer = 0;
>>     s->cleanup_bh = 0;
>>     s->to_dst_file = NULL;
>>     s->rp_state.from_dst_file = NULL;
>>diff --git a/migration/migration.h b/migration/migration.h
>>index 852eb3c4e9..b9efbe9168 100644
>>--- a/migration/migration.h
>>+++ b/migration/migration.h
>>@@ -116,7 +116,6 @@ struct MigrationState
>>     DeviceState parent_obj;
>> 
>>     /*< public >*/
>>-    size_t bytes_xfer;
>>     QemuThread thread;
>>     QEMUBH *cleanup_bh;
>>     QEMUFile *to_dst_file;
>>-- 
>>2.19.1