* [Qemu-devel] [PATCH v0 0/5]: Convert pci_info() to QObject
@ 2010-01-05 20:34 Luiz Capitulino
2010-01-05 20:34 ` [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() Luiz Capitulino
` (4 more replies)
0 siblings, 5 replies; 10+ messages in thread
From: Luiz Capitulino @ 2010-01-05 20:34 UTC (permalink / raw)
To: qemu-devel; +Cc: aliguori, mst
Hi,
This series begins complex handler conversion to QObject by converting
pci_info(), which is called when "info pci" is executed in the Monitor.
Please help reviewing, as this was the more complex conversion I did
so far and I only ran basic tests.
Two points worth noting:
1. I'm still using my old, hand crafted format for the internal
documentation (a better way is coming)
2. I had to split the code which prints information about PCI bridges in
a way I dislike, but this was needed to identify problems with
bisect (as I couldn't test it properly)
More details in the patches themselves.
Thanks.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() 2010-01-05 20:34 [Qemu-devel] [PATCH v0 0/5]: Convert pci_info() to QObject Luiz Capitulino @ 2010-01-05 20:34 ` Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 2/5] QDict: Introduce qdict_get_qdict() Luiz Capitulino ` (3 subsequent siblings) 4 siblings, 0 replies; 10+ messages in thread From: Luiz Capitulino @ 2010-01-05 20:34 UTC (permalink / raw) To: qemu-devel; +Cc: aliguori, mst Iterate over QList entries, it's needed to call qlist_entry_obj() to retrieve the stored QObject. I'm not sure if it's ok to have this, because it's not as easy as qlist_iter() and the QListEntry data type is now exposed to the users, which means we have one more struct to be maintained when we have libqmp. Adding anyway, as it's more compact and people are asking for it. Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com> --- qlist.h | 10 ++++++++++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/qlist.h b/qlist.h index afdc446..a3261e1 100644 --- a/qlist.h +++ b/qlist.h @@ -29,6 +29,16 @@ typedef struct QList { #define qlist_append(qlist, obj) \ qlist_append_obj(qlist, QOBJECT(obj)) +#define QLIST_FOREACH_ENTRY(qlist, var) \ + for ((var) = ((qlist)->head.tqh_first); \ + (var); \ + (var) = ((var)->next.tqe_next)) + +static inline QObject *qlist_entry_obj(const QListEntry *entry) +{ + return entry->value; +} + QList *qlist_new(void); QList *qlist_copy(QList *src); void qlist_append_obj(QList *qlist, QObject *obj); -- 1.6.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Qemu-devel] [PATCH 2/5] QDict: Introduce qdict_get_qdict() 2010-01-05 20:34 [Qemu-devel] [PATCH v0 0/5]: Convert pci_info() to QObject Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() Luiz Capitulino @ 2010-01-05 20:34 ` Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 3/5] PCI: Convert pci_info() to QObject Luiz Capitulino ` (2 subsequent siblings) 4 siblings, 0 replies; 10+ messages in thread From: Luiz Capitulino @ 2010-01-05 20:34 UTC (permalink / raw) To: qemu-devel; +Cc: aliguori, mst A helper to retrieve a QDict from a QDict. Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com> --- qdict.c | 13 +++++++++++++ qdict.h | 1 + 2 files changed, 14 insertions(+), 0 deletions(-) diff --git a/qdict.c b/qdict.c index ba8eef0..c6a5a42 100644 --- a/qdict.c +++ b/qdict.c @@ -216,6 +216,19 @@ QList *qdict_get_qlist(const QDict *qdict, const char *key) } /** + * qdict_get_qdict(): Get the QDict mapped by 'key' + * + * This function assumes that 'key' exists and it stores a + * QDict object. + * + * Return QDict mapped by 'key'. + */ +QDict *qdict_get_qdict(const QDict *qdict, const char *key) +{ + return qobject_to_qdict(qdict_get_obj(qdict, key, QTYPE_QDICT)); +} + +/** * qdict_get_str(): Get a pointer to the stored string mapped * by 'key' * diff --git a/qdict.h b/qdict.h index 5fef1ea..2eaf6d5 100644 --- a/qdict.h +++ b/qdict.h @@ -40,6 +40,7 @@ void qdict_iter(const QDict *qdict, int64_t qdict_get_int(const QDict *qdict, const char *key); int qdict_get_bool(const QDict *qdict, const char *key); QList *qdict_get_qlist(const QDict *qdict, const char *key); +QDict *qdict_get_qdict(const QDict *qdict, const char *key); const char *qdict_get_str(const QDict *qdict, const char *key); int64_t qdict_get_try_int(const QDict *qdict, const char *key, int64_t err_value); -- 1.6.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Qemu-devel] [PATCH 3/5] PCI: Convert pci_info() to QObject 2010-01-05 20:34 [Qemu-devel] [PATCH v0 0/5]: Convert pci_info() to QObject Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 2/5] QDict: Introduce qdict_get_qdict() Luiz Capitulino @ 2010-01-05 20:34 ` Luiz Capitulino 2010-01-07 15:19 ` [Qemu-devel] " Anthony Liguori 2010-01-05 20:34 ` [Qemu-devel] [PATCH 4/5] PCI: do_pci_info(): PCI bridge support Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 5/5] PCI: do_pci_info(): PCI bridge devices support Luiz Capitulino 4 siblings, 1 reply; 10+ messages in thread From: Luiz Capitulino @ 2010-01-05 20:34 UTC (permalink / raw) To: qemu-devel; +Cc: aliguori, mst The returned QObject is a QList of all buses. Each bus is represented by a QDict, which has a key with a QList of all PCI devices attached to it. Each device is represented by a QDict. IMPORTANT: support for printing PCI bridge information and its devices is NOT part of this commit, it's going to be added by next commits. Finally, as has happended to other complex conversions, it's hard to split this commit as part of it are new functions which are called by each other. Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com> --- hw/pci.c | 306 +++++++++++++++++++++++++++++++++++++++++++++---------------- hw/pci.h | 4 +- monitor.c | 3 +- 3 files changed, 232 insertions(+), 81 deletions(-) diff --git a/hw/pci.c b/hw/pci.c index 0814383..8ded418 100644 --- a/hw/pci.c +++ b/hw/pci.c @@ -27,6 +27,7 @@ #include "net.h" #include "sysemu.h" #include "loader.h" +#include "qemu-objects.h" //#define DEBUG_PCI #ifdef DEBUG_PCI @@ -1051,119 +1052,266 @@ static const pci_class_desc pci_class_descriptions[] = { 0, NULL} }; -static void pci_info_device(PCIBus *bus, PCIDevice *d) +static void pci_for_each_device_under_bus(PCIBus *bus, + void (*fn)(PCIBus *b, PCIDevice *d)) { - Monitor *mon = cur_mon; - int i, class; - PCIIORegion *r; - const pci_class_desc *desc; + PCIDevice *d; + int devfn; - monitor_printf(mon, " Bus %2d, device %3d, function %d:\n", - pci_bus_num(d->bus), - PCI_SLOT(d->devfn), PCI_FUNC(d->devfn)); - class = pci_get_word(d->config + PCI_CLASS_DEVICE); + for(devfn = 0; devfn < ARRAY_SIZE(bus->devices); devfn++) { + d = bus->devices[devfn]; + if (d) + fn(bus, d); + } +} + +void pci_for_each_device(PCIBus *bus, int bus_num, + void (*fn)(PCIBus *b, PCIDevice *d)) +{ + bus = pci_find_bus(bus, bus_num); + + if (bus) { + pci_for_each_device_under_bus(bus, fn); + } +} + +static void pci_device_print(Monitor *mon, QDict *device) +{ + QDict *qdict; + QListEntry *entry; + uint64_t addr, size; + + monitor_printf(mon, " Bus %2" PRId64 ", ", qdict_get_int(device, "bus")); + monitor_printf(mon, "device %3" PRId64 ", function %" PRId64 ":\n", + qdict_get_int(device, "slot"), + qdict_get_int(device, "function")); monitor_printf(mon, " "); + + qdict = qdict_get_qdict(device, "class_info"); + if (qdict_haskey(qdict, "desc")) { + monitor_printf(mon, "%s", qdict_get_str(qdict, "desc")); + } else { + monitor_printf(mon, "Class %04" PRId64, qdict_get_int(qdict, "class")); + } + + qdict = qdict_get_qdict(device, "id"); + monitor_printf(mon, ": PCI device %04" PRIx64 ":%04" PRIx64 "\n", + qdict_get_int(qdict, "device"), + qdict_get_int(qdict, "vendor")); + + if (qdict_haskey(device, "IRQ")) { + monitor_printf(mon, " IRQ %" PRId64 ".\n", + qdict_get_int(device, "IRQ")); + } + + /* TODO: PCI bridge info */ + + QLIST_FOREACH_ENTRY(qdict_get_qlist(device, "regions"), entry) { + qdict = qobject_to_qdict(qlist_entry_obj(entry)); + monitor_printf(mon, " BAR%d: ", (int) qdict_get_int(qdict, "BAR")); + + addr = qdict_get_int(qdict, "address"); + size = qdict_get_int(qdict, "size"); + + if (!strcmp(qdict_get_str(qdict, "type"), "io")) { + monitor_printf(mon, "I/O at 0x%04"FMT_PCIBUS + " [0x%04"FMT_PCIBUS"].\n", + addr, addr + size - 1); + } else { + monitor_printf(mon, "%d bit%s memory at 0x%08"FMT_PCIBUS + " [0x%08"FMT_PCIBUS"].\n", + qdict_get_bool(qdict, "mem_type_64") ? 64 : 32, + qdict_get_bool(qdict, "prefetch") ? + " prefetchable" : "", addr, addr + size - 1); + } + } + + monitor_printf(mon, " id \"%s\"\n", qdict_get_str(device, "qdev_id")); + + /* TODO: PCI bridge devices */ +} + +void do_pci_info_print(Monitor *mon, const QObject *data) +{ + QListEntry *bus, *dev; + + QLIST_FOREACH_ENTRY(qobject_to_qlist(data), bus) { + QDict *qdict = qobject_to_qdict(qlist_entry_obj(bus)); + QLIST_FOREACH_ENTRY(qdict_get_qlist(qdict, "devices"), dev) { + pci_device_print(mon, qobject_to_qdict(qlist_entry_obj(dev))); + } + } +} + +static QObject *pci_get_dev_class(const PCIDevice *dev) +{ + int class; + const char *str = ""; + const pci_class_desc *desc; + + class = pci_get_word(dev->config + PCI_CLASS_DEVICE); desc = pci_class_descriptions; while (desc->desc && class != desc->class) desc++; + if (desc->desc) { - monitor_printf(mon, "%s", desc->desc); - } else { - monitor_printf(mon, "Class %04x", class); - } - monitor_printf(mon, ": PCI device %04x:%04x\n", - pci_get_word(d->config + PCI_VENDOR_ID), - pci_get_word(d->config + PCI_DEVICE_ID)); - - if (d->config[PCI_INTERRUPT_PIN] != 0) { - monitor_printf(mon, " IRQ %d.\n", - d->config[PCI_INTERRUPT_LINE]); - } - if (class == 0x0604) { - uint64_t base; - uint64_t limit; - - monitor_printf(mon, " BUS %d.\n", d->config[0x19]); - monitor_printf(mon, " secondary bus %d.\n", - d->config[PCI_SECONDARY_BUS]); - monitor_printf(mon, " subordinate bus %d.\n", - d->config[PCI_SUBORDINATE_BUS]); - - base = pci_bridge_get_base(d, PCI_BASE_ADDRESS_SPACE_IO); - limit = pci_bridge_get_limit(d, PCI_BASE_ADDRESS_SPACE_IO); - monitor_printf(mon, " IO range [0x%04"PRIx64", 0x%04"PRIx64"]\n", - base, limit); - - base = pci_bridge_get_base(d, PCI_BASE_ADDRESS_SPACE_MEMORY); - limit= pci_bridge_get_limit(d, PCI_BASE_ADDRESS_SPACE_MEMORY); - monitor_printf(mon, - " memory range [0x%08"PRIx64", 0x%08"PRIx64"]\n", - base, limit); - - base = pci_bridge_get_base(d, PCI_BASE_ADDRESS_SPACE_MEMORY | - PCI_BASE_ADDRESS_MEM_PREFETCH); - limit = pci_bridge_get_limit(d, PCI_BASE_ADDRESS_SPACE_MEMORY | - PCI_BASE_ADDRESS_MEM_PREFETCH); - monitor_printf(mon, " prefetchable memory range " - "[0x%08"PRIx64", 0x%08"PRIx64"]\n", base, limit); - } - for(i = 0;i < PCI_NUM_REGIONS; i++) { - r = &d->io_regions[i]; + str = desc->desc; + } + + return qobject_from_jsonf("{ 'desc': %s, 'class': %d }", str, class); +} + +static QObject *pci_get_dev_id(const PCIDevice *dev) +{ + return qobject_from_jsonf("{ 'device': %d, 'vendor': %d }", + pci_get_word(dev->config + PCI_VENDOR_ID), + pci_get_word(dev->config + PCI_DEVICE_ID)); +} + +static QObject *pci_get_regions_list(const PCIDevice *dev) +{ + int i; + QList *regions_list; + + regions_list = qlist_new(); + + for (i = 0; i < PCI_NUM_REGIONS; i++) { + const PCIIORegion *r = &dev->io_regions[i]; if (r->size != 0) { - monitor_printf(mon, " BAR%d: ", i); + QObject *obj; if (r->type & PCI_BASE_ADDRESS_SPACE_IO) { - monitor_printf(mon, "I/O at 0x%04"FMT_PCIBUS - " [0x%04"FMT_PCIBUS"].\n", - r->addr, r->addr + r->size - 1); + obj = qobject_from_jsonf("{ 'BAR': %d, 'type': 'io', " + "'address': %" PRId64 ", " + "'size': %" PRId64 " }", + i, r->addr, r->size); } else { - const char *type = r->type & PCI_BASE_ADDRESS_MEM_TYPE_64 ? - "64 bit" : "32 bit"; - const char *prefetch = - r->type & PCI_BASE_ADDRESS_MEM_PREFETCH ? - " prefetchable" : ""; - - monitor_printf(mon, "%s%s memory at 0x%08"FMT_PCIBUS - " [0x%08"FMT_PCIBUS"].\n", - type, prefetch, - r->addr, r->addr + r->size - 1); + int mem_type_64 = r->type & PCI_BASE_ADDRESS_MEM_TYPE_64; + + obj = qobject_from_jsonf("{ 'BAR': %d, 'type': 'memory', " + "'mem_type_64': %i, 'prefetch': %i, " + "'address': %" PRId64 ", " + "'size': %" PRId64 " }", + i, mem_type_64, + r->type &PCI_BASE_ADDRESS_MEM_PREFETCH, + r->addr, r->size); } + qlist_append_obj(regions_list, obj); } } - monitor_printf(mon, " id \"%s\"\n", d->qdev.id ? d->qdev.id : ""); - if (class == 0x0604 && d->config[0x19] != 0) { - pci_for_each_device(bus, d->config[0x19], pci_info_device); + + return QOBJECT(regions_list); +} + +static QObject *pci_get_dev_dict(const PCIDevice *dev, int bus_num) +{ + QObject *obj; + + obj = qobject_from_jsonf("{ 'bus': %d, 'slot': %d, 'function': %d," "'class_info': %p, 'id': %p, 'regions': %p," + " 'qdev_id': %s }", + bus_num, + PCI_SLOT(dev->devfn), PCI_FUNC(dev->devfn), + pci_get_dev_class(dev), pci_get_dev_id(dev), + pci_get_regions_list(dev), + dev->qdev.id ? dev->qdev.id : ""); + + if (dev->config[PCI_INTERRUPT_PIN] != 0) { + QDict *qdict = qobject_to_qdict(obj); + qdict_put(qdict, "IRQ", qint_from_int(dev->config[PCI_INTERRUPT_LINE])); } + + return obj; } -static void pci_for_each_device_under_bus(PCIBus *bus, - void (*fn)(PCIBus *b, PCIDevice *d)) +static QObject *pci_get_devices_list(PCIBus *bus, int bus_num) { - PCIDevice *d; int devfn; + PCIDevice *dev; + QList *dev_list; - for(devfn = 0; devfn < ARRAY_SIZE(bus->devices); devfn++) { - d = bus->devices[devfn]; - if (d) - fn(bus, d); + dev_list = qlist_new(); + + for (devfn = 0; devfn < ARRAY_SIZE(bus->devices); devfn++) { + dev = bus->devices[devfn]; + if (dev) { + qlist_append_obj(dev_list, pci_get_dev_dict(dev, bus_num)); + } } + + return QOBJECT(dev_list); } -void pci_for_each_device(PCIBus *bus, int bus_num, - void (*fn)(PCIBus *b, PCIDevice *d)) +static QObject *pci_get_bus_dict(PCIBus *bus, int bus_num) { bus = pci_find_bus(bus, bus_num); - if (bus) { - pci_for_each_device_under_bus(bus, fn); + return qobject_from_jsonf("{ 'bus': %d, 'devices': %p }", + bus_num, pci_get_devices_list(bus, bus_num)); } + + return NULL; } -void pci_info(Monitor *mon) +/** + * do_pci_info(): PCI buses and devices information + * + * The returned QObject is a QList of all buses. Each bus is + * represented by a QDict, which has a key with a QList of all + * PCI devices attached to it. Each device is represented by + * a QDict. + * + * The bus QDict contains the following: + * + * - "bus": bus number + * - "devices": a QList of QDicts, each QDict represents a PCI + * device + * + * The PCI device QDict contains the following: + * + * - "bus": bus number + * - "slot": slot number + * - "function": function number + * - "class_info": a QDict containing: + * - "desc": device class description + * - "class": device class number + * - "id": a QDict containing: + * - "device": device ID + * - "vendor": vendor ID + * - "qdev_id": qdev id string + * - "regions": a QList of QDicts, each QDict represents a + * memory region of this device + * + * The region QDict can be an I/O region or a memory region, + * the I/O region contains the following: + * + * - "type": "io" + * - "BAR": BAR number + * - "address": memory address + * - "size": memory size + * + * The memory region QDict contains the following: + * + * - "type": "memory" + * - "BAR": BAR number + * - "address": memory address + * - "size": memory size + * - "mem_type_64": true or false + * - "prefetch": true or false + */ +void do_pci_info(Monitor *mon, QObject **ret_data) { + QList *bus_list; struct PCIHostBus *host; + + bus_list = qlist_new(); + QLIST_FOREACH(host, &host_buses, next) { - pci_for_each_device(host->bus, 0, pci_info_device); + QObject *obj = pci_get_bus_dict(host->bus, 0); + if (obj) { + qlist_append_obj(bus_list, obj); + } } + + *ret_data = QOBJECT(bus_list); } static const char * const pci_nic_models[] = { diff --git a/hw/pci.h b/hw/pci.h index fd16460..e1e6d29 100644 --- a/hw/pci.h +++ b/hw/pci.h @@ -2,6 +2,7 @@ #define QEMU_PCI_H #include "qemu-common.h" +#include "qobject.h" #include "qdev.h" @@ -300,7 +301,8 @@ PCIBus *pci_get_bus_devfn(int *devfnp, const char *devaddr); int pci_read_devaddr(Monitor *mon, const char *addr, int *domp, int *busp, unsigned *slotp); -void pci_info(Monitor *mon); +void do_pci_info_print(Monitor *mon, const QObject *data); +void do_pci_info(Monitor *mon, QObject **ret_data); PCIBus *pci_bridge_init(PCIBus *bus, int devfn, uint16_t vid, uint16_t did, pci_map_irq_fn map_irq, const char *name); PCIDevice *pci_bridge_get_device(PCIBus *bus); diff --git a/monitor.c b/monitor.c index 3af1d5c..3fe7aa9 100644 --- a/monitor.c +++ b/monitor.c @@ -2436,7 +2436,8 @@ static const mon_cmd_t info_cmds[] = { .args_type = "", .params = "", .help = "show PCI info", - .mhandler.info = pci_info, + .user_print = do_pci_info_print, + .mhandler.info_new = do_pci_info, }, #if defined(TARGET_I386) || defined(TARGET_SH4) { -- 1.6.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Qemu-devel] Re: [PATCH 3/5] PCI: Convert pci_info() to QObject 2010-01-05 20:34 ` [Qemu-devel] [PATCH 3/5] PCI: Convert pci_info() to QObject Luiz Capitulino @ 2010-01-07 15:19 ` Anthony Liguori 2010-01-07 18:00 ` Luiz Capitulino 0 siblings, 1 reply; 10+ messages in thread From: Anthony Liguori @ 2010-01-07 15:19 UTC (permalink / raw) To: Luiz Capitulino; +Cc: qemu-devel, mst On 01/05/2010 02:34 PM, Luiz Capitulino wrote: > The returned QObject is a QList of all buses. Each bus is > represented by a QDict, which has a key with a QList of all > PCI devices attached to it. Each device is represented by > a QDict. > > IMPORTANT: support for printing PCI bridge information and > its devices is NOT part of this commit, it's going to be > added by next commits. > > Finally, as has happended to other complex conversions, > it's hard to split this commit as part of it are new > functions which are called by each other. > > Signed-off-by: Luiz Capitulino<lcapitulino@redhat.com> > --- > hw/pci.c | 306 +++++++++++++++++++++++++++++++++++++++++++++---------------- > hw/pci.h | 4 +- > monitor.c | 3 +- > 3 files changed, 232 insertions(+), 81 deletions(-) > > diff --git a/hw/pci.c b/hw/pci.c > index 0814383..8ded418 100644 > --- a/hw/pci.c > +++ b/hw/pci.c > @@ -27,6 +27,7 @@ > #include "net.h" > #include "sysemu.h" > #include "loader.h" > +#include "qemu-objects.h" > > //#define DEBUG_PCI > #ifdef DEBUG_PCI > @@ -1051,119 +1052,266 @@ static const pci_class_desc pci_class_descriptions[] = > { 0, NULL} > }; > > -static void pci_info_device(PCIBus *bus, PCIDevice *d) > +static void pci_for_each_device_under_bus(PCIBus *bus, > + void (*fn)(PCIBus *b, PCIDevice *d)) > { > - Monitor *mon = cur_mon; > - int i, class; > - PCIIORegion *r; > - const pci_class_desc *desc; > + PCIDevice *d; > + int devfn; > > - monitor_printf(mon, " Bus %2d, device %3d, function %d:\n", > - pci_bus_num(d->bus), > - PCI_SLOT(d->devfn), PCI_FUNC(d->devfn)); > - class = pci_get_word(d->config + PCI_CLASS_DEVICE); > + for(devfn = 0; devfn< ARRAY_SIZE(bus->devices); devfn++) { > + d = bus->devices[devfn]; > + if (d) > + fn(bus, d); > + } > +} > Coding style is off. You're missing braces on the if. > +void pci_for_each_device(PCIBus *bus, int bus_num, > + void (*fn)(PCIBus *b, PCIDevice *d)) > +{ > + bus = pci_find_bus(bus, bus_num); > + > + if (bus) { > + pci_for_each_device_under_bus(bus, fn); > + } > +} > + > +static void pci_device_print(Monitor *mon, QDict *device) > +{ > + QDict *qdict; > + QListEntry *entry; > + uint64_t addr, size; > + > + monitor_printf(mon, " Bus %2" PRId64 ", ", qdict_get_int(device, "bus")); > + monitor_printf(mon, "device %3" PRId64 ", function %" PRId64 ":\n", > + qdict_get_int(device, "slot"), > + qdict_get_int(device, "function")); > monitor_printf(mon, " "); > + > + qdict = qdict_get_qdict(device, "class_info"); > + if (qdict_haskey(qdict, "desc")) { > + monitor_printf(mon, "%s", qdict_get_str(qdict, "desc")); > + } else { > + monitor_printf(mon, "Class %04" PRId64, qdict_get_int(qdict, "class")); > + } > + > + qdict = qdict_get_qdict(device, "id"); > + monitor_printf(mon, ": PCI device %04" PRIx64 ":%04" PRIx64 "\n", > + qdict_get_int(qdict, "device"), > + qdict_get_int(qdict, "vendor")); > + > + if (qdict_haskey(device, "IRQ")) { > + monitor_printf(mon, " IRQ %" PRId64 ".\n", > + qdict_get_int(device, "IRQ")); > + } > + > + /* TODO: PCI bridge info */ > + > + QLIST_FOREACH_ENTRY(qdict_get_qlist(device, "regions"), entry) { > + qdict = qobject_to_qdict(qlist_entry_obj(entry)); > + monitor_printf(mon, " BAR%d: ", (int) qdict_get_int(qdict, "BAR")); > + > + addr = qdict_get_int(qdict, "address"); > + size = qdict_get_int(qdict, "size"); > + > + if (!strcmp(qdict_get_str(qdict, "type"), "io")) { > + monitor_printf(mon, "I/O at 0x%04"FMT_PCIBUS > + " [0x%04"FMT_PCIBUS"].\n", > + addr, addr + size - 1); > + } else { > + monitor_printf(mon, "%d bit%s memory at 0x%08"FMT_PCIBUS > + " [0x%08"FMT_PCIBUS"].\n", > + qdict_get_bool(qdict, "mem_type_64") ? 64 : 32, > + qdict_get_bool(qdict, "prefetch") ? > + " prefetchable" : "", addr, addr + size - 1); > + } > + } > + > + monitor_printf(mon, " id \"%s\"\n", qdict_get_str(device, "qdev_id")); > + > + /* TODO: PCI bridge devices */ > +} > + > +void do_pci_info_print(Monitor *mon, const QObject *data) > +{ > + QListEntry *bus, *dev; > + > + QLIST_FOREACH_ENTRY(qobject_to_qlist(data), bus) { > + QDict *qdict = qobject_to_qdict(qlist_entry_obj(bus)); > + QLIST_FOREACH_ENTRY(qdict_get_qlist(qdict, "devices"), dev) { > + pci_device_print(mon, qobject_to_qdict(qlist_entry_obj(dev))); > + } > + } > +} > + > +static QObject *pci_get_dev_class(const PCIDevice *dev) > +{ > + int class; > + const char *str = ""; > + const pci_class_desc *desc; > + > + class = pci_get_word(dev->config + PCI_CLASS_DEVICE); > desc = pci_class_descriptions; > while (desc->desc&& class != desc->class) > desc++; > + > if (desc->desc) { > - monitor_printf(mon, "%s", desc->desc); > - } else { > - monitor_printf(mon, "Class %04x", class); > - } > - monitor_printf(mon, ": PCI device %04x:%04x\n", > - pci_get_word(d->config + PCI_VENDOR_ID), > - pci_get_word(d->config + PCI_DEVICE_ID)); > - > - if (d->config[PCI_INTERRUPT_PIN] != 0) { > - monitor_printf(mon, " IRQ %d.\n", > - d->config[PCI_INTERRUPT_LINE]); > - } > - if (class == 0x0604) { > - uint64_t base; > - uint64_t limit; > - > - monitor_printf(mon, " BUS %d.\n", d->config[0x19]); > - monitor_printf(mon, " secondary bus %d.\n", > - d->config[PCI_SECONDARY_BUS]); > - monitor_printf(mon, " subordinate bus %d.\n", > - d->config[PCI_SUBORDINATE_BUS]); > - > - base = pci_bridge_get_base(d, PCI_BASE_ADDRESS_SPACE_IO); > - limit = pci_bridge_get_limit(d, PCI_BASE_ADDRESS_SPACE_IO); > - monitor_printf(mon, " IO range [0x%04"PRIx64", 0x%04"PRIx64"]\n", > - base, limit); > - > - base = pci_bridge_get_base(d, PCI_BASE_ADDRESS_SPACE_MEMORY); > - limit= pci_bridge_get_limit(d, PCI_BASE_ADDRESS_SPACE_MEMORY); > - monitor_printf(mon, > - " memory range [0x%08"PRIx64", 0x%08"PRIx64"]\n", > - base, limit); > - > - base = pci_bridge_get_base(d, PCI_BASE_ADDRESS_SPACE_MEMORY | > - PCI_BASE_ADDRESS_MEM_PREFETCH); > - limit = pci_bridge_get_limit(d, PCI_BASE_ADDRESS_SPACE_MEMORY | > - PCI_BASE_ADDRESS_MEM_PREFETCH); > - monitor_printf(mon, " prefetchable memory range " > - "[0x%08"PRIx64", 0x%08"PRIx64"]\n", base, limit); > - } > - for(i = 0;i< PCI_NUM_REGIONS; i++) { > - r =&d->io_regions[i]; > + str = desc->desc; > + } > + > + return qobject_from_jsonf("{ 'desc': %s, 'class': %d }", str, class); > +} > + > +static QObject *pci_get_dev_id(const PCIDevice *dev) > +{ > + return qobject_from_jsonf("{ 'device': %d, 'vendor': %d }", > + pci_get_word(dev->config + PCI_VENDOR_ID), > + pci_get_word(dev->config + PCI_DEVICE_ID)); > +} > + > +static QObject *pci_get_regions_list(const PCIDevice *dev) > +{ > + int i; > + QList *regions_list; > + > + regions_list = qlist_new(); > + > + for (i = 0; i< PCI_NUM_REGIONS; i++) { > + const PCIIORegion *r =&dev->io_regions[i]; > if (r->size != 0) { > - monitor_printf(mon, " BAR%d: ", i); > + QObject *obj; > if (r->type& PCI_BASE_ADDRESS_SPACE_IO) { > - monitor_printf(mon, "I/O at 0x%04"FMT_PCIBUS > - " [0x%04"FMT_PCIBUS"].\n", > - r->addr, r->addr + r->size - 1); > + obj = qobject_from_jsonf("{ 'BAR': %d, 'type': 'io', " > + "'address': %" PRId64 ", " > + "'size': %" PRId64 " }", > + i, r->addr, r->size); > } else { > - const char *type = r->type& PCI_BASE_ADDRESS_MEM_TYPE_64 ? > - "64 bit" : "32 bit"; > - const char *prefetch = > - r->type& PCI_BASE_ADDRESS_MEM_PREFETCH ? > - " prefetchable" : ""; > - > - monitor_printf(mon, "%s%s memory at 0x%08"FMT_PCIBUS > - " [0x%08"FMT_PCIBUS"].\n", > - type, prefetch, > - r->addr, r->addr + r->size - 1); > + int mem_type_64 = r->type& PCI_BASE_ADDRESS_MEM_TYPE_64; > + > + obj = qobject_from_jsonf("{ 'BAR': %d, 'type': 'memory', " > + "'mem_type_64': %i, 'prefetch': %i, " > + "'address': %" PRId64 ", " > + "'size': %" PRId64 " }", > + i, mem_type_64, > + r->type&PCI_BASE_ADDRESS_MEM_PREFETCH, > + r->addr, r->size); > } > + qlist_append_obj(regions_list, obj); > } > } > - monitor_printf(mon, " id \"%s\"\n", d->qdev.id ? d->qdev.id : ""); > - if (class == 0x0604&& d->config[0x19] != 0) { > - pci_for_each_device(bus, d->config[0x19], pci_info_device); > + > + return QOBJECT(regions_list); > +} > + > +static QObject *pci_get_dev_dict(const PCIDevice *dev, int bus_num) > +{ > + QObject *obj; > + > + obj = qobject_from_jsonf("{ 'bus': %d, 'slot': %d, 'function': %d," "'class_info': %p, 'id': %p, 'regions': %p," > + " 'qdev_id': %s }", > + bus_num, > + PCI_SLOT(dev->devfn), PCI_FUNC(dev->devfn), > + pci_get_dev_class(dev), pci_get_dev_id(dev), > + pci_get_regions_list(dev), > + dev->qdev.id ? dev->qdev.id : ""); > + > + if (dev->config[PCI_INTERRUPT_PIN] != 0) { > + QDict *qdict = qobject_to_qdict(obj); > + qdict_put(qdict, "IRQ", qint_from_int(dev->config[PCI_INTERRUPT_LINE])); > } > + > + return obj; > } > > -static void pci_for_each_device_under_bus(PCIBus *bus, > - void (*fn)(PCIBus *b, PCIDevice *d)) > +static QObject *pci_get_devices_list(PCIBus *bus, int bus_num) > { > - PCIDevice *d; > int devfn; > + PCIDevice *dev; > + QList *dev_list; > > - for(devfn = 0; devfn< ARRAY_SIZE(bus->devices); devfn++) { > - d = bus->devices[devfn]; > - if (d) > - fn(bus, d); > + dev_list = qlist_new(); > + > + for (devfn = 0; devfn< ARRAY_SIZE(bus->devices); devfn++) { > + dev = bus->devices[devfn]; > + if (dev) { > + qlist_append_obj(dev_list, pci_get_dev_dict(dev, bus_num)); > + } > } > + > + return QOBJECT(dev_list); > } > > -void pci_for_each_device(PCIBus *bus, int bus_num, > - void (*fn)(PCIBus *b, PCIDevice *d)) > +static QObject *pci_get_bus_dict(PCIBus *bus, int bus_num) > { > bus = pci_find_bus(bus, bus_num); > - > if (bus) { > - pci_for_each_device_under_bus(bus, fn); > + return qobject_from_jsonf("{ 'bus': %d, 'devices': %p }", > + bus_num, pci_get_devices_list(bus, bus_num)); > } > + > + return NULL; > } > > -void pci_info(Monitor *mon) > +/** > + * do_pci_info(): PCI buses and devices information > + * > + * The returned QObject is a QList of all buses. Each bus is > + * represented by a QDict, which has a key with a QList of all > + * PCI devices attached to it. Each device is represented by > + * a QDict. > + * > + * The bus QDict contains the following: > + * > + * - "bus": bus number > + * - "devices": a QList of QDicts, each QDict represents a PCI > + * device > + * > + * The PCI device QDict contains the following: > + * > + * - "bus": bus number > + * - "slot": slot number > + * - "function": function number > + * - "class_info": a QDict containing: > + * - "desc": device class description > + * - "class": device class number > + * - "id": a QDict containing: > + * - "device": device ID > + * - "vendor": vendor ID > + * - "qdev_id": qdev id string > + * - "regions": a QList of QDicts, each QDict represents a > + * memory region of this device > + * > + * The region QDict can be an I/O region or a memory region, > + * the I/O region contains the following: > + * > + * - "type": "io" > + * - "BAR": BAR number > + * - "address": memory address > + * - "size": memory size > + * > + * The memory region QDict contains the following: > + * > + * - "type": "memory" > + * - "BAR": BAR number > + * - "address": memory address > + * - "size": memory size > + * - "mem_type_64": true or false > + * - "prefetch": true or false > + */ > +void do_pci_info(Monitor *mon, QObject **ret_data) > { > + QList *bus_list; > struct PCIHostBus *host; > + > + bus_list = qlist_new(); > + > QLIST_FOREACH(host,&host_buses, next) { > - pci_for_each_device(host->bus, 0, pci_info_device); > + QObject *obj = pci_get_bus_dict(host->bus, 0); > + if (obj) { > + qlist_append_obj(bus_list, obj); > + } > } > + > + *ret_data = QOBJECT(bus_list); > } > > static const char * const pci_nic_models[] = { > diff --git a/hw/pci.h b/hw/pci.h > index fd16460..e1e6d29 100644 > --- a/hw/pci.h > +++ b/hw/pci.h > @@ -2,6 +2,7 @@ > #define QEMU_PCI_H > > #include "qemu-common.h" > +#include "qobject.h" > > #include "qdev.h" > > @@ -300,7 +301,8 @@ PCIBus *pci_get_bus_devfn(int *devfnp, const char *devaddr); > int pci_read_devaddr(Monitor *mon, const char *addr, int *domp, int *busp, > unsigned *slotp); > > -void pci_info(Monitor *mon); > +void do_pci_info_print(Monitor *mon, const QObject *data); > +void do_pci_info(Monitor *mon, QObject **ret_data); > PCIBus *pci_bridge_init(PCIBus *bus, int devfn, uint16_t vid, uint16_t did, > pci_map_irq_fn map_irq, const char *name); > PCIDevice *pci_bridge_get_device(PCIBus *bus); > diff --git a/monitor.c b/monitor.c > index 3af1d5c..3fe7aa9 100644 > --- a/monitor.c > +++ b/monitor.c > @@ -2436,7 +2436,8 @@ static const mon_cmd_t info_cmds[] = { > .args_type = "", > .params = "", > .help = "show PCI info", > - .mhandler.info = pci_info, > + .user_print = do_pci_info_print, > + .mhandler.info_new = do_pci_info, > }, > #if defined(TARGET_I386) || defined(TARGET_SH4) > { > The conversion otherwise looks good. I'm not sure mem_type_64 is the best property name but I'll let mst comment on that. I'd also prefer we didn't mix case with BAR and IRQ. I'd rather see us just use bar and irq. Regards, Anthony Liguori ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Qemu-devel] Re: [PATCH 3/5] PCI: Convert pci_info() to QObject 2010-01-07 15:19 ` [Qemu-devel] " Anthony Liguori @ 2010-01-07 18:00 ` Luiz Capitulino 0 siblings, 0 replies; 10+ messages in thread From: Luiz Capitulino @ 2010-01-07 18:00 UTC (permalink / raw) To: Anthony Liguori; +Cc: qemu-devel, mst On Thu, 07 Jan 2010 09:19:28 -0600 Anthony Liguori <aliguori@linux.vnet.ibm.com> wrote: > > -static void pci_info_device(PCIBus *bus, PCIDevice *d) > > +static void pci_for_each_device_under_bus(PCIBus *bus, > > + void (*fn)(PCIBus *b, PCIDevice *d)) > > { > > - Monitor *mon = cur_mon; > > - int i, class; > > - PCIIORegion *r; > > - const pci_class_desc *desc; > > + PCIDevice *d; > > + int devfn; > > > > - monitor_printf(mon, " Bus %2d, device %3d, function %d:\n", > > - pci_bus_num(d->bus), > > - PCI_SLOT(d->devfn), PCI_FUNC(d->devfn)); > > - class = pci_get_word(d->config + PCI_CLASS_DEVICE); > > + for(devfn = 0; devfn< ARRAY_SIZE(bus->devices); devfn++) { > > + d = bus->devices[devfn]; > > + if (d) > > + fn(bus, d); > > + } > > +} > > > > Coding style is off. You're missing braces on the if. Was based on the current code, will fix. > The conversion otherwise looks good. I'm not sure mem_type_64 is the > best property name but I'll let mst comment on that. I'd also prefer we > didn't mix case with BAR and IRQ. I'd rather see us just use bar and irq. Ok, will fix too. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Qemu-devel] [PATCH 4/5] PCI: do_pci_info(): PCI bridge support 2010-01-05 20:34 [Qemu-devel] [PATCH v0 0/5]: Convert pci_info() to QObject Luiz Capitulino ` (2 preceding siblings ...) 2010-01-05 20:34 ` [Qemu-devel] [PATCH 3/5] PCI: Convert pci_info() to QObject Luiz Capitulino @ 2010-01-05 20:34 ` Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 5/5] PCI: do_pci_info(): PCI bridge devices support Luiz Capitulino 4 siblings, 0 replies; 10+ messages in thread From: Luiz Capitulino @ 2010-01-05 20:34 UTC (permalink / raw) To: qemu-devel; +Cc: aliguori, mst This commit adds the "pci_bridge" key to the PCI device QDict, it also adds support for printing it in the user protocol. This code is being added separately because I could not test it properly. According to Michael Tsirkin, it depends on ultrasparc and it would take time to do the proper setup. Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com> --- hw/pci.c | 76 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-- 1 files changed, 73 insertions(+), 3 deletions(-) diff --git a/hw/pci.c b/hw/pci.c index 8ded418..4db392d 100644 --- a/hw/pci.c +++ b/hw/pci.c @@ -1077,6 +1077,7 @@ void pci_for_each_device(PCIBus *bus, int bus_num, static void pci_device_print(Monitor *mon, QDict *device) { + int class; QDict *qdict; QListEntry *entry; uint64_t addr, size; @@ -1088,10 +1089,11 @@ static void pci_device_print(Monitor *mon, QDict *device) monitor_printf(mon, " "); qdict = qdict_get_qdict(device, "class_info"); + class = qdict_get_int(qdict, "class"); if (qdict_haskey(qdict, "desc")) { monitor_printf(mon, "%s", qdict_get_str(qdict, "desc")); } else { - monitor_printf(mon, "Class %04" PRId64, qdict_get_int(qdict, "class")); + monitor_printf(mon, "Class %d", class); } qdict = qdict_get_qdict(device, "id"); @@ -1104,7 +1106,36 @@ static void pci_device_print(Monitor *mon, QDict *device) qdict_get_int(device, "IRQ")); } - /* TODO: PCI bridge info */ + if (qdict_haskey(device, "pci_bridge")) { + QDict *info; + + qdict = qdict_get_qdict(device, "pci_bridge"); + + info = qdict_get_qdict(qdict, "bus"); + monitor_printf(mon, " BUS %" PRId64 ".\n", + qdict_get_int(info, "number")); + monitor_printf(mon, " secondary bus %" PRId64 ".\n", + qdict_get_int(info, "secondary")); + monitor_printf(mon, " subordinate bus %" PRId64 ".\n", + qdict_get_int(info, "subordinate")); + + info = qdict_get_qdict(qdict, "io_range"); + monitor_printf(mon, " IO range [0x%04"PRIx64", 0x%04"PRIx64"]\n", + qdict_get_int(info, "base"), + qdict_get_int(info, "limit")); + + info = qdict_get_qdict(qdict, "memory_range"); + monitor_printf(mon, + " memory range [0x%08"PRIx64", 0x%08"PRIx64"]\n", + qdict_get_int(info, "base"), + qdict_get_int(info, "limit")); + + info = qdict_get_qdict(qdict, "prefetchable_range"); + monitor_printf(mon, " prefetchable memory range " + "[0x%08"PRIx64", 0x%08"PRIx64"]\n", + qdict_get_int(info, "base"), + qdict_get_int(info, "limit")); + } QLIST_FOREACH_ENTRY(qdict_get_qlist(device, "regions"), entry) { qdict = qobject_to_qdict(qlist_entry_obj(entry)); @@ -1202,8 +1233,9 @@ static QObject *pci_get_regions_list(const PCIDevice *dev) return QOBJECT(regions_list); } -static QObject *pci_get_dev_dict(const PCIDevice *dev, int bus_num) +static QObject *pci_get_dev_dict(PCIDevice *dev, int bus_num) { + int class; QObject *obj; obj = qobject_from_jsonf("{ 'bus': %d, 'slot': %d, 'function': %d," "'class_info': %p, 'id': %p, 'regions': %p," @@ -1219,6 +1251,31 @@ static QObject *pci_get_dev_dict(const PCIDevice *dev, int bus_num) qdict_put(qdict, "IRQ", qint_from_int(dev->config[PCI_INTERRUPT_LINE])); } + class = pci_get_word(dev->config + PCI_CLASS_DEVICE); + if (class == 0x0604) { + QDict *qdict; + QObject *pci_bridge; + + pci_bridge = qobject_from_jsonf("{ 'bus': " + "{ 'number': %d, 'secondary': %d, 'subordinate': %d }, " + "'io_range': { 'base': %" PRId64 ", 'limit': %" PRId64 "}, " + "'memory_range': { 'base': %" PRId64 ", 'limit': %" PRId64 "}, " + "'prefetchable_range': { 'base': %" PRId64 ", 'limit': %" PRId64 "} }", + dev->config[0x19], dev->config[PCI_SECONDARY_BUS], + dev->config[PCI_SUBORDINATE_BUS], + pci_bridge_get_base(dev, PCI_BASE_ADDRESS_SPACE_IO), + pci_bridge_get_limit(dev, PCI_BASE_ADDRESS_SPACE_IO), + pci_bridge_get_base(dev, PCI_BASE_ADDRESS_SPACE_MEMORY), + pci_bridge_get_limit(dev, PCI_BASE_ADDRESS_SPACE_MEMORY), + pci_bridge_get_base(dev, PCI_BASE_ADDRESS_SPACE_MEMORY | + PCI_BASE_ADDRESS_MEM_PREFETCH), + pci_bridge_get_limit(dev, PCI_BASE_ADDRESS_SPACE_MEMORY | + PCI_BASE_ADDRESS_MEM_PREFETCH)); + + qdict = qobject_to_qdict(obj); + qdict_put_obj(qdict, "pci_bridge", pci_bridge); + } + return obj; } @@ -1277,9 +1334,22 @@ static QObject *pci_get_bus_dict(PCIBus *bus, int bus_num) * - "device": device ID * - "vendor": vendor ID * - "qdev_id": qdev id string + * - "pci_bridge": It's a QDict, only present if this device is a + * PCI bridge, contains: + * - "bus": bus number + * - "secondary": secondary bus number + * - "subordinate": subordinate bus number + * - "io_range": a QDict with memory range information + * - "memory_range": a QDict with memory range information + * - "prefetchable_range": a QDict with memory range information * - "regions": a QList of QDicts, each QDict represents a * memory region of this device * + * The memory range QDict contains the following: + * + * - "base": base memory address + * - "limit": limit value + * * The region QDict can be an I/O region or a memory region, * the I/O region contains the following: * -- 1.6.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Qemu-devel] [PATCH 5/5] PCI: do_pci_info(): PCI bridge devices support 2010-01-05 20:34 [Qemu-devel] [PATCH v0 0/5]: Convert pci_info() to QObject Luiz Capitulino ` (3 preceding siblings ...) 2010-01-05 20:34 ` [Qemu-devel] [PATCH 4/5] PCI: do_pci_info(): PCI bridge support Luiz Capitulino @ 2010-01-05 20:34 ` Luiz Capitulino 4 siblings, 0 replies; 10+ messages in thread From: Luiz Capitulino @ 2010-01-05 20:34 UTC (permalink / raw) To: qemu-devel; +Cc: aliguori, mst This commit completes the do_pci_info() conversion to QObject by adding support to PCI bridge devices. This is done by recursively adding devices in the "pci_bridge" key. This code is being added separately because I could not test it properly. According to Michael Tsirkin, it depends on ultrasparc and it would take time to do the proper setup. Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com> --- hw/pci.c | 22 +++++++++++++++++++--- 1 files changed, 19 insertions(+), 3 deletions(-) diff --git a/hw/pci.c b/hw/pci.c index 4db392d..f38176c 100644 --- a/hw/pci.c +++ b/hw/pci.c @@ -1159,7 +1159,14 @@ static void pci_device_print(Monitor *mon, QDict *device) monitor_printf(mon, " id \"%s\"\n", qdict_get_str(device, "qdev_id")); - /* TODO: PCI bridge devices */ + if (qdict_haskey(device, "pci_bridge")) { + QListEntry *dev; + + qdict = qdict_get_qdict(device, "pci_bridge"); + QLIST_FOREACH_ENTRY(qdict_get_qlist(qdict, "devices"), dev) { + pci_device_print(mon, qobject_to_qdict(qlist_entry_obj(dev))); + } + } } void do_pci_info_print(Monitor *mon, const QObject *data) @@ -1233,7 +1240,9 @@ static QObject *pci_get_regions_list(const PCIDevice *dev) return QOBJECT(regions_list); } -static QObject *pci_get_dev_dict(PCIDevice *dev, int bus_num) +static QObject *pci_get_devices_list(PCIBus *bus, int bus_num); + +static QObject *pci_get_dev_dict(PCIDevice *dev, PCIBus *bus, int bus_num) { int class; QObject *obj; @@ -1272,6 +1281,12 @@ static QObject *pci_get_dev_dict(PCIDevice *dev, int bus_num) pci_bridge_get_limit(dev, PCI_BASE_ADDRESS_SPACE_MEMORY | PCI_BASE_ADDRESS_MEM_PREFETCH)); + if (dev->config[0x19] != 0) { + qdict = qobject_to_qdict(pci_bridge); + qdict_put_obj(qdict, "devices", + pci_get_devices_list(bus, dev->config[0x19])); + } + qdict = qobject_to_qdict(obj); qdict_put_obj(qdict, "pci_bridge", pci_bridge); } @@ -1290,7 +1305,7 @@ static QObject *pci_get_devices_list(PCIBus *bus, int bus_num) for (devfn = 0; devfn < ARRAY_SIZE(bus->devices); devfn++) { dev = bus->devices[devfn]; if (dev) { - qlist_append_obj(dev_list, pci_get_dev_dict(dev, bus_num)); + qlist_append_obj(dev_list, pci_get_dev_dict(dev, bus, bus_num)); } } @@ -1342,6 +1357,7 @@ static QObject *pci_get_bus_dict(PCIBus *bus, int bus_num) * - "io_range": a QDict with memory range information * - "memory_range": a QDict with memory range information * - "prefetchable_range": a QDict with memory range information + * - "devices": a QList of PCI devices if there's any attached * - "regions": a QList of QDicts, each QDict represents a * memory region of this device * -- 1.6.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Qemu-devel] [PATCH v1 0/5]: Convert pci_info() to QObject @ 2010-01-08 18:42 Luiz Capitulino 2010-01-08 18:42 ` [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() Luiz Capitulino 0 siblings, 1 reply; 10+ messages in thread From: Luiz Capitulino @ 2010-01-08 18:42 UTC (permalink / raw) To: qemu-devel; +Cc: aliguori, mst Hi, This is a new version of the pci_info() conversion. changelog --------- V0 -> V1 - Coding style fixes - Make 'BAR' and 'IRQ' keys lowercase - Add 'irq' key to the documentation Thanks. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() 2010-01-08 18:42 [Qemu-devel] [PATCH v1 0/5]: Convert pci_info() to QObject Luiz Capitulino @ 2010-01-08 18:42 ` Luiz Capitulino 0 siblings, 0 replies; 10+ messages in thread From: Luiz Capitulino @ 2010-01-08 18:42 UTC (permalink / raw) To: qemu-devel; +Cc: aliguori, mst Iterate over QList entries, it's needed to call qlist_entry_obj() to retrieve the stored QObject. I'm not sure if it's ok to have this, because it's not as easy as qlist_iter() and the QListEntry data type is now exposed to the users, which means we have one more struct to be maintained when we have libqmp. Adding anyway, as it's more compact and people are asking for it. Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com> --- qlist.h | 10 ++++++++++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/qlist.h b/qlist.h index afdc446..a3261e1 100644 --- a/qlist.h +++ b/qlist.h @@ -29,6 +29,16 @@ typedef struct QList { #define qlist_append(qlist, obj) \ qlist_append_obj(qlist, QOBJECT(obj)) +#define QLIST_FOREACH_ENTRY(qlist, var) \ + for ((var) = ((qlist)->head.tqh_first); \ + (var); \ + (var) = ((var)->next.tqe_next)) + +static inline QObject *qlist_entry_obj(const QListEntry *entry) +{ + return entry->value; +} + QList *qlist_new(void); QList *qlist_copy(QList *src); void qlist_append_obj(QList *qlist, QObject *obj); -- 1.6.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Qemu-devel] [PATCH v2 0/5]: Convert pci_info() to QObject @ 2010-01-20 16:39 Luiz Capitulino 2010-01-20 16:39 ` [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() Luiz Capitulino 0 siblings, 1 reply; 10+ messages in thread From: Luiz Capitulino @ 2010-01-20 16:39 UTC (permalink / raw) To: qemu-devel; +Cc: aliguori, armbru Hi, This new version addresses Markus's comments. changelog --------- V1 -> V2 - Make class_info's key 'desc' optional - Better indentation - Doc fixes V0 -> V1 - Coding style fixes - Make 'BAR' and 'IRQ' keys lowercase - Add 'irq' key to the documentation Thanks. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() 2010-01-20 16:39 [Qemu-devel] [PATCH v2 0/5]: Convert pci_info() to QObject Luiz Capitulino @ 2010-01-20 16:39 ` Luiz Capitulino 0 siblings, 0 replies; 10+ messages in thread From: Luiz Capitulino @ 2010-01-20 16:39 UTC (permalink / raw) To: qemu-devel; +Cc: aliguori, armbru Iterate over QList entries, it's needed to call qlist_entry_obj() to retrieve the stored QObject. I'm not sure if it's ok to have this, because it's not as easy as qlist_iter() and the QListEntry data type is now exposed to the users, which means we have one more struct to be maintained when we have libqmp. Adding anyway, as it's more compact and people are asking for it. Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com> --- qlist.h | 10 ++++++++++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/qlist.h b/qlist.h index afdc446..a3261e1 100644 --- a/qlist.h +++ b/qlist.h @@ -29,6 +29,16 @@ typedef struct QList { #define qlist_append(qlist, obj) \ qlist_append_obj(qlist, QOBJECT(obj)) +#define QLIST_FOREACH_ENTRY(qlist, var) \ + for ((var) = ((qlist)->head.tqh_first); \ + (var); \ + (var) = ((var)->next.tqe_next)) + +static inline QObject *qlist_entry_obj(const QListEntry *entry) +{ + return entry->value; +} + QList *qlist_new(void); QList *qlist_copy(QList *src); void qlist_append_obj(QList *qlist, QObject *obj); -- 1.6.6 ^ permalink raw reply related [flat|nested] 10+ messages in thread
end of thread, other threads:[~2010-01-20 16:39 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2010-01-05 20:34 [Qemu-devel] [PATCH v0 0/5]: Convert pci_info() to QObject Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 2/5] QDict: Introduce qdict_get_qdict() Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 3/5] PCI: Convert pci_info() to QObject Luiz Capitulino 2010-01-07 15:19 ` [Qemu-devel] " Anthony Liguori 2010-01-07 18:00 ` Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 4/5] PCI: do_pci_info(): PCI bridge support Luiz Capitulino 2010-01-05 20:34 ` [Qemu-devel] [PATCH 5/5] PCI: do_pci_info(): PCI bridge devices support Luiz Capitulino 2010-01-08 18:42 [Qemu-devel] [PATCH v1 0/5]: Convert pci_info() to QObject Luiz Capitulino 2010-01-08 18:42 ` [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() Luiz Capitulino 2010-01-20 16:39 [Qemu-devel] [PATCH v2 0/5]: Convert pci_info() to QObject Luiz Capitulino 2010-01-20 16:39 ` [Qemu-devel] [PATCH 1/5] QList: Introduce QLIST_FOREACH_ENTRY() Luiz Capitulino
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.