Re: [PATCH 5/6] vdpa: introduce virtio pci driver

From: Jason Wang
Date: Fri Jun 05 2020 - 04:54:40 EST



On 2020/6/2 äå3:08, Jason Wang wrote:

+static const struct pci_device_id vp_vdpa_id_table[] = {
+ÂÂÂ { PCI_DEVICE(PCI_VENDOR_ID_REDHAT_QUMRANET, PCI_ANY_ID) },
+ÂÂÂ { 0 }
+};
This looks like it'll create a mess with either virtio pci
or vdpa being loaded at random. Maybe just don't specify
any IDs for now. Down the road we could get a
distinct vendor ID or a range of device IDs for this.


Right, will do.

Thanks


Rethink about this. If we don't specify any ID, the binding won't work.

How about using a dedicated subsystem vendor id for this?

Thanks