[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 0/8] CPU DeviceState v9
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [RFC 0/8] CPU DeviceState v9 |
Date: |
Tue, 04 Dec 2012 16:59:38 +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:
> Changes on v9:
> - Instead of moving qemu_[un]register_reset() to reset.c and including
> it on *-user, create stubs for them on libqemustub.a
We compile cpu.c twice. Can't we do the same for qdev.c or whatever uses
those functions? I feel they have no business being used in *-user.
CC'ing Riku and Peter.
Andreas
> - This is based on afaerber's qom-cpu branch, that has some header cleanup
> changes. You can get the complete series in a git tree at:
> https://github.com/ehabkost/qemu-hacks/tree/cpu_qdev.v9
> git://github.com/ehabkost/qemu-hacks.git cpu_qdev.v9
>
> v8:
> - Use a simpler copyright header on qdev-properties-system.c
> - Use the new libqemustub.a mechanism instead of the (now exting)
> QEMU_WEAK_ALIAS mechanism
> - Move the reset-handler registration code to a new hw/reset.c file
>
> v7:
> - Use the new QEMU_WEAK_ALIAS mechanism instead of the (now extinct)
> GCC_WEAK attribute (patches 20 and 21)
>
> v6:
> - Simple rebase against latest qemu.git master
> - Patch 13: some new typedefs were added and others were removed
> - Patch 19: trivial rebase
> v5:
> - Tons of header cleanups just to eliminate qlist.h <-> cpu-common.h circular
> dependency (patches 1-17)
> - Add copyright/license information to qdev-properties.c (patch 17)
> - Add copyright/license information to qdev-properties-system.c (patch 22)
> - use error_report()+abort() instead of hw_error() on qdev.c (patch 18)
> - Move qemu_[un]register_reset() and qemu_devices_reset() to qdev-core.c
> (patch 19)
> - Make vmstate_[un]register() weak stubs, instead of a new function (patch
> 20)
> - Make sysbus_get_default() weak stub, instead of new qbus reset (un)register
> functions (patch 21)
> - Eliminate qdev-system.c (all code is kept on qdev.c, now) (patch 22)
> v4:
> - Add GCC_WEAK_DECL to functions that have GCC_WEAK versions
> - Updated the qdev_init_gpio_in() code on qdev-system.c to current version
> - Patch description updates (moved changelog below "---" and/or move info
> about changes made by different authors between SoB lines)
> v3 (submitted by Igor):
> - rebased on top of 8b4a3df (today's master)
> - slight code reshuffling in (see commit's changelog)
> "qdev: separate core from the code used only by qemu-system-*"
> "move qemu_irq typedef out of cpu-common.h"
> - commit messages cleanup
> v2:
> Removes the CONFIG_USER_ONLY ifdefs, and use weak symbols to move
> the vmstate and qemu_register_reset() handling to qdev-system.c
>
> git tree for testing:
> https://github.com/ehabkost/qemu-hacks/tree/cpu_qdev.v9
> git://github.com/ehabkost/qemu-hacks.git cpu_qdev.v9
>
> References to previous versions:
> v8: http://article.gmane.org/gmane.comp.emulators.qemu/182589
> v7: http://article.gmane.org/gmane.comp.emulators.qemu/179969
> v6: http://article.gmane.org/gmane.comp.emulators.qemu/179918
> v5: http://article.gmane.org/gmane.comp.emulators.qemu/177426
> v4: http://article.gmane.org/gmane.comp.emulators.qemu/176127
> v3: http://article.gmane.org/gmane.comp.emulators.qemu/175980
> v2: http://article.gmane.org/gmane.comp.emulators.qemu/173909
> v1: http://article.gmane.org/gmane.comp.emulators.qemu/166630
>
>
> Eduardo Habkost (7):
> move -I$(SRC_PATH)/include compiler flag to Makefile.objs
> qdev: qdev_create(): use error_report() instead of hw_error()
> libqemustub: add qemu_[un]register_reset() stubs
> libqemustub: vmstate register/unregister stubs
> libqemustub: sysbus_get_default() stub
> qdev-properties.c: separate core from the code used only by
> qemu-system-*
> include qdev code into *-user, too
>
> Igor Mammedov (1):
> qom: make CPU a child of DeviceState
>
> Makefile | 1 -
> Makefile.objs | 23 ++-
> hw/Makefile.objs | 10 +-
> hw/qdev-properties-system.c | 352
> ++++++++++++++++++++++++++++++++++++++++++++
> hw/qdev-properties.c | 321 +---------------------------------------
> hw/qdev-properties.h | 1 +
> hw/qdev.c | 21 +--
> include/qemu/cpu.h | 6 +-
> qom/cpu.c | 3 +-
> stubs/Makefile.objs | 3 +
> stubs/reset.c | 13 ++
> stubs/sysbus.c | 6 +
> stubs/vmstate.c | 17 +++
> 13 files changed, 428 insertions(+), 349 deletions(-)
> create mode 100644 hw/qdev-properties-system.c
> create mode 100644 stubs/reset.c
> create mode 100644 stubs/sysbus.c
> create mode 100644 stubs/vmstate.c
>
--
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 1/8] move -I$(SRC_PATH)/include compiler flag to Makefile.objs, (continued)
Re: [Qemu-devel] [RFC 0/8] CPU DeviceState v9,
Andreas Färber <=