[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 2/8] qdev: qdev_create(): use error_report() inste
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [RFC 2/8] qdev: qdev_create(): use error_report() instead of hw_error() |
Date: |
Tue, 04 Dec 2012 17:12:28 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 |
Am 04.12.2012 14:19, schrieb Eduardo Habkost:
> hw_error() is specific for fatal hardware emulation errors, not for
> internal errors related to the qdev object/class abstraction or object
> initialization.
>
> Replace it with an error_report() call, followed by abort().
>
> This will also help reduce dependencies of the qdev code (as hw_error()
> is from cpus.o, and depends on the CPU list from exec.o).
>
> Signed-off-by: Eduardo Habkost <address@hidden>
This was already posted as a PATCH and I thought I had acked it ...
seems I didn't, so I re-checked cpus.c:hw_error() to abort() as well:
Acked-by: Andreas Färber <address@hidden>
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] [RFC 0/8] CPU DeviceState v9, Eduardo Habkost, 2012/12/04
- [Qemu-devel] [RFC 2/8] qdev: qdev_create(): use error_report() instead of hw_error(), Eduardo Habkost, 2012/12/04
- Re: [Qemu-devel] [RFC 2/8] qdev: qdev_create(): use error_report() instead of hw_error(),
Andreas Färber <=
- [Qemu-devel] [RFC 4/8] libqemustub: vmstate register/unregister stubs, Eduardo Habkost, 2012/12/04
- [Qemu-devel] [RFC 7/8] include qdev code into *-user, too, Eduardo Habkost, 2012/12/04
- [Qemu-devel] [RFC 8/8] qom: make CPU a child of DeviceState, Eduardo Habkost, 2012/12/04
- [Qemu-devel] [RFC 3/8] libqemustub: add qemu_[un]register_reset() stubs, Eduardo Habkost, 2012/12/04
- [Qemu-devel] [RFC 1/8] move -I$(SRC_PATH)/include compiler flag to Makefile.objs, Eduardo Habkost, 2012/12/04
- [Qemu-devel] [RFC 5/8] libqemustub: sysbus_get_default() stub, Eduardo Habkost, 2012/12/04