diff options
author | Alex Williamson <alex.williamson@redhat.com> | 2015-09-23 13:04:45 -0600 |
---|---|---|
committer | Alex Williamson <alex.williamson@redhat.com> | 2015-09-23 13:04:45 -0600 |
commit | 056dfcb695cde3c62b7dc1d5ed6d2e38b3a73e29 (patch) | |
tree | d3ebd0c5118e663def22affc7abfab3c66a44a48 /hw/vfio/pci.h | |
parent | c00d61d8fa22b096b15e19ee2fde846ffc1c0b5d (diff) |
vfio/pci: Cleanup ROM blacklist quirk
Create a vendor:device ID helper that we'll also use as we rework the
rest of the quirks. Re-reading the config entries, even if we get
more blacklist entries, is trivial overhead and only incurred during
device setup. There's no need to typedef the blacklist structure,
it's a static private data type used once. The elements get bumped
up to uint32_t to avoid future maintenance issues if PCI_ANY_ID gets
used for a blacklist entry (avoiding an actual hardware match). Our
test loop is also crying out to be simplified as a for loop.
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Diffstat (limited to 'hw/vfio/pci.h')
-rw-r--r-- | hw/vfio/pci.h | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/hw/vfio/pci.h b/hw/vfio/pci.h index ff94929b45..f6dbe7ff9e 100644 --- a/hw/vfio/pci.h +++ b/hw/vfio/pci.h @@ -150,11 +150,6 @@ typedef struct VFIOPCIDevice { bool no_kvm_msix; } VFIOPCIDevice; -typedef struct VFIORomBlacklistEntry { - uint16_t vendor_id; - uint16_t device_id; -} VFIORomBlacklistEntry; - uint32_t vfio_pci_read_config(PCIDevice *pdev, uint32_t addr, int len); void vfio_pci_write_config(PCIDevice *pdev, uint32_t addr, uint32_t val, int len); |