From 5e520a7d500ec2569d22d80f9ef4272a34cb3c80 Mon Sep 17 00:00:00 2001 From: Blue Swirl Date: Sun, 20 Sep 2009 15:35:55 +0000 Subject: [PATCH] Compile msix only once Get page size in device init. Signed-off-by: Blue Swirl --- Makefile.hw | 2 ++ Makefile.target | 2 +- hw/msix.c | 50 +++++++++++++++++++++++++------------------------- hw/msix.h | 5 +++-- hw/pci.h | 6 ++++++ hw/virtio-pci.c | 3 ++- 6 files changed, 39 insertions(+), 29 deletions(-) diff --git a/Makefile.hw b/Makefile.hw index cb9c9a4a85..648906bed2 100644 --- a/Makefile.hw +++ b/Makefile.hw @@ -22,6 +22,8 @@ obj-y += m48t59.o escc.o # PCI watchdog devices obj-y += wdt_i6300esb.o +obj-y += msix.o + # SCSI layer obj-y += lsi53c895a.o esp.o diff --git a/Makefile.target b/Makefile.target index bc3998a1de..39aac73a94 100644 --- a/Makefile.target +++ b/Makefile.target @@ -156,7 +156,7 @@ endif ifdef CONFIG_SOFTMMU obj-y = vl.o monitor.o pci.o isa_mmio.o machine.o \ - gdbstub.o gdbstub-xml.o msix.o ioport.o + gdbstub.o gdbstub-xml.o ioport.o # virtio has to be here due to weird dependency between PCI and virtio-net. # need to fix this properly obj-y += virtio-blk.o virtio-balloon.o virtio-net.o virtio-console.o virtio-pci.o diff --git a/hw/msix.c b/hw/msix.c index c07d7a1525..3782994ca0 100644 --- a/hw/msix.c +++ b/hw/msix.c @@ -38,15 +38,6 @@ #define MSIX_VECTOR_CTRL 12 #define MSIX_ENTRY_SIZE 16 #define MSIX_VECTOR_MASK 0x1 - -/* How much space does an MSIX table need. */ -/* The spec requires giving the table structure - * a 4K aligned region all by itself. Align it to - * target pages so that drivers can do passthrough - * on the rest of the region. */ -#define MSIX_PAGE_SIZE TARGET_PAGE_ALIGN(0x1000) -/* Reserve second half of the page for pending bits */ -#define MSIX_PAGE_PENDING (MSIX_PAGE_SIZE / 2) #define MSIX_MAX_ENTRIES 32 @@ -62,6 +53,12 @@ /* Flag for interrupt controller to declare MSI-X support */ int msix_supported; +/* Reserve second half of the page for pending bits */ +static int msix_page_pending(PCIDevice *d) +{ + return (d->msix_page_size / 2); +} + /* Add MSI-X capability to the config space for the device. */ /* Given a bar and its size, add MSI-X table on top of it * and fill MSI-X capability in the config space. @@ -80,11 +77,11 @@ static int msix_add_config(struct PCIDevice *pdev, unsigned short nentries, return -ENOSPC; /* Add space for MSI-X structures */ - if (!bar_size) - new_size = MSIX_PAGE_SIZE; - else if (bar_size < MSIX_PAGE_SIZE) { - bar_size = MSIX_PAGE_SIZE; - new_size = MSIX_PAGE_SIZE * 2; + if (!bar_size) { + new_size = pdev->msix_page_size; + } else if (bar_size < pdev->msix_page_size) { + bar_size = pdev->msix_page_size; + new_size = pdev->msix_page_size * 2; } else new_size = bar_size * 2; @@ -98,8 +95,8 @@ static int msix_add_config(struct PCIDevice *pdev, unsigned short nentries, /* Table on top of BAR */ pci_set_long(config + MSIX_TABLE_OFFSET, bar_size | bar_nr); /* Pending bits on top of that */ - pci_set_long(config + MSIX_PBA_OFFSET, (bar_size + MSIX_PAGE_PENDING) | - bar_nr); + pci_set_long(config + MSIX_PBA_OFFSET, (bar_size + msix_page_pending(pdev)) + | bar_nr); pdev->msix_cap = config_offset; /* Make flags bit writeable. */ pdev->wmask[config_offset + MSIX_ENABLE_OFFSET] |= MSIX_ENABLE_MASK; @@ -129,7 +126,7 @@ void msix_write_config(PCIDevice *dev, uint32_t addr, static uint32_t msix_mmio_readl(void *opaque, target_phys_addr_t addr) { PCIDevice *dev = opaque; - unsigned int offset = addr & (MSIX_PAGE_SIZE - 1); + unsigned int offset = addr & (dev->msix_page_size - 1); void *page = dev->msix_table_page; uint32_t val = 0; @@ -151,7 +148,7 @@ static uint8_t msix_pending_mask(int vector) static uint8_t *msix_pending_byte(PCIDevice *dev, int vector) { - return dev->msix_table_page + MSIX_PAGE_PENDING + vector / 8; + return dev->msix_table_page + msix_page_pending(dev) + vector / 8; } static int msix_is_pending(PCIDevice *dev, int vector) @@ -179,7 +176,7 @@ static void msix_mmio_writel(void *opaque, target_phys_addr_t addr, uint32_t val) { PCIDevice *dev = opaque; - unsigned int offset = addr & (MSIX_PAGE_SIZE - 1); + unsigned int offset = addr & (dev->msix_page_size - 1); int vector = offset / MSIX_ENTRY_SIZE; memcpy(dev->msix_table_page + offset, &val, 4); if (!msix_is_masked(dev, vector) && msix_is_pending(dev, vector)) { @@ -208,7 +205,7 @@ void msix_mmio_map(PCIDevice *d, int region_num, { uint8_t *config = d->config + d->msix_cap; uint32_t table = pci_get_long(config + MSIX_TABLE_OFFSET); - uint32_t offset = table & ~(MSIX_PAGE_SIZE - 1); + uint32_t offset = table & ~(d->msix_page_size - 1); /* TODO: for assigned devices, we'll want to make it possible to map * pending bits separately in case they are in a separate bar. */ int table_bir = table & PCI_MSIX_FLAGS_BIRMASK; @@ -224,7 +221,7 @@ void msix_mmio_map(PCIDevice *d, int region_num, /* Initialize the MSI-X structures. Note: if MSI-X is supported, BAR size is * modified, it should be retrieved with msix_bar_size. */ int msix_init(struct PCIDevice *dev, unsigned short nentries, - unsigned bar_nr, unsigned bar_size) + unsigned bar_nr, unsigned bar_size, target_phys_addr_t page_size) { int ret; /* Nothing to do if MSI is not supported by interrupt controller */ @@ -237,7 +234,8 @@ int msix_init(struct PCIDevice *dev, unsigned short nentries, dev->msix_entry_used = qemu_mallocz(MSIX_MAX_ENTRIES * sizeof *dev->msix_entry_used); - dev->msix_table_page = qemu_mallocz(MSIX_PAGE_SIZE); + dev->msix_page_size = page_size; + dev->msix_table_page = qemu_mallocz(dev->msix_page_size); dev->msix_mmio_index = cpu_register_io_memory(msix_mmio_read, msix_mmio_write, dev); @@ -292,7 +290,8 @@ void msix_save(PCIDevice *dev, QEMUFile *f) } qemu_put_buffer(f, dev->msix_table_page, n * MSIX_ENTRY_SIZE); - qemu_put_buffer(f, dev->msix_table_page + MSIX_PAGE_PENDING, (n + 7) / 8); + qemu_put_buffer(f, dev->msix_table_page + msix_page_pending(dev), + (n + 7) / 8); } /* Should be called after restoring the config space. */ @@ -306,7 +305,8 @@ void msix_load(PCIDevice *dev, QEMUFile *f) msix_free_irq_entries(dev); qemu_get_buffer(f, dev->msix_table_page, n * MSIX_ENTRY_SIZE); - qemu_get_buffer(f, dev->msix_table_page + MSIX_PAGE_PENDING, (n + 7) / 8); + qemu_get_buffer(f, dev->msix_table_page + msix_page_pending(dev), + (n + 7) / 8); } /* Does device support MSI-X? */ @@ -356,7 +356,7 @@ void msix_reset(PCIDevice *dev) return; msix_free_irq_entries(dev); dev->config[dev->msix_cap + MSIX_ENABLE_OFFSET] &= MSIX_ENABLE_MASK; - memset(dev->msix_table_page, 0, MSIX_PAGE_SIZE); + memset(dev->msix_table_page, 0, dev->msix_page_size); } /* PCI spec suggests that devices make it possible for software to configure diff --git a/hw/msix.h b/hw/msix.h index 3427778dc1..9367ba38f8 100644 --- a/hw/msix.h +++ b/hw/msix.h @@ -3,8 +3,9 @@ #include "qemu-common.h" -int msix_init(PCIDevice *pdev, unsigned short nentries, - unsigned bar_nr, unsigned bar_size); +int msix_init(struct PCIDevice *dev, unsigned short nentries, + unsigned bar_nr, unsigned bar_size, + target_phys_addr_t page_size); void msix_write_config(PCIDevice *pci_dev, uint32_t address, uint32_t val, int len); diff --git a/hw/pci.h b/hw/pci.h index e7bf33aef0..caba5c8518 100644 --- a/hw/pci.h +++ b/hw/pci.h @@ -212,6 +212,12 @@ struct PCIDevice { uint32_t msix_bar_size; /* Version id needed for VMState */ int32_t version_id; + /* How much space does an MSIX table need. */ + /* The spec requires giving the table structure + * a 4K aligned region all by itself. Align it to + * target pages so that drivers can do passthrough + * on the rest of the region. */ + target_phys_addr_t msix_page_size; }; PCIDevice *pci_register_device(PCIBus *bus, const char *name, diff --git a/hw/virtio-pci.c b/hw/virtio-pci.c index f7a51ff761..bd5a7c4d3a 100644 --- a/hw/virtio-pci.c +++ b/hw/virtio-pci.c @@ -411,7 +411,8 @@ static void virtio_init_pci(VirtIOPCIProxy *proxy, VirtIODevice *vdev, config[0x3d] = 1; - if (vdev->nvectors && !msix_init(&proxy->pci_dev, vdev->nvectors, 1, 0)) { + if (vdev->nvectors && !msix_init(&proxy->pci_dev, vdev->nvectors, 1, 0, + TARGET_PAGE_SIZE)) { pci_register_bar(&proxy->pci_dev, 1, msix_bar_size(&proxy->pci_dev), PCI_ADDRESS_SPACE_MEM, -- 2.11.0