Re: [Qemu-devel] [PATCH 0/3] memory: add IOMMU notifier type

[David Gibson]

On Mon, Sep 05, 2016 at 03:21:18PM +0800, Peter Xu wrote:
> In the thread:
> 
>   https://lists.gnu.org/archive/html/qemu-devel/2016-09/msg00254.html
> 
> Alex proposed a way for vhost DMAR to be enabled without breaking
> existing protections on vIOMMU and device assignments. This series
> tried to implement the idea, by introducing a IOMMU notifier type for
> each IOMMU memory region.

Hrm, I'm pretty dubious about this concept, since it's basically just
an interim hack for an incomplete notifier implementation on x86.
What makes just fixing the notifier so difficult?

> 
> Thanks,
> 
> Peter Xu (3):
>   memory: add one flag for IOMMU notifier
>   memory: add iommu_notify_flag
>   intel_iommu: allow IOMMU_NONE typed notifiers
> 
>  hw/i386/intel_iommu.c    | 13 ++++++++-----
>  hw/ppc/spapr_iommu.c     |  5 +++--
>  hw/s390x/s390-pci-inst.c |  2 +-
>  hw/vfio/common.c         |  3 ++-
>  include/exec/memory.h    | 14 +++++++++++---
>  memory.c                 | 12 +++++++++---
>  6 files changed, 34 insertions(+), 15 deletions(-)
> 

-- 
David Gibson                    | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au  | minimalist, thank you.  NOT _the_ _other_
                                | _way_ _around_!
http://www.ozlabs.org/~dgibson

signature.asc
Description: PGP signature