[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 19/19] monitor: remove monitor parameter from sa
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH 19/19] monitor: remove monitor parameter from save_vmstate |
Date: |
Wed, 19 Apr 2017 09:19:25 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Paolo Bonzini <address@hidden> 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.
- Re: [Qemu-devel] [PATCH 16/19] migration: Export ram.c functions in its own file, (continued)