[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v1 1/7] memory: fix address space initialization/d
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [RFC v1 1/7] memory: fix address space initialization/destruction |
Date: |
Thu, 11 Oct 2012 15:31:16 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120911 Thunderbird/15.0.1 |
Il 11/10/2012 15:26, Avi Kivity ha scritto:
> A couple of fields were left uninitialized. This was not observed earlier
> because all address spaces were statically allocated. Also free allocation
> for those fields.
Patch is obvious, but there are at least two alternatives: 1) using
memset 2) using a compound initializer.
Paolo
> Signed-off-by: Avi Kivity <address@hidden>
> ---
> memory.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/memory.c b/memory.c
> index 2f68d67..5df6177 100644
> --- a/memory.c
> +++ b/memory.c
> @@ -1538,6 +1538,8 @@ void address_space_init(AddressSpace *as, MemoryRegion
> *root)
> as->root = root;
> as->current_map = g_new(FlatView, 1);
> flatview_init(as->current_map);
> + as->ioeventfd_nb = 0;
> + as->ioeventfds = NULL;
> QTAILQ_INSERT_TAIL(&address_spaces, as, address_spaces_link);
> as->name = NULL;
> memory_region_transaction_commit();
> @@ -1554,6 +1556,7 @@ void address_space_destroy(AddressSpace *as)
> address_space_destroy_dispatch(as);
> flatview_destroy(as->current_map);
> g_free(as->current_map);
> + g_free(as->ioeventfds);
> }
>
> uint64_t io_mem_read(MemoryRegion *mr, target_phys_addr_t addr, unsigned
> size)
>
- [Qemu-devel] [RFC v1 0/7] IOMMU support, Avi Kivity, 2012/10/11
- [Qemu-devel] [RFC v1 3/7] memory: iommu support, Avi Kivity, 2012/10/11
- Re: [Qemu-devel] [RFC v1 3/7] memory: iommu support, Paolo Bonzini, 2012/10/11
- Re: [Qemu-devel] [RFC v1 3/7] memory: iommu support, Avi Kivity, 2012/10/11
- Re: [Qemu-devel] [RFC v1 3/7] memory: iommu support, Paolo Bonzini, 2012/10/11
- Re: [Qemu-devel] [RFC v1 3/7] memory: iommu support, Avi Kivity, 2012/10/11
- Re: [Qemu-devel] [RFC v1 3/7] memory: iommu support, Benjamin Herrenschmidt, 2012/10/11
- Re: [Qemu-devel] [RFC v1 3/7] memory: iommu support, Avi Kivity, 2012/10/15
- Re: [Qemu-devel] [RFC v1 3/7] memory: iommu support, Benjamin Herrenschmidt, 2012/10/11