Hi Alex
On Mon, Jul 16, 2018 at 03:17:57PM -0600, Alex Williamson wrote:
static bool vfio_pci_nointx(struct pci_dev *pdev) {
- /*
* Per PCI, no VF's should have INTx* Simply disable it here*/- if (pdev->is_virtfn)
return true;- switch (pdev->vendor) { case PCI_VENDOR_ID_INTEL: switch (pdev->device) {
Nak, this is not what vfio_pci_nointx() is meant for, it's been tried before and it was broken:
That seems to make sense. I'll spin a new patch and have that tested before reposting with the following suggestions.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/dr...
This would cause *every* VF to print:
dev_info(&pdev->dev, "Masking broken INTx support\n");
If the device is incapable of generating INTx then it's probably better to drop it at vfio_pci_get_irq_count(). Please also note the subject is misidentified, this is a "vfio/pci" patch, not an "x86/pci" patch. kvm@vger is the official list for vfio patches. Thanks,
Alex