[Libosinfo] [PATCH] device: add virtio-gpu

[Daniel P. Berrange]

On Fri, Feb 26, 2016 at 02:23:24PM +0100, Marc-André Lureau wrote:
> From: Marc-André Lureau <marcandre.lureau at gmail.com>
> 
> virtio-gpu is a non-legacy device, so device_id is always 0x10 + 0x1040.
> 
> Signed-off-by: Marc-André Lureau <marcandre.lureau at redhat.com>
> ---
>  data/device/pcisig.com/Makefile.am                  | 1 +
>  data/device/pcisig.com/pci-1af4-1050.d/Makefile.am  | 1 +
>  data/device/pcisig.com/pci-1af4-1050.d/class.xml.in | 6 ++++++
>  3 files changed, 8 insertions(+)
>  create mode 100644 data/device/pcisig.com/pci-1af4-1050.d/Makefile.am
>  create mode 100644 data/device/pcisig.com/pci-1af4-1050.d/class.xml.in
> 
> diff --git a/data/device/pcisig.com/Makefile.am b/data/device/pcisig.com/Makefile.am
> index 567618a..4fd99e9 100644
> --- a/data/device/pcisig.com/Makefile.am
> +++ b/data/device/pcisig.com/Makefile.am
> @@ -13,6 +13,7 @@ SUBDIRS = \
>  	pci-1af4-1002.d \
>  	pci-1af4-1003.d \
>  	pci-1af4-1009.d \
> +	pci-1af4-1050.d \
>  	pci-1b36-0100.d \
>  	pci-8086-100e.d \
>  	pci-8086-2415.d \
> diff --git a/data/device/pcisig.com/pci-1af4-1050.d/Makefile.am b/data/device/pcisig.com/pci-1af4-1050.d/Makefile.am
> new file mode 100644
> index 0000000..4ce2491
> --- /dev/null
> +++ b/data/device/pcisig.com/pci-1af4-1050.d/Makefile.am
> @@ -0,0 +1 @@
> +include ../../../Makefile.inc
> diff --git a/data/device/pcisig.com/pci-1af4-1050.d/class.xml.in b/data/device/pcisig.com/pci-1af4-1050.d/class.xml.in
> new file mode 100644
> index 0000000..4d3ec16
> --- /dev/null
> +++ b/data/device/pcisig.com/pci-1af4-1050.d/class.xml.in
> @@ -0,0 +1,6 @@
> +<libosinfo version="0.0.1">
> +  <device id="http://pcisig.com/pci/1af4/1050">
> +    <name>virtio-gpu</name>
> +    <class>video</class>
> +  </device>
> +</libosinfo>

That is sufficient *if* the device is included in the master pci.ids
file, but all the virtio-1.0 devices appear to be missing from it :-(

  https://pci-ids.ucw.cz/v2.2/pci.ids

We should poke someone to get this fixed.

In the mean time though you'll need to add the full set of fields

    <bus-type>pci</bus-type>
    <_vendor>Red Hat, Inc</_vendor>
    <vendor-id>1af4</vendor-id>
    <product>virtio-vga</product>
    <product-id>1050</product-id>

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|