This structure will be used for legacy MSI as well and will become part of the KVM API. Signed-off-by: Jan Kiszka <jan.kiszka@xxxxxxxxxxx> --- hw/msix.c | 10 +++++----- hw/pci.h | 10 ++-------- kvm.h | 7 +++++++ 3 files changed, 14 insertions(+), 13 deletions(-) diff --git a/hw/msix.c b/hw/msix.c index 8b68a6a..42870c5 100644 --- a/hw/msix.c +++ b/hw/msix.c @@ -51,7 +51,7 @@ int msix_supported; static void kvm_msix_free(PCIDevice *dev) { int vector, changed = 0; - struct kvm_msix_message *kmm; + KVMMsiMessage *kmm; for (vector = 0; vector < dev->msix_entries_nr; ++vector) { if (dev->msix_entry_used[vector]) { @@ -66,7 +66,7 @@ static void kvm_msix_free(PCIDevice *dev) } static void kvm_msix_message_from_vector(PCIDevice *dev, unsigned vector, - struct kvm_msix_message *kmm) + KVMMsiMessage *kmm) { uint8_t *table_entry = dev->msix_table_page + vector * MSIX_ENTRY_SIZE; @@ -78,7 +78,7 @@ static void kvm_msix_message_from_vector(PCIDevice *dev, unsigned vector, static void kvm_msix_update(PCIDevice *dev, int vector, int was_masked, int is_masked) { - struct kvm_msix_message e = {}, *entry; + KVMMsiMessage e = {}, *entry; int mask_cleared = was_masked && !is_masked; /* It is only legal to change an entry when it is masked. Therefore, it is * enough to update the routing in kernel when mask is being cleared. */ @@ -114,7 +114,7 @@ static void kvm_msix_update(PCIDevice *dev, int vector, static int kvm_msix_add(PCIDevice *dev, unsigned vector) { - struct kvm_msix_message *kmm = dev->msix_irq_entries + vector; + KVMMsiMessage *kmm = dev->msix_irq_entries + vector; int r; if (!kvm_has_gsi_routing()) { @@ -147,7 +147,7 @@ static int kvm_msix_add(PCIDevice *dev, unsigned vector) static void kvm_msix_del(PCIDevice *dev, unsigned vector) { - struct kvm_msix_message *kmm; + KVMMsiMessage *kmm; if (dev->msix_entry_used[vector]) { return; diff --git a/hw/pci.h b/hw/pci.h index dd09494..dc5df17 100644 --- a/hw/pci.h +++ b/hw/pci.h @@ -5,6 +5,7 @@ #include "qobject.h" #include "qdev.h" +#include "kvm.h" /* PCI includes legacy ISA access. */ #include "isa.h" @@ -129,13 +130,6 @@ enum { typedef int (*msix_mask_notifier_func)(PCIDevice *, unsigned vector, int masked); -struct kvm_msix_message { - uint32_t gsi; - uint32_t addr_lo; - uint32_t addr_hi; - uint32_t data; -}; - struct PCIDevice { DeviceState qdev; /* PCI config space */ @@ -208,7 +202,7 @@ struct PCIDevice { * on the rest of the region. */ target_phys_addr_t msix_page_size; - struct kvm_msix_message *msix_irq_entries; + KVMMsiMessage *msix_irq_entries; msix_mask_notifier_func msix_mask_notifier; }; diff --git a/kvm.h b/kvm.h index b890b5d..8ece0b3 100644 --- a/kvm.h +++ b/kvm.h @@ -214,6 +214,13 @@ int kvm_set_irqfd(int gsi, int fd, bool assigned) int kvm_set_ioeventfd_pio_word(int fd, uint16_t adr, uint16_t val, bool assign); +typedef struct KVMMsiMessage { + uint32_t gsi; + uint32_t addr_lo; + uint32_t addr_hi; + uint32_t data; +} KVMMsiMessage; + int kvm_has_gsi_routing(void); int kvm_get_irq_route_gsi(void); int kvm_add_msix(uint32_t gsi, uint32_t addr_lo, -- 1.7.1 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html