From patchwork Fri Oct 30 12:21:12 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Isaku Yamahata X-Patchwork-Id: 37282 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 EC735B7C26 for ; Fri, 30 Oct 2009 23:56:10 +1100 (EST) Received: from localhost ([127.0.0.1]:54470 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1N3r1T-0000pI-Gw for incoming@patchwork.ozlabs.org; Fri, 30 Oct 2009 08:56:07 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1N3qVr-0001uq-KY 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 1N3qVe-0001nO-Ra for qemu-devel@nongnu.org; Fri, 30 Oct 2009 08:23:20 -0400 Received: from [199.232.76.173] (port=52349 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1N3qVe-0001nG-M2 for qemu-devel@nongnu.org; Fri, 30 Oct 2009 08:23:14 -0400 Received: from mail.valinux.co.jp ([210.128.90.3]:34043) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1N3qVd-0002rV-DP 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 4105318062; 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-0006dX-Rx; 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:12 +0900 Message-Id: <1256905286-25435-19-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 18/32] pci: remove bus_num member from struct PCIBus. 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 Since It can be retrieved from pci configuration space, the member is unnecessary. Signed-off-by: Isaku Yamahata Acked-by: Michael S. Tsirkin Acked-by: Michael S. Tsirkin --- hw/pci.c | 21 ++++++++++----------- 1 files changed, 10 insertions(+), 11 deletions(-) diff --git a/hw/pci.c b/hw/pci.c index 7da3db9..a75d981 100644 --- a/hw/pci.c +++ b/hw/pci.c @@ -36,7 +36,6 @@ struct PCIBus { BusState qbus; - int bus_num; int devfn_min; pci_set_irq_fn set_irq; pci_map_irq_fn map_irq; @@ -192,7 +191,9 @@ static void pci_register_secondary_bus(PCIBus *bus, int pci_bus_num(PCIBus *s) { - return s->bus_num; + if (!s->parent_dev) + return 0; /* pci host bridge */ + return s->parent_dev->config[PCI_SECONDARY_BUS]; } static int get_pci_config_device(QEMUFile *f, void *pv, size_t size) @@ -624,7 +625,7 @@ void pci_data_write(void *opaque, uint32_t addr, uint32_t val, int len) addr, val, len); #endif bus_num = (addr >> 16) & 0xff; - while (s && s->bus_num != bus_num) + while (s && pci_bus_num(s) != bus_num) s = s->next; if (!s) return; @@ -645,7 +646,7 @@ uint32_t pci_data_read(void *opaque, uint32_t addr, int len) uint32_t val; bus_num = (addr >> 16) & 0xff; - while (s && s->bus_num != bus_num) + while (s && pci_bus_num(s) != bus_num) s= s->next; if (!s) goto fail; @@ -760,7 +761,8 @@ static void pci_info_device(PCIDevice *d) const pci_class_desc *desc; monitor_printf(mon, " Bus %2d, device %3d, function %d:\n", - d->bus->bus_num, PCI_SLOT(d->devfn), PCI_FUNC(d->devfn)); + pci_bus_num(d->bus), + PCI_SLOT(d->devfn), PCI_FUNC(d->devfn)); class = pci_get_word(d->config + PCI_CLASS_DEVICE); monitor_printf(mon, " "); desc = pci_class_descriptions; @@ -816,7 +818,7 @@ void pci_for_each_device(int bus_num, void (*fn)(PCIDevice *d)) PCIDevice *d; int devfn; - while (bus && bus->bus_num != bus_num) + while (bus && pci_bus_num(bus) != bus_num) bus = bus->next; if (bus) { for(devfn = 0; devfn < 256; devfn++) { @@ -913,17 +915,14 @@ typedef struct { static void pci_bridge_write_config(PCIDevice *d, uint32_t address, uint32_t val, int len) { - PCIBridge *s = (PCIBridge *)d; - pci_default_write_config(d, address, val, len); - s->bus.bus_num = d->config[PCI_SECONDARY_BUS]; } PCIBus *pci_find_bus(int bus_num) { PCIBus *bus = first_bus; - while (bus && bus->bus_num != bus_num) + while (bus && pci_bus_num(bus) != bus_num) bus = bus->next; return bus; @@ -1149,7 +1148,7 @@ static void pcibus_dev_print(Monitor *mon, DeviceState *dev, int indent) monitor_printf(mon, "%*sclass %s, addr %02x:%02x.%x, " "pci id %04x:%04x (sub %04x:%04x)\n", indent, "", ctxt, - d->bus->bus_num, PCI_SLOT(d->devfn), PCI_FUNC(d->devfn), + pci_bus_num(d->bus), PCI_SLOT(d->devfn), PCI_FUNC(d->devfn), pci_get_word(d->config + PCI_VENDOR_ID), pci_get_word(d->config + PCI_DEVICE_ID), pci_get_word(d->config + PCI_SUBSYSTEM_VENDOR_ID),