On Fri, Aug 25, 2017 at 04:51:29PM +0100, Dr. David Alan Gilbert wrote: [...]
> > PS, in migration_incoming_get_current() we do > > mis_current.state = MIGRATION_STATUS_NONE; > > memset(&mis_current, 0, sizeof(MigrationIncomingState)); > > > > and the first line there is pointless because the memset > > blasts zeroes over it anyway. > > Hmm yes. I'll include this cleanup within my (future) patch to init incoming migration object in migration_object_init() as well. Thanks, -- Peter Xu