msi: Use msi/msix_present more consistently
Replace some open-coded msi/msix_present checks. Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
This commit is contained in:
parent
95d6580024
commit
44701ab71a
2
hw/msi.c
2
hw/msi.c
@ -175,7 +175,7 @@ void msi_uninit(struct PCIDevice *dev)
|
||||
uint16_t flags;
|
||||
uint8_t cap_size;
|
||||
|
||||
if (!(dev->cap_present & QEMU_PCI_CAP_MSI)) {
|
||||
if (!msi_present(dev)) {
|
||||
return;
|
||||
}
|
||||
flags = pci_get_word(dev->config + msi_flags_off(dev));
|
||||
|
13
hw/msix.c
13
hw/msix.c
@ -319,8 +319,9 @@ static void msix_free_irq_entries(PCIDevice *dev)
|
||||
/* Clean up resources for the device. */
|
||||
int msix_uninit(PCIDevice *dev, MemoryRegion *bar)
|
||||
{
|
||||
if (!(dev->cap_present & QEMU_PCI_CAP_MSIX))
|
||||
if (!msix_present(dev)) {
|
||||
return 0;
|
||||
}
|
||||
pci_del_capability(dev, PCI_CAP_ID_MSIX, MSIX_CAP_LENGTH);
|
||||
dev->msix_cap = 0;
|
||||
msix_free_irq_entries(dev);
|
||||
@ -339,7 +340,7 @@ void msix_save(PCIDevice *dev, QEMUFile *f)
|
||||
{
|
||||
unsigned n = dev->msix_entries_nr;
|
||||
|
||||
if (!(dev->cap_present & QEMU_PCI_CAP_MSIX)) {
|
||||
if (!msix_present(dev)) {
|
||||
return;
|
||||
}
|
||||
|
||||
@ -353,7 +354,7 @@ void msix_load(PCIDevice *dev, QEMUFile *f)
|
||||
unsigned n = dev->msix_entries_nr;
|
||||
unsigned int vector;
|
||||
|
||||
if (!(dev->cap_present & QEMU_PCI_CAP_MSIX)) {
|
||||
if (!msix_present(dev)) {
|
||||
return;
|
||||
}
|
||||
|
||||
@ -407,8 +408,9 @@ void msix_notify(PCIDevice *dev, unsigned vector)
|
||||
|
||||
void msix_reset(PCIDevice *dev)
|
||||
{
|
||||
if (!(dev->cap_present & QEMU_PCI_CAP_MSIX))
|
||||
if (!msix_present(dev)) {
|
||||
return;
|
||||
}
|
||||
msix_free_irq_entries(dev);
|
||||
dev->config[dev->msix_cap + MSIX_CONTROL_OFFSET] &=
|
||||
~dev->wmask[dev->msix_cap + MSIX_CONTROL_OFFSET];
|
||||
@ -447,8 +449,9 @@ void msix_vector_unuse(PCIDevice *dev, unsigned vector)
|
||||
|
||||
void msix_unuse_all_vectors(PCIDevice *dev)
|
||||
{
|
||||
if (!(dev->cap_present & QEMU_PCI_CAP_MSIX))
|
||||
if (!msix_present(dev)) {
|
||||
return;
|
||||
}
|
||||
msix_free_irq_entries(dev);
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user