[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 04/13] qom: delete properties before calling
From: |
Peter Crosthwaite |
Subject: |
Re: [Qemu-devel] [RFC PATCH 04/13] qom: delete properties before calling instance_finalize |
Date: |
Tue, 17 Jun 2014 22:29:24 +1000 |
On Thu, Jun 12, 2014 at 12:03 AM, Paolo Bonzini <address@hidden> wrote:
> Il 11/06/2014 15:42, Peter Crosthwaite ha scritto:
>
>>> >
>>> > - object_deinit(obj, ti);
>>> > object_property_del_all(obj);
>>> > + object_deinit(obj, ti);
>>
>> Does this means you lose access to your own properties before calling
>> your own finalisation? I guess its a non-issue if all clients just use
>> private fields directly, but it seems weird you cant use your own
>> getter at the end because it's dissappeared on you already.
>
>
> It is just as weird that a property's finalizer cannot do its work because
> it needs something that instance_finalize has already freed. :)
>
> This is important with the previous patch, that calls arbitrary code of
> children objects at finalization time.
>
Ok fair enough.
Acked-by: Peter Crosthwaite <address@hidden>
> Paolo
>
- Re: [Qemu-devel] [RFC PATCH 03/13] qom: move unparenting to the child property's release callback, (continued)
[Qemu-devel] [RFC PATCH 04/13] qom: delete properties before calling instance_finalize, Paolo Bonzini, 2014/06/11
[Qemu-devel] [RFC PATCH 11/13] memory: MemoryRegion: Add container and addr props, Paolo Bonzini, 2014/06/11
[Qemu-devel] [RFC PATCH 07/13] memory: MemoryRegion: use /machine as default owner, Paolo Bonzini, 2014/06/11
[Qemu-devel] [RFC PATCH 08/13] memory: MemoryRegion: rename parent to container, Paolo Bonzini, 2014/06/11
[Qemu-devel] [RFC PATCH 12/13] memory: MemoryRegion: Add may-overlap and priority props, Paolo Bonzini, 2014/06/11
[Qemu-devel] [RFC PATCH 13/13] memory: MemoryRegion: Add size property, Paolo Bonzini, 2014/06/11
[Qemu-devel] [RFC PATCH 10/13] memory: MemoryRegion: replace owner field with QOM parent, Paolo Bonzini, 2014/06/11
Re: [Qemu-devel] [RFC PATCH 00/13] "Light" memory region QOMification, Peter Crosthwaite, 2014/06/17