Re: [PATCH 1/1] qxl: add subsystem_vendor_id property

[Gerd Hoffmann]

On Wed, Sep 28, 2022 at 05:52:44PM +0200, Denis V. Lunev wrote:
> This property is needed for WHQL/inboxing of Windows drivers. We do need
> to get drivers to be separated by the hypervisor vendors and that should
> be done as PCI subvendor ID.
> 
> This patch adds PCI subsystem vendor ID to QXL device to match that
> convention.

We have pci_default_sub_vendor_id + pci_default_sub_device_id in
hw/pci/pci.c.  If you want another subsystem id for another vendor
there is a single place to change it for all devices.

Right now there is no runtime switch for them, so updating it requires
a two-liner patch for your vendor build.  We can discuss changing that,
but that should best be coordinated with libvirt folks to make sure
the management stack actually allows setting the subsystem id without
needing hacks.

take care,
  Gerd