[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/5] qmp: create qmp_savevm command
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 2/5] qmp: create qmp_savevm command |
Date: |
Tue, 01 Dec 2015 16:05:47 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
"Denis V. Lunev" <address@hidden> writes:
> On 11/17/2015 01:10 PM, Markus Armbruster wrote:
>> "Denis V. Lunev" <address@hidden> writes:
>>
>>> Signed-off-by: Denis V. Lunev <address@hidden>
>>> CC: Juan Quintela <address@hidden>
>>> CC: Amit Shah <address@hidden>
>>> CC: Markus Armbruster <address@hidden>
>>> CC: Eric Blake <address@hidden>
>>> ---
>>> migration/savevm.c | 5 +++++
>>> qapi-schema.json | 13 +++++++++++++
>>> qmp-commands.hx | 25 +++++++++++++++++++++++++
>>> 3 files changed, 43 insertions(+)
>>>
>>> diff --git a/migration/savevm.c b/migration/savevm.c
>>> index f83ffd0..565b10a 100644
>>> --- a/migration/savevm.c
>>> +++ b/migration/savevm.c
>>> @@ -2010,6 +2010,11 @@ void hmp_savevm(Monitor *mon, const QDict *qdict)
>>> }
>>> }
>>> +void qmp_savevm(bool has_name, const char *name, Error **errp)
>>> +{
>>> + do_savevm(has_name ? name : NULL, errp);
>>> +}
>>> +
>> Please name do_savevm() qmp_savevm() and drop this wrapper.
>>
>> We're working on omitting has_FOO for pointer-valued FOO.
> error code is used in the qemu-img.
> I can drop error code check there and replace it with checking of
> Error pointer.
> Is this OK for you?
Let me rephrase my request:
1. Instead of wrapping do_savevm() in qmp_savevm(), use it directly.
2. Rename do_savevm() to qmp_savevm(), and change its arguments to make
it work as QMP command handler.
Does that make sense to you?