[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC 2/5] qdev: Avoid qerror_report_err() outside
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH RFC 2/5] qdev: Avoid qerror_report_err() outside QMP command handlers |
Date: |
Mon, 02 Mar 2015 11:11:32 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
zhanghailiang <address@hidden> writes:
> qerror_report_err() is a transitional interface to help with
> converting existing monitor commands to QMP. It should not be used
> elsewhere. Replace by error_report_err() in qdev_init() and
> device_post_init().
>
> Signed-off-by: zhanghailiang <address@hidden>
> ---
> hw/core/qdev.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/hw/core/qdev.c b/hw/core/qdev.c
> index 44c6b93..9adcd27 100644
> --- a/hw/core/qdev.c
> +++ b/hw/core/qdev.c
> @@ -181,8 +181,7 @@ int qdev_init(DeviceState *dev)
>
> object_property_set_bool(OBJECT(dev), true, "realized", &local_err);
> if (local_err != NULL) {
> - qerror_report_err(local_err);
> - error_free(local_err);
> + error_report_err(local_err);
> object_unparent(OBJECT(dev));
> return -1;
> }
This breaks QMP device_add's error reply.
The patches I posted together remove all calls of qdev_init(). Once
they're all merged, the function should be simply dropped.
> @@ -1189,8 +1188,7 @@ static void device_post_init(Object *obj)
> Error *err = NULL;
> qdev_prop_set_globals(DEVICE(obj), &err);
> if (err) {
> - qerror_report_err(err);
> - error_free(err);
> + error_report_err(err);
> exit(EXIT_FAILURE);
> }
> }
This one is part of a wider problem I solved in
[PATCH] qdev: Don't exit when running into bad -global
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH RFC 2/5] qdev: Avoid qerror_report_err() outside QMP command handlers,
Markus Armbruster <=