From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41160) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cCCDu-0003xS-OQ for qemu-devel@nongnu.org; Wed, 30 Nov 2016 16:15:27 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cCCDp-0006w5-Oi for qemu-devel@nongnu.org; Wed, 30 Nov 2016 16:15:26 -0500 Received: from mx1.redhat.com ([209.132.183.28]:46608) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1cCCDp-0006vT-Fk for qemu-devel@nongnu.org; Wed, 30 Nov 2016 16:15:21 -0500 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 94C986611 for ; Wed, 30 Nov 2016 21:15:20 +0000 (UTC) Date: Wed, 30 Nov 2016 23:15:16 +0200 From: "Michael S. Tsirkin" Message-ID: <20161130231224-mutt-send-email-mst@kernel.org> References: <1480535094-23831-1-git-send-email-eblake@redhat.com> <1480535094-23831-2-git-send-email-eblake@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1480535094-23831-2-git-send-email-eblake@redhat.com> Subject: Re: [Qemu-devel] [PATCH 01/36] pci: Use struct instead of QDict to pass back parameters List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake Cc: qemu-devel@nongnu.org, armbru@redhat.com, pbonzini@redhat.com, Marcel Apfelbaum On Wed, Nov 30, 2016 at 01:44:19PM -0600, Eric Blake wrote: > It's simpler to just use a C struct than it is to bundle things > into a QDict in one function just to pull them back out in the > caller. Plus, doing this gets rid of one more user of dynamic > JSON through qobject_from_jsonf(). > > Signed-off-by: Eric Blake Acked-by: Michael S. Tsirkin > --- > hw/pci/pcie_aer.c | 36 +++++++++++++++++------------------- > 1 file changed, 17 insertions(+), 19 deletions(-) > > diff --git a/hw/pci/pcie_aer.c b/hw/pci/pcie_aer.c > index 048ce6a..0735796 100644 > --- a/hw/pci/pcie_aer.c > +++ b/hw/pci/pcie_aer.c > @@ -43,6 +43,13 @@ > #define PCI_ERR_SRC_COR_OFFS 0 > #define PCI_ERR_SRC_UNCOR_OFFS 2 > > +typedef struct PCIEErrorInject { > + const char *id; > + const char *root_bus; > + int bus; > + int devfn; > +} PCIEErrorInject; > + > /* From 6.2.7 Error Listing and Rules. Table 6-2, 6-3 and 6-4 */ > static uint32_t pcie_aer_uncor_default_severity(uint32_t status) > { > @@ -946,7 +953,8 @@ static int pcie_aer_parse_error_string(const char *error_name, > } > > static int do_pcie_aer_inject_error(Monitor *mon, > - const QDict *qdict, QObject **ret_data) > + const QDict *qdict, > + PCIEErrorInject *ret_data) > { > const char *id = qdict_get_str(qdict, "id"); > const char *error_name; > @@ -1007,34 +1015,24 @@ static int do_pcie_aer_inject_error(Monitor *mon, > err.prefix[2] = qdict_get_try_int(qdict, "prefix2", 0); > err.prefix[3] = qdict_get_try_int(qdict, "prefix3", 0); > > - ret = pcie_aer_inject_error(dev, &err); > - *ret_data = qobject_from_jsonf("{'id': %s, " > - "'root_bus': %s, 'bus': %d, 'devfn': %d, " > - "'ret': %d}", > - id, pci_root_bus_path(dev), > - pci_bus_num(dev->bus), dev->devfn, > - ret); > - assert(*ret_data); > + pcie_aer_inject_error(dev, &err); > + ret_data->id = id; > + ret_data->root_bus = pci_root_bus_path(dev); > + ret_data->bus = pci_bus_num(dev->bus); > + ret_data->devfn = dev->devfn; > > return 0; > } > > void hmp_pcie_aer_inject_error(Monitor *mon, const QDict *qdict) > { > - QObject *data; > - int devfn; > + PCIEErrorInject data; > > if (do_pcie_aer_inject_error(mon, qdict, &data) < 0) { > return; > } > > - assert(qobject_type(data) == QTYPE_QDICT); > - qdict = qobject_to_qdict(data); > - > - devfn = (int)qdict_get_int(qdict, "devfn"); > monitor_printf(mon, "OK id: %s root bus: %s, bus: %x devfn: %x.%x\n", > - qdict_get_str(qdict, "id"), > - qdict_get_str(qdict, "root_bus"), > - (int) qdict_get_int(qdict, "bus"), > - PCI_SLOT(devfn), PCI_FUNC(devfn)); > + data.id, data.root_bus, data.bus, > + PCI_SLOT(data.devfn), PCI_FUNC(data.devfn)); > } > -- > 2.7.4