[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] memory: give name every AddressSpace
From: |
Alexey Kardashevskiy |
Subject: |
Re: [Qemu-devel] [PATCH] memory: give name every AddressSpace |
Date: |
Tue, 30 Apr 2013 12:14:49 +1000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130402 Thunderbird/17.0.5 |
On 04/29/2013 07:25 PM, Paolo Bonzini wrote:
> Il 29/04/2013 10:21, Alexey Kardashevskiy ha scritto:
>>>>>> + g_free((void *)as->name);
>>>>
>>>> No cast here.
>> ?
>>
>> CC ppc64-softmmu/memory.o
>> /home/alexey/pcipassthru/qemu-impreza/memory.c: In function
>> 'address_space_destroy':
>> /home/alexey/pcipassthru/qemu-impreza/memory.c:1626:5: warning: passing
>> argument 1 of 'g_free' discards 'const' qualifier from pointer target type
>> [enabled by default]
>> g_free(/*(void *)*/as->name);
>> ^
>
> Please remove the const from as->name instead. Since you are strdup-ing
> it, and the field is meant to be private to memory.c anyway, you do not
> need protection against changing it.
ok, no prob, I'll repost the patch.
In meanwhile, could you please update docs/memory.txt and include
AddressSpace and MemoryListener things in it as it is not absolutely clear
what is what and for what purpose? The current version of the doc is too
old. Thanks.
--
Alexey
- Re: [Qemu-devel] [PATCH 3/4] vfio: Move container list to iommu MemoryRegion, (continued)
- Re: [Qemu-devel] [PATCH 3/4] vfio: Move container list to iommu MemoryRegion, Paolo Bonzini, 2013/04/29
- Re: [Qemu-devel] [PATCH 3/4] vfio: Move container list to iommu MemoryRegion, David Gibson, 2013/04/29
- Re: [Qemu-devel] [PATCH 3/4] vfio: Move container list to iommu MemoryRegion, Paolo Bonzini, 2013/04/29
- Re: [Qemu-devel] [PATCH 3/4] vfio: Move container list to iommu MemoryRegion, David Gibson, 2013/04/29
- Re: [Qemu-devel] [PATCH 3/4] vfio: Move container list to iommu MemoryRegion, David Gibson, 2013/04/29
- [Qemu-devel] [PATCH] memory: give name every AddressSpace, Alexey Kardashevskiy, 2013/04/28
- Re: [Qemu-devel] [PATCH] memory: give name every AddressSpace, Paolo Bonzini, 2013/04/29
- Re: [Qemu-devel] [PATCH] memory: give name every AddressSpace, Alexey Kardashevskiy, 2013/04/29
- Re: [Qemu-devel] [PATCH] memory: give name every AddressSpace, Paolo Bonzini, 2013/04/29
- Re: [Qemu-devel] [PATCH] memory: give name every AddressSpace, David Gibson, 2013/04/29
- Re: [Qemu-devel] [PATCH] memory: give name every AddressSpace,
Alexey Kardashevskiy <=
Re: [Qemu-devel] [0/4] RFC: Preparations for VFIO and guest IOMMUs (v2), Alex Williamson, 2013/04/26