From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37234) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YLDlb-0000va-Nk for qemu-devel@nongnu.org; Tue, 10 Feb 2015 11:34:32 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YLDlW-0006wy-2v for qemu-devel@nongnu.org; Tue, 10 Feb 2015 11:34:27 -0500 Received: from mx1.redhat.com ([209.132.183.28]:39927) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YLDlV-0006wg-Rh for qemu-devel@nongnu.org; Tue, 10 Feb 2015 11:34:22 -0500 From: Markus Armbruster Date: Tue, 10 Feb 2015 17:34:06 +0100 Message-Id: <1423586055-4932-1-git-send-email-armbru@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PATCH 0/9] Clean up around error_get_pretty(), qerror_report_err() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , Peter Maydell , "Michael S. Tsirkin" , Jason Wang , Gerd Hoffmann , Stefan Hajnoczi , =?UTF-8?q?Andreas=20F=C3=A4rber?= Tree-wide cleanup, but the tree-wide changes are mechanical. If nobody objects, I'll route it through my own tree. Depends on my "[PATCH] vhost-scsi: Improve error reporting for invalid vhostfd". PATCH 01 is responsible for the bulk of the diffstat. I could split off one or more of block, arm, x86+pci, scsi, usb, char, net for maintainers to pick up after the main patch got in. Cc'ing a few maintainers who might prefer that, to give them a chance to ask for it. PATCH 02 gets rid of an unnecessary use of qerror_report_err(). This is the patch that causes the dependency mentioned above. The remaining patches clean up inappropriate uses of qerror_report_err(). Cc: Kevin Wolf Cc: Stefan Hajnoczi Cc: Peter Maydell Cc: Michael S. Tsirkin Cc: Jason Wang Cc: Gerd Hoffmann Cc: Andreas F=C3=A4rber Markus Armbruster (9): error: New convenience function error_report_err() monitor: Clean up around monitor_handle_fd_param() monitor: Avoid qerror_report_err() outside QMP command handlers net: Avoid qerror_report_err() outside QMP command handlers numa: Avoid qerror_report_err() outside QMP command handlers tpm: Avoid qerror_report_err() outside QMP command handlers vl: Avoid qerror_report_err() outside QMP command handlers qemu-img: Avoid qerror_report_err() outside QMP command handlers qemu-char: Avoid qerror_report_err() outside QMP command handlers arch_init.c | 3 +-- block/sheepdog.c | 39 +++++++++++++-------------------------- blockdev.c | 12 ++++-------- hw/arm/exynos4210.c | 4 ++-- hw/arm/highbank.c | 4 ++-- hw/arm/integratorcp.c | 4 ++-- hw/arm/realview.c | 6 +++--- hw/arm/versatilepb.c | 4 ++-- hw/arm/vexpress.c | 2 +- hw/arm/xilinx_zynq.c | 8 ++++---- hw/block/virtio-blk.c | 3 +-- hw/char/serial.c | 6 ++---- hw/i386/kvm/pci-assign.c | 14 +++++--------- hw/i386/pc.c | 5 ++--- hw/i386/smbios.c | 14 +++++++------- hw/ide/qdev.c | 3 +-- hw/pci/pci-hotplug-old.c | 3 +-- hw/pci/pci.c | 3 +-- hw/scsi/vhost-scsi.c | 2 +- hw/usb/dev-network.c | 3 +-- hw/usb/host-libusb.c | 3 +-- hw/usb/redirect.c | 3 +-- include/monitor/monitor.h | 3 +-- include/qapi/error.h | 5 +++++ monitor.c | 18 ++---------------- net/net.c | 9 +++------ net/socket.c | 4 +++- net/tap.c | 11 ++++++++--- numa.c | 8 +++----- qemu-char.c | 6 ++---- qemu-img.c | 9 +++------ qemu-io.c | 3 +-- qemu-nbd.c | 12 ++++-------- qom/cpu.c | 3 +-- target-i386/cpu.c | 3 +-- target-sparc/cpu.c | 3 +-- tpm.c | 6 ++---- util/error.c | 14 ++++++++++---- util/qemu-config.c | 9 +++------ vl.c | 18 +++++++----------- 40 files changed, 118 insertions(+), 174 deletions(-) --=20 1.9.3