[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 Maydell |
Subject: |
Re: [Qemu-devel] [PATCH memory v3 6/9] memory: MemoryRegion: QOMify |
Date: |
Mon, 2 Jun 2014 13:18:50 +0100 |
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?
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
- Re: [Qemu-devel] [PATCH memory v3 6/9] memory: MemoryRegion: QOMify,
Peter Maydell <=
- [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