[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 1/4] qdev: using error_abort instead of using
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v5 1/4] qdev: using error_abort instead of using local_err |
Date: |
Wed, 03 Sep 2014 16:29:47 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.7.0 |
Am 02.09.2014 14:03, schrieb address@hidden:
> From: Gonglei <address@hidden>
>
> This error can not happen normally. If it happens indicates
> something very wrong, we should abort QEMU. moreover, The
> user can only refer to /machine/peripheral, not
> /machine/unattached.
>
> Meanwhile remove superfluous check about local_err.
>
> Signed-off-by: Gonglei <address@hidden>
> Reviewed-by: Peter Crosthwaite <address@hidden>
> ---
> hw/core/qdev.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/core/qdev.c b/hw/core/qdev.c
> index da1ba48..4a1ac5b 100644
> --- a/hw/core/qdev.c
> +++ b/hw/core/qdev.c
> @@ -820,13 +820,13 @@ static void device_set_realized(Object *obj, bool
> value, Error **errp)
> }
>
> if (value && !dev->realized) {
> - if (!obj->parent && local_err == NULL) {
> + if (!obj->parent) {
> static int unattached_count;
> gchar *name = g_strdup_printf("device[%d]", unattached_count++);
>
> object_property_add_child(container_get(qdev_get_machine(),
> "/unattached"),
> - name, obj, &local_err);
> + name, obj, &error_abort);
> g_free(name);
> }
>
One way to trigger this abort would be for the user to add a property of
that format via -object or similar mechanisms, when allowing a custom
path. Today, objects are always under /objects though.
In fact the only way to get an Error** back from
object_property_add[_child]() is if a duplicate property exists. Any
other error conditions would already abort in GLib code.
Anyway, this is not strictly an error fix and it's a question of how
much we want to protect the user from his own stupidity, so I'm leaning
towards applying this patch. I'd prefer to decouple it from the other
cleanup though.
Regards,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- [Qemu-devel] [PATCH v5 0/4] Refactor device_set_realized to avoid resource leak., arei.gonglei, 2014/09/02
- [Qemu-devel] [PATCH v5 3/4] qdev: add cleanup logic in device_set_realized() to avoid resource leak, arei.gonglei, 2014/09/02
- [Qemu-devel] [PATCH v5 1/4] qdev: using error_abort instead of using local_err, arei.gonglei, 2014/09/02
- Re: [Qemu-devel] [PATCH v5 1/4] qdev: using error_abort instead of using local_err,
Andreas Färber <=
- [Qemu-devel] [PATCH v5 2/4] qdev: using NULL instead of local_err for qbus_child unrealize, arei.gonglei, 2014/09/02
- [Qemu-devel] [PATCH v5 4/4] pcie: don't assert when hotplug a PCIe device with 'function != 0', arei.gonglei, 2014/09/02
- Re: [Qemu-devel] [PATCH v5 0/4] Refactor device_set_realized to avoid resource leak., Michael S. Tsirkin, 2014/09/02