From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44896) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YJ3pH-0005c2-2m for qemu-devel@nongnu.org; Wed, 04 Feb 2015 12:33:19 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YJ3p9-0004EQ-FA for qemu-devel@nongnu.org; Wed, 04 Feb 2015 12:33:19 -0500 Received: from mx1.redhat.com ([209.132.183.28]:45632) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YJ3p9-0004E8-8F for qemu-devel@nongnu.org; Wed, 04 Feb 2015 12:33:11 -0500 From: Markus Armbruster Date: Wed, 4 Feb 2015 18:33:00 +0100 Message-Id: <1423071188-1085-1-git-send-email-armbru@redhat.com> Subject: [Qemu-devel] [PATCH 0/8] Cleanups around unchecked qdev_init() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: afaerber@suse.de Board setup should normally use qdev_init_nofail(), not qdev_init(), because inability to create an onboard device is normally fatal. Board setup should certainly not ignore qdev_init() failure. Even when we know that realization can't actually fail now, qdev_init_nofail() is clearer and more robust. Correct a few obvious instances. Markus Armbruster (8): qdev: Improve qdev_init_nofail()'s error reporting ide/isa: Replace unchecked qdev_init() by qdev_init_nofail() leon3: Replace unchecked qdev_init() by qdev_init_nofail() etsec: Replace qdev_init() by qdev_init_nofail() serial: Factor out common serial_hds_isa_init() serial: serial_hds_isa_init() shouldn't fail parallel: Factor out common parallel_hds_isa_init() parallel: parallel_hds_isa_init() shouldn't fail hw/alpha/dp264.c | 6 +----- hw/char/parallel.c | 25 +++++++++++++++++++++++++ hw/char/serial-isa.c | 23 +++++++++++++++-------- hw/core/qdev.c | 11 ++++++++--- hw/i386/pc.c | 13 ++----------- hw/ide/isa.c | 4 +--- hw/mips/mips_fulong2e.c | 12 ++---------- hw/mips/mips_malta.c | 7 +++---- hw/mips/mips_r4k.c | 6 +----- hw/net/fsl_etsec/etsec.c | 5 +---- hw/sparc64/sun4u.c | 12 ++---------- include/hw/char/serial.h | 2 +- include/hw/i386/pc.h | 17 +---------------- include/hw/sparc/grlib.h | 12 +++--------- 14 files changed, 66 insertions(+), 89 deletions(-) -- 1.9.3