Re: [Qemu-devel] [PATCH v2 1/3] memory: introduce IOMMUNotifier and its caps

2016-09-06 Thread Peter Xu
On Tue, Sep 06, 2016 at 04:29:52PM +0200, Paolo Bonzini wrote: > Since you're walking the notifier list manually anyway, I think it's > simpler to get rid of Notifier completely. Otherwise, this looks pretty > good! Yes that should be cleaner. Will fix. Thanks! -- peterx

Re: [Qemu-devel] [PATCH v2 1/3] memory: introduce IOMMUNotifier and its caps

2016-09-06 Thread Paolo Bonzini
On 06/09/2016 15:24, Peter Xu wrote: > IOMMU Notifier list is used for notifying IO address mapping changes. > Currently VFIO is the only user. > > However it is possible that future consumer like vhost would like to > only listen to part of its notifications (e.g., cache invalidations). > > Th

[Qemu-devel] [PATCH v2 1/3] memory: introduce IOMMUNotifier and its caps

2016-09-06 Thread Peter Xu
IOMMU Notifier list is used for notifying IO address mapping changes. Currently VFIO is the only user. However it is possible that future consumer like vhost would like to only listen to part of its notifications (e.g., cache invalidations). This patch introduced IOMMUNotifier and IOMMUNotfierCap