[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 00/12] Introduce new iommu notifier framework
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v3 00/12] Introduce new iommu notifier framework for virt-SVA |
Date: |
Wed, 7 Mar 2018 13:38:00 +0800 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Tue, Mar 06, 2018 at 07:45:39AM +0000, Liu, Yi L wrote:
[...]
> > Do you have online branch so that I can check out?
>
> yes, I should have pasted it. Here it is:
> https://github.com/luxis1999/sva_notifier.git
Thanks.
>
> > The patches are a bit scattered and it's really hard for me to
> > reference things within it... So a complete tree to read would be
> > nice.
> >
> > I roughly went over most of the patches, and the framework you
> > introduced is still not that clear to me. For now I feel like it can
> > be simplified somehow, but I'll hold and speak after I read the whole
> > tree again.
> >
> > Also, it'll be good too if you can always provide some status update
> > of the kernel-counterpart it.
>
> Good suggestion. For this patchset, it only affects Qemu. Yeah, but for
> the whole virt-SVA enabling, there is kernel-counterparts. I would do
> it in the virt-SVA patchset series.
If you still want to post separately - I'm thinking whether it'll be
good you put the vfio changes into the 2nd virt-sva series, since that
looks more like in that category. Or say, we can introduce
SVAOps/PASIDOps, we implement more vIOMMU invalidation request
handling, we call it in IOMMU code, but we don't implement any of the
device (vfio) that provide that ops.
Or maybe we can just post the whole stuff altogether, since after all
these two series are still closely related IMHO (e.g., the SVAOps
definition should be closely related to how the first vfio user would
like to use it).
Only my two cents, and I don't know how other people think. It's up
to you after all. :)
Thanks,
--
Peter Xu
- [Qemu-devel] [PATCH v3 10/12] intel_iommu: bind guest pasid table to host, (continued)
- [Qemu-devel] [PATCH v3 10/12] intel_iommu: bind guest pasid table to host, Liu, Yi L, 2018/03/01
- [Qemu-devel] [PATCH v3 11/12] intel_iommu: add framework for PASID AddressSpace management, Liu, Yi L, 2018/03/01
- Re: [Qemu-devel] [PATCH v3 11/12] intel_iommu: add framework for PASID AddressSpace management, Paolo Bonzini, 2018/03/02
- Re: [Qemu-devel] [PATCH v3 11/12] intel_iommu: add framework for PASID AddressSpace management, Liu, Yi L, 2018/03/05
- Re: [Qemu-devel] [PATCH v3 11/12] intel_iommu: add framework for PASID AddressSpace management, Paolo Bonzini, 2018/03/06
- Re: [Qemu-devel] [PATCH v3 11/12] intel_iommu: add framework for PASID AddressSpace management, Liu, Yi L, 2018/03/08
Re: [Qemu-devel] [PATCH v3 00/12] Introduce new iommu notifier framework for virt-SVA, Peter Xu, 2018/03/06