[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 6/8] memory: Sanity check that no listeners rema
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 6/8] memory: Sanity check that no listeners remain on a destroyed AddressSpace |
Date: |
Mon, 13 May 2013 12:10:10 +0100 |
On 13 May 2013 11:54, David Gibson <address@hidden> wrote:
> 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.
Hmm, is this the ideal semantics? Typically the owner of the
MemoryListener isn't the owner of the AddressSpace so it isn't
necessarily in a position to guarantee that it can unregister
the listener before the address space is destroyed. In fact
as the listener API is currently documented, the filter
argument is just an optimisation to save the callbacks having
to filter out irrelevant information themselves.
Perhaps an "address space being destroyed" callback for
listeners would be better? Then VFIO could just do the
necessary disposal operations automatically when the AS
goes away.
thanks
-- PMM
- [Qemu-devel] [PATCH 2/8] pci: Don't del_subgregion on a non subregion, (continued)
- [Qemu-devel] [PATCH 2/8] pci: Don't del_subgregion on a non subregion, David Gibson, 2013/05/13
- [Qemu-devel] [PATCH 4/8] pci: Use AddressSpace rather than MemoryRegion to represent PCI DMA space, David Gibson, 2013/05/13
- [Qemu-devel] [PATCH 1/8] iommu: Fix compile error in ioapic.c, David Gibson, 2013/05/13
- [Qemu-devel] [PATCH 7/8] vfio: Introduce VFIO address spaces, David Gibson, 2013/05/13
- [Qemu-devel] [PATCH 6/8] memory: Sanity check that no listeners remain on a destroyed AddressSpace, David Gibson, 2013/05/13
- Re: [Qemu-devel] [PATCH 6/8] memory: Sanity check that no listeners remain on a destroyed AddressSpace,
Peter Maydell <=
[Qemu-devel] [PATCH 5/8] pci: Introduce helper to retrieve a PCI device's DMA address space, David Gibson, 2013/05/13
[Qemu-devel] [PATCH 8/8] vfio: Create VFIOAddressSpace objects as needed, David Gibson, 2013/05/13
[Qemu-devel] [PATCH 3/8] pci: Rework PCI iommu lifetime assumptions, David Gibson, 2013/05/13