[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC 0/3] qom: Generalize qdev init to "realize"
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH RFC 0/3] qom: Generalize qdev init to "realize" |
Date: |
Tue, 27 Mar 2012 17:33:38 +0100 |
On 27 March 2012 17:28, Anthony Liguori <address@hidden> wrote:
> On 03/27/2012 11:19 AM, Paolo Bonzini wrote:
>> I think a major difference between realize and init should be that the
>> realize property also propagates down the whole composition tree (in
>> pre-order for setting to true, and post-order for setting to false).
>
>
> Yes, I haven't reviewed this series yet, but my expectation would be that
> realize propagates and that the default implementation of
> DeviceClass::realize() would explicitly *not* propagate and instead just
> call ::init.
So who calls realize for non-qdev QOM objects which are children of
qdev QOM objects?
I really don't like having the object lifecycle methods be different
for DeviceClass than for base objects if we can avoid it.
-- PMM
- [Qemu-devel] [PATCH RFC 0/3] qom: Generalize qdev init to "realize", Andreas Färber, 2012/03/26
- [Qemu-devel] [PATCH RFC 2/3] qom: Introduce object_realize(), Andreas Färber, 2012/03/26
- [Qemu-devel] [PATCH RFC 3/3] qdev: Hook up DeviceClass::init to ObjectClass::realize, Andreas Färber, 2012/03/26
- [Qemu-devel] [PATCH RFC 1/3] qom: Add "realized" property to Object, Andreas Färber, 2012/03/26
- Re: [Qemu-devel] [PATCH RFC 0/3] qom: Generalize qdev init to "realize", Paolo Bonzini, 2012/03/27
- Re: [Qemu-devel] [PATCH RFC 0/3] qom: Generalize qdev init to "realize", Paolo Bonzini, 2012/03/27