On 31/10/2017 12:25, Pavel Dovgalyuk wrote: > > - bdrv_drain_all(); > - replay_disable_events(); > - ret = bdrv_flush_all(); > + if (!replay_events_enabled()) { > + bdrv_drain_all(); > + ret = bdrv_flush_all(); > + } > > return ret; > } > diff --git a/migration/savevm.c b/migration/savevm.c > index 20cebe1..41a13c0 100644 > --- a/migration/savevm.c > +++ b/migration/savevm.c > @@ -2143,8 +2143,8 @@ int save_snapshot(const char *name, Error **errp) > AioContext *aio_context; > > if (!replay_can_snapshot()) { > - monitor_printf(mon, "Record/replay does not allow making snapshot " > - "right now. Try once more later.\n"); > + error_report("Record/replay does not allow making snapshot " > + "right now. Try once more later.\n");
Why this change? Paolo > return ret;