From patchwork Tue Jun 15 22:38:29 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jan Kiszka X-Patchwork-Id: 55810 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 CA3AFB7D89 for ; Wed, 16 Jun 2010 08:46:38 +1000 (EST) Received: from localhost ([127.0.0.1]:48863 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OOetv-00036U-Ik for incoming@patchwork.ozlabs.org; Tue, 15 Jun 2010 18:46:35 -0400 Received: from [140.186.70.92] (port=33358 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OOemk-0007t5-G2 for qemu-devel@nongnu.org; Tue, 15 Jun 2010 18:39:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OOemh-0005WB-Aq for qemu-devel@nongnu.org; Tue, 15 Jun 2010 18:39:10 -0400 Received: from fmmailgate03.web.de ([217.72.192.234]:51431) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OOemg-0005Vj-PQ for qemu-devel@nongnu.org; Tue, 15 Jun 2010 18:39:07 -0400 Received: from smtp04.web.de ( [172.20.0.225]) by fmmailgate03.web.de (Postfix) with ESMTP id 2D60E156935BB; Wed, 16 Jun 2010 00:39:06 +0200 (CEST) Received: from [92.74.52.35] (helo=localhost.localdomain) by smtp04.web.de with asmtp (TLSv1:AES256-SHA:256) (WEB.DE 4.110 #4) id 1OOemf-0003C9-03; Wed, 16 Jun 2010 00:39:06 +0200 From: Jan Kiszka To: qemu-devel@nongnu.org, Anthony Liguori Date: Wed, 16 Jun 2010 00:38:29 +0200 Message-Id: <9349cfe4d0877a0d9e5b0948c5b73ac800dd51f5.1276641524.git.jan.kiszka@web.de> X-Mailer: git-send-email 1.6.0.2 In-Reply-To: References: In-Reply-To: References: X-Sender: jan.kiszka@web.de X-Provags-ID: V01U2FsdGVkX19rKGGlQQEc6PfHrAe5VA8/quXYxtuLK3eYEyQ7 3wWhF+/JyhuMjH58gKJvgfdAoORwdugLWLawCwZITqTuUkLFxi f9TgMfoRE= X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.4-2.6 Cc: Juan Quintela , Jan Kiszka , Markus Armbruster , Luiz Capitulino , Blue Swirl , Avi Kivity Subject: [Qemu-devel] [PATCH v4 05/23] qdev: Convert device and bus lists to QTAILQ 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 From: Jan Kiszka Cosmetic change to align the instance number assignment with bus ordering. The current ordering due to QLIST_INSERT_HEAD is a bit annoying when you dump the qtree or address devices via 'driver.instance'. Signed-off-by: Jan Kiszka --- hw/acpi_piix4.c | 2 +- hw/i2c.c | 2 +- hw/pci-hotplug.c | 2 +- hw/qdev.c | 41 +++++++++++++++++++++-------------------- hw/qdev.h | 8 ++++---- hw/ssi.c | 6 +++--- 6 files changed, 31 insertions(+), 30 deletions(-) diff --git a/hw/acpi_piix4.c b/hw/acpi_piix4.c index 8d1a628..38e8289 100644 --- a/hw/acpi_piix4.c +++ b/hw/acpi_piix4.c @@ -564,7 +564,7 @@ static void pciej_write(void *opaque, uint32_t addr, uint32_t val) PCIDevice *dev; int slot = ffs(val) - 1; - QLIST_FOREACH_SAFE(qdev, &bus->children, sibling, next) { + QTAILQ_FOREACH_SAFE(qdev, &bus->children, sibling, next) { dev = DO_UPCAST(PCIDevice, qdev, qdev); if (PCI_SLOT(dev->devfn) == slot) { qdev_free(qdev); diff --git a/hw/i2c.c b/hw/i2c.c index bee8e88..61ab6fa 100644 --- a/hw/i2c.c +++ b/hw/i2c.c @@ -84,7 +84,7 @@ int i2c_start_transfer(i2c_bus *bus, uint8_t address, int recv) DeviceState *qdev; i2c_slave *slave = NULL; - QLIST_FOREACH(qdev, &bus->qbus.children, sibling) { + QTAILQ_FOREACH(qdev, &bus->qbus.children, sibling) { i2c_slave *candidate = I2C_SLAVE_FROM_QDEV(qdev); if (candidate->address == address) { slave = candidate; diff --git a/hw/pci-hotplug.c b/hw/pci-hotplug.c index c39e640..2793269 100644 --- a/hw/pci-hotplug.c +++ b/hw/pci-hotplug.c @@ -74,7 +74,7 @@ static int scsi_hot_add(Monitor *mon, DeviceState *adapter, SCSIBus *scsibus; SCSIDevice *scsidev; - scsibus = DO_UPCAST(SCSIBus, qbus, QLIST_FIRST(&adapter->child_bus)); + scsibus = DO_UPCAST(SCSIBus, qbus, QTAILQ_FIRST(&adapter->child_bus)); if (!scsibus || strcmp(scsibus->qbus.info->name, "SCSI") != 0) { error_report("Device is not a SCSI adapter"); return -1; diff --git a/hw/qdev.c b/hw/qdev.c index f4ae4a6..25f6e62 100644 --- a/hw/qdev.c +++ b/hw/qdev.c @@ -85,10 +85,11 @@ static DeviceState *qdev_create_from_info(BusState *bus, DeviceInfo *info) dev = qemu_mallocz(info->size); dev->info = info; dev->parent_bus = bus; + QTAILQ_INIT(&dev->child_bus); qdev_prop_set_defaults(dev, dev->info->props); qdev_prop_set_defaults(dev, dev->parent_bus->info->props); qdev_prop_set_globals(dev); - QLIST_INSERT_HEAD(&bus->children, dev, sibling); + QTAILQ_INSERT_TAIL(&bus->children, dev, sibling); if (qdev_hotplug) { assert(bus->allow_hotplug); dev->hotplugged = 1; @@ -338,7 +339,7 @@ void qdev_free(DeviceState *dev) if (dev->state == DEV_STATE_INITIALIZED) { while (dev->num_child_bus) { - bus = QLIST_FIRST(&dev->child_bus); + bus = QTAILQ_FIRST(&dev->child_bus); qbus_free(bus); } if (dev->info->vmsd) @@ -349,7 +350,7 @@ void qdev_free(DeviceState *dev) qemu_opts_del(dev->opts); } qemu_unregister_reset(qdev_reset, dev); - QLIST_REMOVE(dev, sibling); + QTAILQ_REMOVE(&dev->parent_bus->children, dev, sibling); for (prop = dev->info->props; prop && prop->name; prop++) { if (prop->info->free) { prop->info->free(dev, prop); @@ -438,7 +439,7 @@ BusState *qdev_get_child_bus(DeviceState *dev, const char *name) { BusState *bus; - QLIST_FOREACH(bus, &dev->child_bus, sibling) { + QTAILQ_FOREACH(bus, &dev->child_bus, sibling) { if (strcmp(name, bus->name) == 0) { return bus; } @@ -463,8 +464,8 @@ static BusState *qbus_find_recursive(BusState *bus, const char *name, return bus; } - QLIST_FOREACH(dev, &bus->children, sibling) { - QLIST_FOREACH(child, &dev->child_bus, sibling) { + QTAILQ_FOREACH(dev, &bus->children, sibling) { + QTAILQ_FOREACH(child, &dev->child_bus, sibling) { ret = qbus_find_recursive(child, name, info); if (ret) { return ret; @@ -479,10 +480,10 @@ static DeviceState *qdev_find_recursive(BusState *bus, const char *id) DeviceState *dev, *ret; BusState *child; - QLIST_FOREACH(dev, &bus->children, sibling) { + QTAILQ_FOREACH(dev, &bus->children, sibling) { if (dev->id && strcmp(dev->id, id) == 0) return dev; - QLIST_FOREACH(child, &dev->child_bus, sibling) { + QTAILQ_FOREACH(child, &dev->child_bus, sibling) { ret = qdev_find_recursive(child, id); if (ret) { return ret; @@ -497,7 +498,7 @@ static int qdev_instance_no(DeviceState *dev) struct DeviceState *sibling; int instance = 0; - QLIST_FOREACH(sibling, &dev->parent_bus->children, sibling) { + QTAILQ_FOREACH(sibling, &dev->parent_bus->children, sibling) { if (sibling->info == dev->info) { if (sibling == dev) { break; @@ -515,7 +516,7 @@ static void qbus_list_bus(DeviceState *dev) error_printf("child busses at \"%s.%d\":", dev->info->name, qdev_instance_no(dev)); - QLIST_FOREACH(child, &dev->child_bus, sibling) { + QTAILQ_FOREACH(child, &dev->child_bus, sibling) { error_printf("%s\"%s\"", sep, child->name); sep = ", "; } @@ -528,7 +529,7 @@ static void qbus_list_dev(BusState *bus) const char *sep = " "; error_printf("devices at \"%s\":", bus->name); - QLIST_FOREACH(dev, &bus->children, sibling) { + QTAILQ_FOREACH(dev, &bus->children, sibling) { error_printf("%s\"%s.%d\"", sep, dev->info->name, qdev_instance_no(dev)); sep = ", "; @@ -540,7 +541,7 @@ static BusState *qbus_find_bus(DeviceState *dev, char *elem) { BusState *child; - QLIST_FOREACH(child, &dev->child_bus, sibling) { + QTAILQ_FOREACH(child, &dev->child_bus, sibling) { if (strcmp(child->name, elem) == 0) { return child; } @@ -567,14 +568,14 @@ static DeviceState *qbus_find_dev(BusState *bus, const char *elem) } n = 0; - QLIST_FOREACH(dev, &bus->children, sibling) { + QTAILQ_FOREACH(dev, &bus->children, sibling) { if (strcmp(dev->info->name, elem) == 0 && n++ == instance) { return dev; } } n = 0; - QLIST_FOREACH(dev, &bus->children, sibling) { + QTAILQ_FOREACH(dev, &bus->children, sibling) { if (dev->info->alias && strcmp(dev->info->alias, elem) == 0 && n++ == instance) { return dev; @@ -677,9 +678,9 @@ void qbus_create_inplace(BusState *bus, BusInfo *info, bus->name = buf; } - QLIST_INIT(&bus->children); + QTAILQ_INIT(&bus->children); if (parent) { - QLIST_INSERT_HEAD(&parent->child_bus, bus, sibling); + QTAILQ_INSERT_TAIL(&parent->child_bus, bus, sibling); parent->num_child_bus++; } @@ -699,11 +700,11 @@ void qbus_free(BusState *bus) { DeviceState *dev; - while ((dev = QLIST_FIRST(&bus->children)) != NULL) { + while ((dev = QTAILQ_FIRST(&bus->children)) != NULL) { qdev_free(dev); } if (bus->parent) { - QLIST_REMOVE(bus, sibling); + QTAILQ_REMOVE(&bus->parent->child_bus, bus, sibling); bus->parent->num_child_bus--; } qemu_free((void*)bus->name); @@ -753,7 +754,7 @@ static void qdev_print(Monitor *mon, DeviceState *dev, int indent) qdev_print_props(mon, dev, dev->parent_bus->info->props, "bus", indent); if (dev->parent_bus->info->print_dev) dev->parent_bus->info->print_dev(mon, dev, indent); - QLIST_FOREACH(child, &dev->child_bus, sibling) { + QTAILQ_FOREACH(child, &dev->child_bus, sibling) { qbus_print(mon, child, indent); } } @@ -765,7 +766,7 @@ static void qbus_print(Monitor *mon, BusState *bus, int indent) qdev_printf("bus: %s\n", bus->name); indent += 2; qdev_printf("type %s\n", bus->info->name); - QLIST_FOREACH(dev, &bus->children, sibling) { + QTAILQ_FOREACH(dev, &bus->children, sibling) { qdev_print(mon, dev, indent); } } diff --git a/hw/qdev.h b/hw/qdev.h index be5ad67..170a63a 100644 --- a/hw/qdev.h +++ b/hw/qdev.h @@ -41,9 +41,9 @@ struct DeviceState { qemu_irq *gpio_out; int num_gpio_in; qemu_irq *gpio_in; - QLIST_HEAD(, BusState) child_bus; + QTAILQ_HEAD(, BusState) child_bus; int num_child_bus; - QLIST_ENTRY(DeviceState) sibling; + QTAILQ_ENTRY(DeviceState) sibling; int instance_id_alias; int alias_required_for_version; }; @@ -62,8 +62,8 @@ struct BusState { const char *name; int allow_hotplug; int qdev_allocated; - QLIST_HEAD(, DeviceState) children; - QLIST_ENTRY(BusState) sibling; + QTAILQ_HEAD(, DeviceState) children; + QTAILQ_ENTRY(BusState) sibling; }; struct Property { diff --git a/hw/ssi.c b/hw/ssi.c index cfe7c07..2c18436 100644 --- a/hw/ssi.c +++ b/hw/ssi.c @@ -25,8 +25,8 @@ static int ssi_slave_init(DeviceState *dev, DeviceInfo *base_info) SSIBus *bus; bus = FROM_QBUS(SSIBus, qdev_get_parent_bus(dev)); - if (QLIST_FIRST(&bus->qbus.children) != dev - || QLIST_NEXT(dev, sibling) != NULL) { + if (QTAILQ_FIRST(&bus->qbus.children) != dev + || QTAILQ_NEXT(dev, sibling) != NULL) { hw_error("Too many devices on SSI bus"); } @@ -61,7 +61,7 @@ uint32_t ssi_transfer(SSIBus *bus, uint32_t val) { DeviceState *dev; SSISlave *slave; - dev = QLIST_FIRST(&bus->qbus.children); + dev = QTAILQ_FIRST(&bus->qbus.children); if (!dev) { return 0; }