Paolo Bonzini <pbonz...@redhat.com> wrote: > On 18/04/2017 11:44, Paolo Bonzini wrote: >> >> >> On 17/04/2017 22:00, Juan Quintela wrote: >>> load_vmstate() already use error_report, so be consistent. >> >> Better: make both return Error* via an Error** parameter, and add >> >> hmp_handle_error(mon, &err); >> >> to hmp_savevm and error_report_err(err) on the loading side. > > Not really, loadvm is also a monitor command (I was confusing it with > -incoming). So it can use hmp_handle_error too.
I think that I *understood* (famous last words) how the error should be done on hmp commands. We will see on next respin. Later, Juan.
