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

[Wei Yang <richardw.yang@linux.intel.com>]

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 <richardw.yang@linux.intel.com>

Hi, David

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

-- 
Wei Yang
Help you, Help me