[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 1/4] qdev: using error_abort instead of using
From: |
Gonglei (Arei) |
Subject: |
Re: [Qemu-devel] [PATCH v4 1/4] qdev: using error_abort instead of using local_err |
Date: |
Tue, 2 Sep 2014 08:19:22 +0000 |
> From: Gonglei (Arei)
> Sent: Tuesday, September 02, 2014 4:04 PM
> Subject: [PATCH v4 1/4] qdev: using error_abort instead of using local_err
> Importance: High
>
> 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.
>
> Signed-off-by: Gonglei <address@hidden>
> ---
Oops, I forget to add 'Reviewed-by' Peter in this patch.
Next version will fix this. Sorry!
Best regards,
-Gonglei
> 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);
> }
>
> --
> 1.7.12.4
>
[Qemu-devel] [PATCH v4 4/4] pcie: don't assert when hotplug a PCIe device with 'function != 0', arei.gonglei, 2014/09/02