From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:60973) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RyRIy-0005J3-72 for qemu-devel@nongnu.org; Fri, 17 Feb 2012 12:09:13 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RyRIk-00065D-LC for qemu-devel@nongnu.org; Fri, 17 Feb 2012 12:09:04 -0500 Received: from smtp02.citrix.com ([66.165.176.63]:55066) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RyRIk-00064h-D4 for qemu-devel@nongnu.org; Fri, 17 Feb 2012 12:08:54 -0500 From: Anthony PERARD Date: Fri, 17 Feb 2012 17:08:40 +0000 Message-ID: <1329498525-8454-7-git-send-email-anthony.perard@citrix.com> In-Reply-To: <1329498525-8454-1-git-send-email-anthony.perard@citrix.com> References: <1329498525-8454-1-git-send-email-anthony.perard@citrix.com> MIME-Version: 1.0 Content-Type: text/plain Subject: [Qemu-devel] [PATCH V7 06/11] pci.c: Add pci_check_bar_overlap List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: QEMU-devel Cc: Anthony PERARD , Yuji Shimada , Xen Devel , "Michael S . Tsirkin" , Stefano Stabellini From: Yuji Shimada This function helps Xen PCI Passthrough device to check for overlap. Signed-off-by: Yuji Shimada Signed-off-by: Anthony PERARD --- hw/pci.c | 47 +++++++++++++++++++++++++++++++++++++++++++++++ hw/pci.h | 3 +++ 2 files changed, 50 insertions(+), 0 deletions(-) diff --git a/hw/pci.c b/hw/pci.c index 678a8c1..75d6529 100644 --- a/hw/pci.c +++ b/hw/pci.c @@ -1985,6 +1985,53 @@ MemoryRegion *pci_address_space_io(PCIDevice *dev) return dev->bus->address_space_io; } +int pci_check_bar_overlap(PCIDevice *dev, + pcibus_t addr, pcibus_t size, uint8_t type) +{ + PCIBus *bus = dev->bus; + PCIDevice *devices = NULL; + PCIIORegion *r; + int i, j; + int rc = 0; + + /* check Overlapped to Base Address */ + for (i = 0; i < ARRAY_SIZE(bus->devices); i++) { + devices = bus->devices[i]; + if (!devices) { + continue; + } + + /* skip itself */ + if (devices->devfn == dev->devfn) { + continue; + } + + for (j = 0; j < PCI_NUM_REGIONS; j++) { + r = &devices->io_regions[j]; + + /* skip different resource type, but don't skip when + * prefetch and non-prefetch memory are compared. + */ + if (type != r->type) { + if (type & PCI_BASE_ADDRESS_SPACE_IO || + r->type & PCI_BASE_ADDRESS_SPACE_IO) { + continue; + } + } + + if ((addr < (r->addr + r->size)) && ((addr + size) > r->addr)) { + printf("Overlapped to device[%02x:%02x.%x][Region:%d]" + "[Address:%"PRIx64"h][Size:%"PRIx64"h]\n", + pci_bus_num(bus), PCI_SLOT(devices->devfn), + PCI_FUNC(devices->devfn), j, r->addr, r->size); + rc = 1; + } + } + } + + return rc; +} + static void pci_device_class_init(ObjectClass *klass, void *data) { DeviceClass *k = DEVICE_CLASS(klass); diff --git a/hw/pci.h b/hw/pci.h index 33b0b18..f05fda5 100644 --- a/hw/pci.h +++ b/hw/pci.h @@ -566,4 +566,7 @@ extern const VMStateDescription vmstate_pci_device; .offset = vmstate_offset_pointer(_state, _field, PCIDevice), \ } +int pci_check_bar_overlap(PCIDevice *dev, + pcibus_t addr, pcibus_t size, uint8_t type); + #endif -- Anthony PERARD