qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 00/21] qbus QOM conversion, rebased on top of my


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH 00/21] qbus QOM conversion, rebased on top of my patches
Date: Fri, 04 May 2012 18:15:13 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120425 Thunderbird/12.0

Il 02/05/2012 13:30, Paolo Bonzini ha scritto:
> Here is Anthony's qbus conversion, with the qdev_add_properties changes
> replaced by my bus property changes.
> 
> I left out the removal of bus methods; I'm not sure I agree with all of
> them (though I may just give up :)) and anyway they can be left to 1.2.
> 
> Following Andreas's remark, I moved all TYPE_* definitions for buses
> to the same file that defines the struct, and added typecasting macros.
> 
> I'm still a bit weary of putting this in this late, but I'll defer to
> others on this too.
> 
> Anthony Liguori (6):
>   qdev: fix adding of ptr properties
>   qdev: use wrapper for qdev_get_path
>   qdev: convert busses to QEMU Object Model
>   qdev: connect busses with their parent devices
>   qbus: make child devices links
>   qbus: initialize in standard way
> 
> Paolo Bonzini (15):
>   qom: documentation addition
>   qom: add object_class_get_parent
>   qom: add class_base_init
>   qom: make Object a type
>   qom: assert that public types have a non-NULL parent field
>   qdev: push "type" property up to Object
>   qdev: fix -device foo,?
>   qdev: use object_property_print in info qtree
>   qdev: move bus properties to a separate global
>   qdev: do not propagate properties to subclasses
>   qdev: move bus properties to abstract superclasses
>   pc: add back PCI.rombar compat property
>   qdev: clean up global properties
>   qdev: remove qdev_prop_set_defaults
>   qdev: move sysbus initialization to sysbus.c

Guys, I'll let you decide what to do about this.  If anyone wnts to pick
up the reviews and resubmit in my place, that would be nice.

In any case, patches 7-8 should be blockers for rc1.

Paolo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]