[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-6.0 1/6] qapi: Add query-accel command
From: |
Eric Blake |
Subject: |
Re: [PATCH for-6.0 1/6] qapi: Add query-accel command |
Date: |
Mon, 16 Nov 2020 10:20:04 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0 |
On 11/16/20 7:10 AM, Roman Bolshakov wrote:
> There's a problem for management applications to determine if certain
> accelerators available. Generic QMP command should help with that.
>
> Signed-off-by: Roman Bolshakov <r.bolshakov@yadro.com>
> ---
> monitor/qmp-cmds.c | 15 +++++++++++++++
> qapi/machine.json | 19 +++++++++++++++++++
> 2 files changed, 34 insertions(+)
>
> +++ b/qapi/machine.json
> @@ -591,6 +591,25 @@
> ##
> { 'command': 'query-kvm', 'returns': 'KvmInfo' }
>
> +##
> +# @query-accel:
> +#
> +# Returns information about an accelerator
> +#
> +# Returns: @KvmInfo
> +#
> +# Since: 6.0.0
We're inconsistent on whether we have 'Since: x.y' or 'Since: x.y.z',
although I prefer the shorter form. Maybe Markus has an opnion on that.
> +#
> +# Example:
> +#
> +# -> { "execute": "query-accel", "arguments": { "name": "kvm" } }
> +# <- { "return": { "enabled": true, "present": true } }
> +#
> +##
> +{ 'command': 'query-accel',
> + 'data': { 'name': 'str' },
> + 'returns': 'KvmInfo' }
'@name' is undocumented and an open-coded string. Better would be
requiring 'name' to be one of an enum type. Even better would be
returning an array of KvmInfo with information on all supported
accelerators at once, rather than making the user call this command once
per name.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
- [PATCH for-6.0 0/6] Add HMP/QMP commands to query accelerator, Roman Bolshakov, 2020/11/16
- [PATCH for-6.0 1/6] qapi: Add query-accel command, Roman Bolshakov, 2020/11/16
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command,
Eric Blake <=
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command, Roman Bolshakov, 2020/11/16
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command, Eduardo Habkost, 2020/11/16
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command, Markus Armbruster, 2020/11/17
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command, Roman Bolshakov, 2020/11/17
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command, Markus Armbruster, 2020/11/18
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command, Paolo Bonzini, 2020/11/18
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command, Markus Armbruster, 2020/11/18
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command, Paolo Bonzini, 2020/11/18
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command, Markus Armbruster, 2020/11/18
- Re: [PATCH for-6.0 1/6] qapi: Add query-accel command, Paolo Bonzini, 2020/11/18