[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v5 01/11] memory: Sanity check that no listeners r
From: |
Alexey Kardashevskiy |
Subject: |
Re: [Qemu-ppc] [PATCH v5 01/11] memory: Sanity check that no listeners remain on a destroyed AddressSpace |
Date: |
Thu, 27 Mar 2014 16:40:37 +1100 |
User-agent: |
Mozilla/5.0 (X11; Linux i686 on x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 |
On 03/20/2014 09:20 PM, Paolo Bonzini wrote:
> Il 12/03/2014 06:52, Alexey Kardashevskiy ha scritto:
>> From: David Gibson <address@hidden>
>>
>> At the moment, most AddressSpace objects last as long as the guest system
>> in practice, but that could well change in future. In addition, for VFIO
>> we will be introducing some private per-AdressSpace information, which must
>> be disposed of before the AddressSpace itself is destroyed.
>>
>> To reduce the chances of subtle bugs in this area, this patch adds
>> asssertions to ensure that when an AddressSpace is destroyed, there are no
>> remaining MemoryListeners using that AS as a filter.
>>
>> Signed-off-by: David Gibson <address@hidden>
>> Signed-off-by: Alexey Kardashevskiy <address@hidden>
>> ---
>> memory.c | 7 +++++++
>> 1 file changed, 7 insertions(+)
>>
>> diff --git a/memory.c b/memory.c
>> index 3f1df23..678661e 100644
>> --- a/memory.c
>> +++ b/memory.c
>> @@ -1722,12 +1722,19 @@ void address_space_init(AddressSpace *as,
>> MemoryRegion *root, const char *name)
>>
>> void address_space_destroy(AddressSpace *as)
>> {
>> + MemoryListener *listener;
>> +
>> /* Flush out anything from MemoryListeners listening in on this */
>> memory_region_transaction_begin();
>> as->root = NULL;
>> memory_region_transaction_commit();
>> QTAILQ_REMOVE(&address_spaces, as, address_spaces_link);
>> address_space_destroy_dispatch(as);
>> +
>> + QTAILQ_FOREACH(listener, &memory_listeners, link) {
>> + assert(listener->address_space_filter != as);
>> + }
>> +
>> flatview_unref(as->current_map);
>> g_free(as->name);
>> g_free(as->ioeventfds);
>>
>
> Reviewed-by: Paolo Bonzini <address@hidden>
What happens next to this patch and the next one ("int128: add
int128_exts64()")? I mean who you expect to pull them? Alex Graf? :) Thanks.
>
> An alternative is to add a count of listeners to the address space and
> assert that it is 0.
>
> Paolo
--
Alexey
- Re: [Qemu-ppc] [PATCH v5 07/11] vfio: Add guest side IOMMU support, (continued)
- Re: [Qemu-ppc] [PATCH v5 07/11] vfio: Add guest side IOMMU support, Alex Williamson, 2014/03/19
- Re: [Qemu-ppc] [PATCH v5 07/11] vfio: Add guest side IOMMU support, Alexey Kardashevskiy, 2014/03/21
- Re: [Qemu-ppc] [PATCH v5 07/11] vfio: Add guest side IOMMU support, Alex Williamson, 2014/03/21
- Re: [Qemu-ppc] [PATCH v5 07/11] vfio: Add guest side IOMMU support, Paolo Bonzini, 2014/03/21
- Re: [Qemu-ppc] [PATCH v5 07/11] vfio: Add guest side IOMMU support, Alexey Kardashevskiy, 2014/03/28
[Qemu-ppc] [PATCH v5 01/11] memory: Sanity check that no listeners remain on a destroyed AddressSpace, Alexey Kardashevskiy, 2014/03/12
[Qemu-ppc] [PATCH v5 06/11] vfio: Create VFIOAddressSpace objects as needed, Alexey Kardashevskiy, 2014/03/12
[Qemu-ppc] [PATCH v5 08/11] spapr-iommu: add SPAPR VFIO IOMMU device, Alexey Kardashevskiy, 2014/03/12
[Qemu-ppc] [PATCH v5 04/11] vfio: rework to have error paths, Alexey Kardashevskiy, 2014/03/12
[Qemu-ppc] [PATCH v5 09/11] spapr vfio: add vfio_container_spapr_get_info(), Alexey Kardashevskiy, 2014/03/12
[Qemu-ppc] [PATCH v5 05/11] vfio: Introduce VFIO address spaces, Alexey Kardashevskiy, 2014/03/12