From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56436) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a9bWb-00079Q-3O for qemu-devel@nongnu.org; Thu, 17 Dec 2015 11:35:30 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a9bWX-0003rY-VJ for qemu-devel@nongnu.org; Thu, 17 Dec 2015 11:35:28 -0500 Received: from mx1.redhat.com ([209.132.183.28]:32964) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a9bWX-0003rI-7w for qemu-devel@nongnu.org; Thu, 17 Dec 2015 11:35:25 -0500 Received: from int-mx13.intmail.prod.int.phx2.redhat.com (int-mx13.intmail.prod.int.phx2.redhat.com [10.5.11.26]) by mx1.redhat.com (Postfix) with ESMTPS id DB854693D8 for ; Thu, 17 Dec 2015 16:35:24 +0000 (UTC) Received: from blackfin.pond.sub.org ([10.3.113.12]) by int-mx13.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id tBHGZMfD025801 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO) for ; Thu, 17 Dec 2015 11:35:24 -0500 From: Markus Armbruster Date: Thu, 17 Dec 2015 17:35:08 +0100 Message-Id: <1450370121-5768-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 v3 00/13] Clean up some hw_error() misuse List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org I intend to take this through my tree, but I'm certainly fine with maintainers picking up their parts if that makes their job easier. Just be clear about it. v3: * PATCH 02: Cover a missed instance, fix botched code motion [Thomas] * PATCH 11: Add some assertions [Marcel, Michael] v2: * PATCH 01: Supply missing exit() [Thomas] * PATCH 02: Improve commit message [Peter] * PATCH 03: Fix up error message [Peter] * PATCH 06: New, to unbreak the next patch [Peter] Markus Armbruster (13): hw: Don't use hw_error() for machine initialization errors omap: Don't use hw_error() in device init() methods arm_mptimer: Don't use hw_error() in realize() method etraxfs_eth: Don't use hw_error() in init() method raven: Mark use of hw_error() in realize() FIXME error: Don't append a newline when printing the error hint hw/arm/virt: Fix property "gic-version" error handling sysbus: Don't use hw_error() in machine_init_done_notifiers isa: Trivially convert remaining PCI-ISA bridges to realize() isa: Clean up error handling around isa_bus_new() isa: Clean up inappropriate hw_error() audio: Clean up inappropriate and unreachable use of hw_error() xen-hvm: Mark inappropriate error handling FIXME audio/audio.c | 11 ++--------- hw/alpha/dp264.c | 11 ++++++----- hw/alpha/typhoon.c | 3 ++- hw/arm/highbank.c | 6 ++++-- hw/arm/virt.c | 5 ++--- hw/char/exynos4210_uart.c | 9 ++++++--- hw/core/platform-bus.c | 25 ++++++++++++------------- hw/gpio/omap_gpio.c | 29 +++++++++++++++++++++-------- hw/i2c/omap_i2c.c | 8 ++++++-- hw/i386/pc_piix.c | 3 ++- hw/intc/omap_intc.c | 10 +++++++--- hw/isa/i82378.c | 5 ++++- hw/isa/isa-bus.c | 18 +++++------------- hw/isa/lpc_ich9.c | 11 +++++++---- hw/isa/piix4.c | 6 ++++-- hw/isa/vt82c686.c | 5 ++++- hw/m68k/an5206.c | 4 +++- hw/mips/mips_jazz.c | 2 +- hw/mips/mips_r4k.c | 2 +- hw/net/etraxfs_eth.c | 4 +++- hw/pci-host/piix.c | 6 ++++-- hw/pci-host/prep.c | 6 ++++-- hw/ppc/mac_newworld.c | 11 ++++++----- hw/ppc/mac_oldworld.c | 16 +++++++++------- hw/ppc/prep.c | 11 +++++++---- hw/sparc64/sun4u.c | 12 ++++++------ hw/timer/arm_mptimer.c | 5 +++-- hw/unicore32/puv3.c | 10 +++++++--- include/hw/isa/isa.h | 2 +- qdev-monitor.c | 2 ++ util/error.c | 2 +- util/qemu-option.c | 4 ++-- xen-hvm.c | 7 +++++++ 33 files changed, 161 insertions(+), 110 deletions(-) --=20 2.4.3