[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] qapi: Add 'acpi' field to 'query-machines' output
From: |
Markus Armbruster |
Subject: |
Re: [PATCH] qapi: Add 'acpi' field to 'query-machines' output |
Date: |
Mon, 27 Feb 2023 16:42:54 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Peter Krempa <pkrempa@redhat.com> writes:
> Report which machine types support ACPI so that management applications
> can properly use the 'acpi' property even on platforms such as ARM where
> support for ACPI depends on the machine type and thus checking presence
> of '-machine acpi=' in 'query-command-line-options' is insufficient.
>
> Signed-off-by: Peter Krempa <pkrempa@redhat.com>
[...]
> diff --git a/qapi/machine.json b/qapi/machine.json
> index b9228a5e46..f82a00963b 100644
> --- a/qapi/machine.json
> +++ b/qapi/machine.json
> @@ -155,6 +155,8 @@
> #
> # @default-ram-id: the default ID of initial RAM memory backend (since 5.2)
> #
> +# @acpi: machine type supports acpi (since 8.0)
supports ACPI
> +#
> # Since: 1.2
> ##
> { 'struct': 'MachineInfo',
> @@ -162,7 +164,7 @@
> '*is-default': 'bool', 'cpu-max': 'int',
> 'hotpluggable-cpus': 'bool', 'numa-mem-supported': 'bool',
> 'deprecated': 'bool', '*default-cpu-type': 'str',
> - '*default-ram-id': 'str' } }
> + '*default-ram-id': 'str', 'acpi': 'bool' } }
>
> ##
> # @query-machines:
With that
Acked-by: Markus Armbruster <armbru@redhat.com>