[libvirt] [PATCH 5/5] domaincaps: Report graphics spiceGL

Michal Privoznik mprivozn at redhat.com
Mon May 9 09:27:02 UTC 2016


On 08.05.2016 19:49, Cole Robinson wrote:
> Reports a tristate enum value for acceptable video model=virtio
> <acceleration accel3d=XXX/>.
> 
> Wire it up for qemu too. 'no' is always a valid value, so we
> unconditionally report it.
> ---
>  docs/formatdomaincaps.html.in                           | 9 +++++++++
>  src/conf/domain_capabilities.c                          | 1 +
>  src/conf/domain_capabilities.h                          | 2 ++
>  src/qemu/qemu_capabilities.c                            | 4 ++++
>  tests/domaincapsschemadata/domaincaps-full.xml          | 5 +++++
>  tests/domaincapsschemadata/domaincaps-qemu_1.6.50-1.xml | 3 +++
>  tests/domaincapsschemadata/domaincaps-qemu_2.6.0-1.xml  | 4 ++++
>  tests/domaincapsschemadata/domaincaps-qemu_2.6.0-2.xml  | 3 +++
>  tests/domaincapsschemadata/domaincaps-qemu_2.6.0-3.xml  | 3 +++
>  tests/domaincapsschemadata/domaincaps-qemu_2.6.0-4.xml  | 3 +++
>  tests/domaincapsschemadata/domaincaps-qemu_2.6.0-5.xml  | 3 +++
>  tests/domaincapstest.c                                  | 1 +
>  12 files changed, 41 insertions(+)
> 
> diff --git a/docs/formatdomaincaps.html.in b/docs/formatdomaincaps.html.in
> index c424107..b87a45a 100644
> --- a/docs/formatdomaincaps.html.in
> +++ b/docs/formatdomaincaps.html.in
> @@ -268,6 +268,10 @@
>          <value>qxl</value>
>          <value>virtio</value>
>        </enum>
> +      <enum name='virtioAccel3D'>
> +        <value>yes</value>
> +        <value>no</value>
> +      </enum>
>      </video>
>      ...
>    </devices>
> @@ -278,6 +282,11 @@
>        <dt><code>modelType</code></dt>
>        <dd>Options for the <code>type</code> attribute of the
>        <video><model> element.</dd>
> +
> +      <dt><code>spiceGL</code></dt>

Copy paste error. This should have been virtioAccel3D. This is more or
less the same problem (maybe too strong term though) that I've raised in
3/5. Just accel3D perhaps? Otherwise the code looks good.

Michal




More information about the libvir-list mailing list