xemu/hw/virtio
Ladi Prosek f2bc54de47 virtio-pci: Don't force Subsystem Vendor ID = Vendor ID
The statement being removed doesn't change anything as virtio PCI devices already
have Subsystem Vendor ID set to pci_default_sub_vendor_id (0x1af4), same as Vendor
ID. And the Virtio spec does not require the two to be equal, either:

  "The PCI Subsystem Vendor ID and the PCI Subsystem Device ID MAY reflect the PCI
  Vendor and Device ID of the environment (for informational purposes by the driver)."

Background:

Following the recent virtio-win licensing change, several vendors are planning to
ship their own certified version of Windows guest Virtio drivers, potentially taking
advantage of Windows Update as a distribution channel. It is therefore critical that
each vendor uses their own PCI Subsystem Vendor ID for Virtio devices to prevent
drivers from other vendors binding to it.

This would be trivially done by adding:

  k->subsystem_vendor_id = ...

to virtio_pci_class_init(). Except for the problematic statement deleted by this
patch, which reverts the Subsystem Vendor ID back to 0x1af4 for legacy devices for
no good reason.

Signed-off-by: Ladi Prosek <lprosek@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Gerd Hoffmann <kraxel@redhat.com>
2017-12-22 01:42:03 +02:00
..
Makefile.objs stubs: move vhost stubs to stubs/vhost.o 2017-01-16 17:52:35 +01:00
trace-events trace-events: fix code style: print 0x before hex numbers 2017-08-01 12:13:07 +01:00
vhost-backend.c vhost: fix a memory leak 2017-08-02 00:13:25 +03:00
vhost-stub.c stubs: move vhost stubs to stubs/vhost.o 2017-01-16 17:52:35 +01:00
vhost-user.c vhost-user: fix legacy cross-endian configurations 2017-08-02 00:13:25 +03:00
vhost-vsock.c migration: pre_save return int 2017-09-27 11:35:59 +01:00
vhost.c vhost: fix error check in vhost_verify_ring_mappings() 2017-12-01 19:05:58 +02:00
virtio-balloon.c Convert single line fprintf(.../n) to warn_report() 2017-09-19 14:09:34 +02:00
virtio-bus.c virtio: Fix error handling in virtio_bus_device_plugged 2017-03-22 17:54:32 +02:00
virtio-crypto-pci.c virtio-crypto: Convert to DEFINE_PROP_LINK 2017-07-14 12:04:43 +02:00
virtio-crypto.c virtio-crypto: Convert to DEFINE_PROP_LINK 2017-07-14 12:04:43 +02:00
virtio-mmio.c virtio-mmio: switch to linux headers 2017-01-18 22:59:53 +02:00
virtio-pci.c virtio-pci: Don't force Subsystem Vendor ID = Vendor ID 2017-12-22 01:42:03 +02:00
virtio-pci.h virtio-pci: Replace modern_as with direct access to modern_bar 2017-10-15 05:54:44 +03:00
virtio-rng.c virtio-rng: Convert to DEFINE_PROP_LINK 2017-07-14 12:04:42 +02:00
virtio.c virtio_error: don't invoke status callbacks 2017-12-19 23:41:00 +02:00