openkylin
/
qemu
mirror of https://gitee.com/openkylin/qemu.git
9
0
Fork 0
Code Issues Projects Releases Wiki Activity

vfio-pci: Remove setting of MSI qsize 

This was a misinterpretation of the spec, hardware doesn't get to
specify how many were actually enabled through this field.

Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
This commit is contained in:
Alex Williamson 2012-10-08 08:45:30 -06:00
parent 5976cdd58b
commit 9b1e45c8f1
1 changed files with 0 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
hw/vfio_pci.c
View File

@ -508,20 +508,6 @@ static MSIMessage msi_get_msg(PCIDevice *pdev, unsigned int vector)
return msg; return msg;
} }
/* So should this */
static void msi_set_qsize(PCIDevice *pdev, uint8_t size)
{
uint8_t *config = pdev->config + pdev->msi_cap;
uint16_t flags;
flags = pci_get_word(config + PCI_MSI_FLAGS);
flags = le16_to_cpu(flags);
flags &= ~PCI_MSI_FLAGS_QSIZE;
flags |= (size & 0x7) << 4;
flags = cpu_to_le16(flags);
pci_set_word(config + PCI_MSI_FLAGS, flags);
}
static void vfio_enable_msix(VFIODevice *vdev) static void vfio_enable_msix(VFIODevice *vdev)
{ {
vfio_disable_interrupts(vdev); vfio_disable_interrupts(vdev);
@ -609,8 +595,6 @@ retry:
return; return;
} }
msi_set_qsize(&vdev->pdev, vdev->nr_vectors);
vdev->interrupt = VFIO_INT_MSI; vdev->interrupt = VFIO_INT_MSI;
DPRINTF("%s(%04x:%02x:%02x.%x) Enabled %d MSI vectors\n", __func__, DPRINTF("%s(%04x:%02x:%02x.%x) Enabled %d MSI vectors\n", __func__,
@ -671,8 +655,6 @@ static void vfio_disable_msi(VFIODevice *vdev)
vfio_disable_msi_common(vdev); vfio_disable_msi_common(vdev);
msi_set_qsize(&vdev->pdev, 0); /* Actually still means 1 vector */
DPRINTF("%s(%04x:%02x:%02x.%x)\n", __func__, vdev->host.domain, DPRINTF("%s(%04x:%02x:%02x.%x)\n", __func__, vdev->host.domain,
vdev->host.bus, vdev->host.slot, vdev->host.function); vdev->host.bus, vdev->host.slot, vdev->host.function);
} }