Re: [Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-12-02 Thread Peter Maydell
On 2 December 2012 05:44, Andreas Färber wrote: > Am 01.12.2012 12:26, schrieb Peter Maydell: >> On 30 November 2012 21:38, Eduardo Habkost wrote: >>> cpu_reset() is not that well-defined, otherwise we wouldn't have this on >>> linux-user: >>> >>> #if defined(TARGET_I386) || defined(TARGET_SPARC)

Re: [Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-12-01 Thread Andreas Färber
Am 01.12.2012 12:26, schrieb Peter Maydell: > On 30 November 2012 21:38, Eduardo Habkost wrote: >> cpu_reset() is not that well-defined, otherwise we wouldn't have this on >> linux-user: >> >> #if defined(TARGET_I386) || defined(TARGET_SPARC) || defined(TARGET_PPC) >> cpu_reset(ENV_GET_CPU(env

Re: [Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-12-01 Thread Peter Maydell
On 30 November 2012 21:38, Eduardo Habkost wrote: > cpu_reset() is not that well-defined, otherwise we wouldn't have this on > linux-user: > > #if defined(TARGET_I386) || defined(TARGET_SPARC) || defined(TARGET_PPC) > cpu_reset(ENV_GET_CPU(env)); > #endif > > (I have no idea why we have that #

Re: [Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-11-30 Thread Eduardo Habkost
On Fri, Nov 30, 2012 at 05:56:26PM +0100, Igor Mammedov wrote: > On Thu, 15 Nov 2012 16:42:41 -0200 > Eduardo Habkost wrote: > > > On Thu, Nov 15, 2012 at 02:54:57AM +0100, Andreas Färber wrote: > > > Am 09.11.2012 15:56, schrieb Eduardo Habkost: > > > > The core qdev code uses the reset handler

Re: [Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-11-30 Thread Igor Mammedov
On Thu, 15 Nov 2012 16:42:41 -0200 Eduardo Habkost wrote: > On Thu, Nov 15, 2012 at 02:54:57AM +0100, Andreas Färber wrote: > > Am 09.11.2012 15:56, schrieb Eduardo Habkost: > > > The core qdev code uses the reset handler list from vl.c, so move > > > qemu_register_reset(), qemu_unregister_reset(

Re: [Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-11-15 Thread Peter Maydell
On 15 November 2012 18:42, Eduardo Habkost wrote: > DeviceState CPUs have to be reset too, and DeviceState uses the > reset-handler system to make sure DeviceState objects are reset, so it > won't be softmmu-specific. > > An alternative is to make empty stubs for qemu_[un]register_reset(), and > n

Re: [Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-11-15 Thread Eduardo Habkost
On Thu, Nov 15, 2012 at 02:54:57AM +0100, Andreas Färber wrote: > Am 09.11.2012 15:56, schrieb Eduardo Habkost: > > The core qdev code uses the reset handler list from vl.c, so move > > qemu_register_reset(), qemu_unregister_reset() and qemu_devices_reset() > > to qdev.c. > > > > The function decl

Re: [Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-11-14 Thread Andreas Färber
Am 09.11.2012 15:56, schrieb Eduardo Habkost: > The core qdev code uses the reset handler list from vl.c, so move > qemu_register_reset(), qemu_unregister_reset() and qemu_devices_reset() > to qdev.c. > > The function declarations were moved to a new qdev-reset.h file, that is > included by hw.h t

[Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-11-09 Thread Eduardo Habkost
The core qdev code uses the reset handler list from vl.c, so move qemu_register_reset(), qemu_unregister_reset() and qemu_devices_reset() to qdev.c. The function declarations were moved to a new qdev-reset.h file, that is included by hw.h to keep compatibility, so we don't need to change all files

[Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-11-09 Thread Eduardo Habkost
The core qdev code uses the reset handler list from vl.c, so move qemu_register_reset(), qemu_unregister_reset() and qemu_devices_reset() to qdev.c. The function declarations were moved to a new qdev-reset.h file, that is included by hw.h to keep compatibility, so we don't need to change all files

[Qemu-devel] [PATCH 19/24] qdev: move reset handler list from vl.c to qdev.c

2012-10-23 Thread Eduardo Habkost
The core qdev code uses the reset handler list from vl.c, so move qemu_register_reset(), qemu_unregister_reset() and qemu_devices_reset() to qdev.c. The function declarations were moved to a new qdev-reset.h file, that is included by hw.h to keep compatibility, so we don't need to change all files