On 2/4/21 6:18 PM, Daniel P. Berrangé wrote: > This is an incremental step in converting vmstate loading code to report > via Error objects instead of printing directly to the console/monitor. > > Signed-off-by: Daniel P. Berrangé <berra...@redhat.com> > --- > migration/savevm.c | 18 +++++++----------- > 1 file changed, 7 insertions(+), 11 deletions(-) > > diff --git a/migration/savevm.c b/migration/savevm.c > index c505526406..447596383f 100644 > --- a/migration/savevm.c > +++ b/migration/savevm.c > @@ -1909,14 +1909,15 @@ static void *postcopy_ram_listen_thread(void *opaque) > } ...
> @@ -1937,12 +1938,12 @@ static int > loadvm_postcopy_handle_listen(MigrationIncomingState *mis) > if (migrate_postcopy_ram()) { > if (postcopy_ram_incoming_setup(mis)) { > postcopy_ram_incoming_cleanup(mis); > + error_setg(errp, "Failed to setup incoming postcoyp RAM blocks"); New error, OK. Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> > return -1; > } > }