[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH memory v3 6/9] memory: MemoryRegion: QOMify
From: |
Peter Crosthwaite |
Subject: |
Re: [Qemu-devel] [PATCH memory v3 6/9] memory: MemoryRegion: QOMify |
Date: |
Tue, 3 Jun 2014 08:39:56 +1000 |
On Mon, Jun 2, 2014 at 10:18 PM, Peter Maydell <address@hidden> wrote:
> On 2 June 2014 05:17, Peter Crosthwaite <address@hidden> wrote:
>> QOMify memory regions as an Object. The former init() and destroy()
>> routines become instance_init() and instance_finalize() resp.
>>
>> memory_region_init() is re-implemented to be:
>> object_initialize() + set fields
>
>> +static void memory_region_initfn(Object *obj)
>> +{
>> + MemoryRegion *mr = MEMORY_REGION(obj);
>> +
>> + mr->ops = &unassigned_mem_ops;
>> mr->enabled = true;
>> - mr->terminates = false;
>> - mr->ram = false;
>> mr->romd_mode = true;
>> - mr->readonly = false;
>> - mr->rom_device = false;
>> mr->destructor = memory_region_destructor_none;
>> - mr->priority = 0;
>> - mr->may_overlap = false;
>> - mr->alias = NULL;
>> QTAILQ_INIT(&mr->subregions);
>> memset(&mr->subregions_link, 0, sizeof mr->subregions_link);
>
> We rely on QOM objects being zero-initialized for other
> fields, so why leave the explicit memset for this one?
>
It slipped through the cracks in my scan for 0s. Will Fix.
Regards,
Peter
> thanks
> -- PMM
>
- [Qemu-devel] [PATCH memory v3 0/9] Memory Region QOMification, Peter Crosthwaite, 2014/06/02
- [Qemu-devel] [PATCH memory v3 1/9] memory: Simplify mr_add_subregion() if-else, Peter Crosthwaite, 2014/06/02
- [Qemu-devel] [PATCH memory v3 2/9] qom: object: Ignore refs/unrefs of NULL, Peter Crosthwaite, 2014/06/02
- [Qemu-devel] [PATCH memory v3 3/9] qom: Publish object_resolve_link, Peter Crosthwaite, 2014/06/02
- [Qemu-devel] [PATCH memory v3 4/9] memory: Coreify subregion add functionality, Peter Crosthwaite, 2014/06/02
- [Qemu-devel] [PATCH memory v3 5/9] memory: MemoryRegion: factor out memory region re-adder, Peter Crosthwaite, 2014/06/02
- [Qemu-devel] [PATCH memory v3 6/9] memory: MemoryRegion: QOMify, Peter Crosthwaite, 2014/06/02
- [Qemu-devel] [PATCH memory v3 7/9] memory: MemoryRegion: Add container and addr props, Peter Crosthwaite, 2014/06/02
- [Qemu-devel] [PATCH memory v3 8/9] memory: MemoryRegion: Add may-overlap and priority props, Peter Crosthwaite, 2014/06/02
- [Qemu-devel] [PATCH memory v3 9/9] memory: MemoryRegion: Add size property, Peter Crosthwaite, 2014/06/02
- Re: [Qemu-devel] [PATCH memory v3 0/9] Memory Region QOMification, Peter Maydell, 2014/06/02