[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 5/6] qmp/hmp: add set-vm-generation-id comman
From: |
Ben Warren |
Subject: |
Re: [Qemu-devel] [PATCH v2 5/6] qmp/hmp: add set-vm-generation-id commands |
Date: |
Wed, 18 Jan 2017 14:14:16 -0800 |
> On Jan 18, 2017, at 12:55 AM, Igor Mammedov <address@hidden> wrote:
>
> On Mon, 16 Jan 2017 11:20:57 -0800
> address@hidden <mailto:address@hidden> wrote:
>
>> From: Igor Mammedov <address@hidden>
>>
>> Add set-vm-generation-id command to set Virtual Machine
>> Generation ID counter.
>>
>> QMP command example:
>> { "execute": "set-vm-generation-id",
>> "arguments": {
>> "guid": "324e6eaf-d1d1-4bf6-bf41-b9bb6c91fb87"
>> }
>> }
>>
>> HMP command example:
>> set-vm-generation-id guid=324e6eaf-d1d1-4bf6-bf41-b9bb6c91fb87
>>
>> Signed-off-by: Ben Warren <address@hidden>
>> Cc: Igor Mammedov <address@hidden>
>> Cc: Eric Blake <address@hidden>
> when you are reposting someone else written patch,
> you are supposed to keep SoB lines patch has had and just add
> your own SoB after original SoB lines.
>
> If you made non trivial change to the patch then ask
> original author(s) if they wish to keep their SoB
> before posting modified patch.
> (Usually I do it offline).
>
Yeah, this was a mistake on my part. This is essentially your older patch
rebased to ToT. I meant to just append my SoB
>> ---
>> hmp-commands.hx | 13 +++++++++++++
>> hmp.c | 12 ++++++++++++
>> hmp.h | 1 +
>> qapi-schema.json | 12 ++++++++++++
>> stubs/vmgenid.c | 6 ++++++
>> 5 files changed, 44 insertions(+)
>>
>> diff --git a/hmp-commands.hx b/hmp-commands.hx
>> index 8819281..56744aa 100644
>> --- a/hmp-commands.hx
>> +++ b/hmp-commands.hx
>> @@ -1775,5 +1775,18 @@ ETEXI
>> },
>>
>> STEXI
>> address@hidden set-vm-generation-id @var{uuid}
>> +Set Virtual Machine Generation ID counter to @var{guid}
>> +ETEXI
>> +
>> + {
>> + .name = "set-vm-generation-id",
>> + .args_type = "guid:s",
>> + .params = "guid",
>> + .help = "Set Virtual Machine Generation ID counter",
>> + .cmd = hmp_set_vm_generation_id,
>> + },
>> +
>> +STEXI
>> @end table
>> ETEXI
>> diff --git a/hmp.c b/hmp.c
>> index 9ec27ae..a54a312 100644
>> --- a/hmp.c
>> +++ b/hmp.c
>> @@ -2579,3 +2579,15 @@ void hmp_info_vm_generation_id(Monitor *mon, const
>> QDict *qdict)
>> }
>> qapi_free_GuidInfo(info);
>> }
>> +
>> +void hmp_set_vm_generation_id(Monitor *mon, const QDict *qdict)
>> +{
>> + Error *errp = NULL;
>> + const char *guid = qdict_get_str(qdict, "guid");
>> +
>> + qmp_set_vm_generation_id(guid, &errp);
>> + if (errp) {
>> + hmp_handle_error(mon, &errp);
>> + return;
>> + }
>> +}
>> diff --git a/hmp.h b/hmp.h
>> index 799fd37..e0ac1e8 100644
>> --- a/hmp.h
>> +++ b/hmp.h
>> @@ -138,5 +138,6 @@ void hmp_rocker_of_dpa_groups(Monitor *mon, const QDict
>> *qdict);
>> void hmp_info_dump(Monitor *mon, const QDict *qdict);
>> void hmp_hotpluggable_cpus(Monitor *mon, const QDict *qdict);
>> void hmp_info_vm_generation_id(Monitor *mon, const QDict *qdict);
>> +void hmp_set_vm_generation_id(Monitor *mon, const QDict *qdict);
>>
>> #endif
>> diff --git a/qapi-schema.json b/qapi-schema.json
>> index 2348391..c5ebea4 100644
>> --- a/qapi-schema.json
>> +++ b/qapi-schema.json
>> @@ -4796,3 +4796,15 @@
>> # Since 2.9
>> ##
>> { 'command': 'query-vm-generation-id', 'returns': 'GuidInfo' }
>> +
>> +##
>> +# @set-vm-generation-id
>> +#
>> +# Set Virtual Machine Generation ID
>> +#
>> +# @changed: Is the Virtual Machine Generation ID a new value?
>> +# @guid: new GUID to set as Virtual Machine Generation ID
>> +#
>> +# Since 2.9
>> +##
>> +{ 'command': 'set-vm-generation-id', 'data': {'guid': 'str'} }
>> diff --git a/stubs/vmgenid.c b/stubs/vmgenid.c
>> index 8c448ac..d25d41b 100644
>> --- a/stubs/vmgenid.c
>> +++ b/stubs/vmgenid.c
>> @@ -6,3 +6,9 @@ GuidInfo *qmp_query_vm_generation_id(Error **errp)
>> error_setg(errp, "this command is not currently supported");
>> return NULL;
>> }
>> +
>> +void qmp_set_vm_generation_id(const char *guid, Error **errp)
>> +{
>> + error_setg(errp, "this command is not currently supported");
>> + return;
>> +}
smime.p7s
Description: S/MIME cryptographic signature
Message not available
Message not available
Message not available
Message not available