From patchwork Fri Oct 30 12:21:08 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Isaku Yamahata X-Patchwork-Id: 37279 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [199.232.76.165]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 60664B7C22 for ; Fri, 30 Oct 2009 23:45:06 +1100 (EST) Received: from localhost ([127.0.0.1]:44959 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1N3qql-0002ne-0U for incoming@patchwork.ozlabs.org; Fri, 30 Oct 2009 08:45:03 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1N3qVp-0001um-SD for qemu-devel@nongnu.org; Fri, 30 Oct 2009 08:23:27 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1N3qVf-0001nK-Dl for qemu-devel@nongnu.org; Fri, 30 Oct 2009 08:23:21 -0400 Received: from [199.232.76.173] (port=52348 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1N3qVe-0001nF-Kk for qemu-devel@nongnu.org; Fri, 30 Oct 2009 08:23:14 -0400 Received: from mail.valinux.co.jp ([210.128.90.3]:34036) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1N3qVc-0002r9-SX for qemu-devel@nongnu.org; Fri, 30 Oct 2009 08:23:14 -0400 Received: from nm.local.valinux.co.jp (vagw.valinux.co.jp [210.128.90.14]) by mail.valinux.co.jp (Postfix) with ESMTP id 1C02A1805E; Fri, 30 Oct 2009 21:23:05 +0900 (JST) Received: from yamahata by nm.local.valinux.co.jp with local (Exim 4.69) (envelope-from ) id 1N3qTv-0006dL-OG; Fri, 30 Oct 2009 21:21:27 +0900 From: Isaku Yamahata To: qemu-devel@nongnu.org, mst@redhat.com Date: Fri, 30 Oct 2009 21:21:08 +0900 Message-Id: <1256905286-25435-15-git-send-email-yamahata@valinux.co.jp> X-Mailer: git-send-email 1.6.0.2 In-Reply-To: <1256905286-25435-1-git-send-email-yamahata@valinux.co.jp> References: <1256905286-25435-1-git-send-email-yamahata@valinux.co.jp> X-Virus-Scanned: clamav-milter 0.95.2 at va-mail.local.valinux.co.jp X-Virus-Status: Clean X-detected-operating-system: by monty-python.gnu.org: GNU/Linux 2.6 (newer, 3) Cc: yamahata@valinux.co.jp Subject: [Qemu-devel] [PATCH V6 14/32] pci: introduce pcibus_t to represent pci bus address/size instead of uint32_t X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org This patch is preliminary for 64 bit BAR support. Introduce dedicated type, pcibus_t, to represent pci bus address/size instead of uint32_t. Later this type will be changed to uint64_t. Signed-off-by: Isaku Yamahata Acked-by: Michael S. Tsirkin --- hw/ac97.c | 2 +- hw/cirrus_vga.c | 4 ++-- hw/e1000.c | 6 +++--- hw/eepro100.c | 4 ++-- hw/es1370.c | 2 +- hw/ide/cmd646.c | 4 ++-- hw/ide/piix.c | 2 +- hw/lsi53c895a.c | 6 +++--- hw/macio.c | 2 +- hw/msix.c | 2 +- hw/msix.h | 3 ++- hw/ne2000.c | 2 +- hw/openpic.c | 2 +- hw/pci.c | 6 +++--- hw/pci.h | 12 +++++++----- hw/pcnet.c | 4 ++-- hw/rtl8139.c | 4 ++-- hw/sun4u.c | 2 +- hw/usb-ohci.c | 2 +- hw/usb-uhci.c | 2 +- hw/vga-pci.c | 2 +- hw/virtio-pci.c | 2 +- hw/vmware_vga.c | 4 ++-- hw/wdt_i6300esb.c | 2 +- 24 files changed, 43 insertions(+), 40 deletions(-) diff --git a/hw/ac97.c b/hw/ac97.c index 534f899..f72c46d 100644 --- a/hw/ac97.c +++ b/hw/ac97.c @@ -1250,7 +1250,7 @@ static int ac97_load (QEMUFile *f, void *opaque, int version_id) } static void ac97_map (PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { AC97LinkState *s = DO_UPCAST (AC97LinkState, dev, pci_dev); PCIDevice *d = &s->dev; diff --git a/hw/cirrus_vga.c b/hw/cirrus_vga.c index 0eb058d..ef72c62 100644 --- a/hw/cirrus_vga.c +++ b/hw/cirrus_vga.c @@ -3172,7 +3172,7 @@ void isa_cirrus_vga_init(void) ***************************************/ static void cirrus_pci_lfb_map(PCIDevice *d, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { CirrusVGAState *s = &DO_UPCAST(PCICirrusVGAState, dev, d)->cirrus_vga; @@ -3193,7 +3193,7 @@ static void cirrus_pci_lfb_map(PCIDevice *d, int region_num, } static void cirrus_pci_mmio_map(PCIDevice *d, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { CirrusVGAState *s = &DO_UPCAST(PCICirrusVGAState, dev, d)->cirrus_vga; diff --git a/hw/e1000.c b/hw/e1000.c index ebaadc9..9ae1e87 100644 --- a/hw/e1000.c +++ b/hw/e1000.c @@ -144,8 +144,8 @@ static const char phy_regcap[0x20] = { }; static void -ioport_map(PCIDevice *pci_dev, int region_num, uint32_t addr, - uint32_t size, int type) +ioport_map(PCIDevice *pci_dev, int region_num, pcibus_t addr, + pcibus_t size, int type) { DBGOUT(IO, "e1000_ioport_map addr=0x%04x size=0x%08x\n", addr, size); } @@ -1010,7 +1010,7 @@ static CPUReadMemoryFunc * const e1000_mmio_read[] = { static void e1000_mmio_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { E1000State *d = DO_UPCAST(E1000State, dev, pci_dev); int i; diff --git a/hw/eepro100.c b/hw/eepro100.c index 829731e..55e7717 100644 --- a/hw/eepro100.c +++ b/hw/eepro100.c @@ -1384,7 +1384,7 @@ static void ioport_write4(void *opaque, uint32_t addr, uint32_t val) /* PCI EEPRO100 definitions */ static void pci_map(PCIDevice * pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { EEPRO100State *s = DO_UPCAST(EEPRO100State, dev, pci_dev); @@ -1463,7 +1463,7 @@ static CPUReadMemoryFunc * const pci_mmio_read[] = { }; static void pci_mmio_map(PCIDevice * pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { EEPRO100State *s = DO_UPCAST(EEPRO100State, dev, pci_dev); diff --git a/hw/es1370.c b/hw/es1370.c index f048951..10da250 100644 --- a/hw/es1370.c +++ b/hw/es1370.c @@ -907,7 +907,7 @@ static void es1370_adc_callback (void *opaque, int avail) } static void es1370_map (PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { ES1370State *s = DO_UPCAST (ES1370State, dev, pci_dev); diff --git a/hw/ide/cmd646.c b/hw/ide/cmd646.c index b6e7866..b1907a7 100644 --- a/hw/ide/cmd646.c +++ b/hw/ide/cmd646.c @@ -45,7 +45,7 @@ static void cmd646_update_irq(PCIIDEState *d); static void ide_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { PCIIDEState *d = DO_UPCAST(PCIIDEState, dev, pci_dev); IDEBus *bus; @@ -136,7 +136,7 @@ static void bmdma_writeb(void *opaque, uint32_t addr, uint32_t val) } static void bmdma_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { PCIIDEState *d = DO_UPCAST(PCIIDEState, dev, pci_dev); int i; diff --git a/hw/ide/piix.c b/hw/ide/piix.c index 5652c99..17910dc 100644 --- a/hw/ide/piix.c +++ b/hw/ide/piix.c @@ -69,7 +69,7 @@ static void bmdma_writeb(void *opaque, uint32_t addr, uint32_t val) } static void bmdma_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { PCIIDEState *d = DO_UPCAST(PCIIDEState, dev, pci_dev); int i; diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index 74f9bbf..a1e48ca 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -1926,7 +1926,7 @@ static void lsi_io_writel(void *opaque, uint32_t addr, uint32_t val) } static void lsi_io_mapfunc(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { LSIState *s = DO_UPCAST(LSIState, dev, pci_dev); @@ -1941,7 +1941,7 @@ static void lsi_io_mapfunc(PCIDevice *pci_dev, int region_num, } static void lsi_ram_mapfunc(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { LSIState *s = DO_UPCAST(LSIState, dev, pci_dev); @@ -1951,7 +1951,7 @@ static void lsi_ram_mapfunc(PCIDevice *pci_dev, int region_num, } static void lsi_mmio_mapfunc(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { LSIState *s = DO_UPCAST(LSIState, dev, pci_dev); diff --git a/hw/macio.c b/hw/macio.c index 0703abc..e92e82a 100644 --- a/hw/macio.c +++ b/hw/macio.c @@ -40,7 +40,7 @@ struct macio_state_t { }; static void macio_map (PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { macio_state_t *macio_state; int i; diff --git a/hw/msix.c b/hw/msix.c index b0dea91..ee565bc 100644 --- a/hw/msix.c +++ b/hw/msix.c @@ -203,7 +203,7 @@ static CPUReadMemoryFunc * const msix_mmio_read[] = { /* Should be called from device's map method. */ void msix_mmio_map(PCIDevice *d, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { uint8_t *config = d->config + d->msix_cap; uint32_t table = pci_get_long(config + MSIX_TABLE_OFFSET); diff --git a/hw/msix.h b/hw/msix.h index 3427778..39fa568 100644 --- a/hw/msix.h +++ b/hw/msix.h @@ -2,6 +2,7 @@ #define QEMU_MSIX_H #include "qemu-common.h" +#include "pci.h" int msix_init(PCIDevice *pdev, unsigned short nentries, unsigned bar_nr, unsigned bar_size); @@ -10,7 +11,7 @@ void msix_write_config(PCIDevice *pci_dev, uint32_t address, uint32_t val, int len); void msix_mmio_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type); + pcibus_t addr, pcibus_t size, int type); int msix_uninit(PCIDevice *d); diff --git a/hw/ne2000.c b/hw/ne2000.c index 5fe3883..63efc3a 100644 --- a/hw/ne2000.c +++ b/hw/ne2000.c @@ -679,7 +679,7 @@ const VMStateDescription vmstate_pci_ne2000 = { /* PCI NE2000 definitions */ static void ne2000_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { PCINE2000State *d = DO_UPCAST(PCINE2000State, dev, pci_dev); NE2000State *s = &d->ne2000; diff --git a/hw/openpic.c b/hw/openpic.c index 5de44a3..f2d2937 100644 --- a/hw/openpic.c +++ b/hw/openpic.c @@ -1026,7 +1026,7 @@ static CPUReadMemoryFunc * const openpic_read[] = { }; static void openpic_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { openpic_t *opp; diff --git a/hw/pci.c b/hw/pci.c index 3315820..5114ed3 100644 --- a/hw/pci.c +++ b/hw/pci.c @@ -455,12 +455,12 @@ static int pci_unregister_device(DeviceState *dev) } void pci_register_bar(PCIDevice *pci_dev, int region_num, - uint32_t size, int type, + pcibus_t size, int type, PCIMapIORegionFunc *map_func) { PCIIORegion *r; uint32_t addr; - uint32_t wmask; + pcibus_t wmask; if ((unsigned int)region_num >= PCI_NUM_REGIONS) return; @@ -492,7 +492,7 @@ static void pci_update_mappings(PCIDevice *d) { PCIIORegion *r; int cmd, i; - uint32_t last_addr, new_addr; + pcibus_t last_addr, new_addr; cmd = pci_get_word(d->config + PCI_COMMAND); for(i = 0; i < PCI_NUM_REGIONS; i++) { diff --git a/hw/pci.h b/hw/pci.h index e9ba6d3..9509a6b 100644 --- a/hw/pci.h +++ b/hw/pci.h @@ -71,18 +71,20 @@ extern target_phys_addr_t pci_mem_base; #define PCI_DEVICE_ID_VIRTIO_BALLOON 0x1002 #define PCI_DEVICE_ID_VIRTIO_CONSOLE 0x1003 +typedef uint32_t pcibus_t; + typedef void PCIConfigWriteFunc(PCIDevice *pci_dev, uint32_t address, uint32_t data, int len); typedef uint32_t PCIConfigReadFunc(PCIDevice *pci_dev, uint32_t address, int len); typedef void PCIMapIORegionFunc(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type); + pcibus_t addr, pcibus_t size, int type); typedef int PCIUnregisterFunc(PCIDevice *pci_dev); typedef struct PCIIORegion { - uint32_t addr; /* current PCI mapping address. -1 means not mapped */ -#define PCI_BAR_UNMAPPED (~(uint32_t)0) - uint32_t size; + pcibus_t addr; /* current PCI mapping address. -1 means not mapped */ +#define PCI_BAR_UNMAPPED (~(pcibus_t)0) + pcibus_t size; uint8_t type; PCIMapIORegionFunc *map_func; } PCIIORegion; @@ -224,7 +226,7 @@ PCIDevice *pci_register_device(PCIBus *bus, const char *name, PCIConfigWriteFunc *config_write); void pci_register_bar(PCIDevice *pci_dev, int region_num, - uint32_t size, int type, + pcibus_t size, int type, PCIMapIORegionFunc *map_func); int pci_add_capability(PCIDevice *pci_dev, uint8_t cap_id, uint8_t cap_size); diff --git a/hw/pcnet.c b/hw/pcnet.c index 77e59ca..2716f5f 100644 --- a/hw/pcnet.c +++ b/hw/pcnet.c @@ -1727,7 +1727,7 @@ static uint32_t pcnet_ioport_readl(void *opaque, uint32_t addr) } static void pcnet_ioport_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { PCNetState *d = &DO_UPCAST(PCIPCNetState, pci_dev, pci_dev)->state; @@ -1921,7 +1921,7 @@ static CPUReadMemoryFunc * const pcnet_mmio_read[] = { }; static void pcnet_mmio_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { PCIPCNetState *d = DO_UPCAST(PCIPCNetState, pci_dev, pci_dev); diff --git a/hw/rtl8139.c b/hw/rtl8139.c index becfca8..8b6c714 100644 --- a/hw/rtl8139.c +++ b/hw/rtl8139.c @@ -3192,7 +3192,7 @@ static const VMStateDescription vmstate_rtl8139 = { /* PCI RTL8139 definitions */ static void rtl8139_mmio_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { RTL8139State *s = DO_UPCAST(RTL8139State, dev, pci_dev); @@ -3200,7 +3200,7 @@ static void rtl8139_mmio_map(PCIDevice *pci_dev, int region_num, } static void rtl8139_ioport_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { RTL8139State *s = DO_UPCAST(RTL8139State, dev, pci_dev); diff --git a/hw/sun4u.c b/hw/sun4u.c index 6a7c698..e93393e 100644 --- a/hw/sun4u.c +++ b/hw/sun4u.c @@ -338,7 +338,7 @@ void cpu_tick_set_limit(void *opaque, uint64_t limit) } static void ebus_mmio_mapfunc(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { DPRINTF("Mapping region %d registers at %08x\n", region_num, addr); switch (region_num) { diff --git a/hw/usb-ohci.c b/hw/usb-ohci.c index d498401..567fb17 100644 --- a/hw/usb-ohci.c +++ b/hw/usb-ohci.c @@ -1707,7 +1707,7 @@ typedef struct { } OHCIPCIState; static void ohci_mapfunc(PCIDevice *pci_dev, int i, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { OHCIPCIState *ohci = DO_UPCAST(OHCIPCIState, pci_dev, pci_dev); cpu_register_physical_memory(addr, size, ohci->state.mem); diff --git a/hw/usb-uhci.c b/hw/usb-uhci.c index a32e409..86f12ce 100644 --- a/hw/usb-uhci.c +++ b/hw/usb-uhci.c @@ -1047,7 +1047,7 @@ static void uhci_frame_timer(void *opaque) } static void uhci_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { UHCIState *s = (UHCIState *)pci_dev; diff --git a/hw/vga-pci.c b/hw/vga-pci.c index 91449e0..2de8d8f 100644 --- a/hw/vga-pci.c +++ b/hw/vga-pci.c @@ -47,7 +47,7 @@ static const VMStateDescription vmstate_vga_pci = { }; static void vga_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { PCIVGAState *d = (PCIVGAState *)pci_dev; VGACommonState *s = &d->vga; diff --git a/hw/virtio-pci.c b/hw/virtio-pci.c index 1b0702a..aebcf9d 100644 --- a/hw/virtio-pci.c +++ b/hw/virtio-pci.c @@ -344,7 +344,7 @@ static void virtio_pci_config_writel(void *opaque, uint32_t addr, uint32_t val) } static void virtio_map(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { VirtIOPCIProxy *proxy = container_of(pci_dev, VirtIOPCIProxy, pci_dev); VirtIODevice *vdev = proxy->vdev; diff --git a/hw/vmware_vga.c b/hw/vmware_vga.c index baafedd..33e876e 100644 --- a/hw/vmware_vga.c +++ b/hw/vmware_vga.c @@ -1126,7 +1126,7 @@ static void vmsvga_init(struct vmsvga_state_s *s, int vga_ram_size) } static void pci_vmsvga_map_ioport(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { struct pci_vmsvga_state_s *d = (struct pci_vmsvga_state_s *) pci_dev; struct vmsvga_state_s *s = &d->chip; @@ -1146,7 +1146,7 @@ static void pci_vmsvga_map_ioport(PCIDevice *pci_dev, int region_num, } static void pci_vmsvga_map_mem(PCIDevice *pci_dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { struct pci_vmsvga_state_s *d = (struct pci_vmsvga_state_s *) pci_dev; struct vmsvga_state_s *s = &d->chip; diff --git a/hw/wdt_i6300esb.c b/hw/wdt_i6300esb.c index 75f1fc6..6da396a 100644 --- a/hw/wdt_i6300esb.c +++ b/hw/wdt_i6300esb.c @@ -343,7 +343,7 @@ static void i6300esb_mem_writel(void *vp, target_phys_addr_t addr, uint32_t val) } static void i6300esb_map(PCIDevice *dev, int region_num, - uint32_t addr, uint32_t size, int type) + pcibus_t addr, pcibus_t size, int type) { static CPUReadMemoryFunc * const mem_read[3] = { i6300esb_mem_readb,