* Peter Xu ([email protected]) wrote: > On Thu, Oct 06, 2022 at 05:59:51PM +0100, Dr. David Alan Gilbert wrote: > > > @@ -2999,8 +3010,8 @@ void > > > ram_postcopy_send_discard_bitmap(MigrationState *ms) > > > migration_bitmap_sync(rs); > > > > > > /* Easiest way to make sure we don't resume in the middle of a > > > host-page */ > > > + rs->pss[RAM_CHANNEL_PRECOPY].last_sent_block = NULL; > > > > Why don't we reset the postcopy one here as well? > > Because ram_postcopy_send_discard_bitmap() is only called before postcopy > starts, so the other field should be NULL already. Thanks,
Ah, yes, OK Reviewed-by: Dr. David Alan Gilbert <[email protected]> > -- > Peter Xu > -- Dr. David Alan Gilbert / [email protected] / Manchester, UK
