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.y...@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
