Re: [Qemu-devel] [PATCH v7 0/3] Introduce IOMMUNotifier struct

2016-09-23 Thread David Gibson
On Fri, Sep 23, 2016 at 01:02:25PM +0800, Peter Xu wrote: > V7: > - add comments to memory_region_notify_iommu() to better clarify the > interface [David] > - vfio_iommu_map_notify(): remove pointless "IOMMUTLBEntry *iotlb = > data" [David] > - typo fix on English [David] > > V6: > - use IOMMU

Re: [Qemu-devel] [PATCH v7 0/3] Introduce IOMMUNotifier struct

2016-09-23 Thread Peter Xu
On Fri, Sep 23, 2016 at 05:22:03PM +1000, David Gibson wrote: > I think we're ready to go with this. I still think 3/3 is premature, > but I don't care that much. > > Peter, thanks for your patience with my nitpicking of the interface. My pleasure to have discussion with you on this. And (always

[Qemu-devel] [PATCH v7 0/3] Introduce IOMMUNotifier struct

2016-09-22 Thread Peter Xu
V7: - add comments to memory_region_notify_iommu() to better clarify the interface [David] - vfio_iommu_map_notify(): remove pointless "IOMMUTLBEntry *iotlb = data" [David] - typo fix on English [David] V6: - use IOMMUTLBEntry directly for IOMMUNotifier.notify() [David] - add explicit comment