[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC v2 00/17] VT-d: vfio enablement and misc enh
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH RFC v2 00/17] VT-d: vfio enablement and misc enhances |
Date: |
Fri, 13 Jan 2017 10:19:27 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Thu, Jan 12, 2017 at 04:27:30PM +0200, Michael S. Tsirkin wrote:
> On Tue, Jan 03, 2017 at 03:29:37PM +0800, Peter Xu wrote:
> > (I renamed the title for this RFC v2, since starting from this version
> > the series will be based on master, also I picked up some more fixes
> > for vt-d into this series)
> >
> > v2:
> > - change comment for "end" parameter in vtd_page_walk() [Tianyu]
> > - change comment for "a iova" to "an iova" [Yi]
> > - fix fault printed val for GPA address in vtd_page_walk_level (debug
> > only)
> > - rebased to master (rather than Aviv's v6 series) and merged Aviv's
> > series v6: picked patch 1 (as patch 1 in this series), dropped patch
> > 2, re-wrote patch 3 (as patch 17 of this series).
> > - picked up two more bugfix patches from Jason's DMAR series
> > - picked up the following patch as well:
> > "[PATCH v3] intel_iommu: allow dynamic switch of IOMMU region"
> >
> > This RFC series is a re-work for Aviv B.D.'s vfio enablement series
> > with vt-d:
> >
> > https://lists.gnu.org/archive/html/qemu-devel/2016-11/msg01452.html
> >
> > Aviv has done a great job there, and what we still lack there are
> > mostly the following:
> >
> > (1) VFIO got duplicated IOTLB notifications due to splitted VT-d IOMMU
> > memory region.
> >
> > (2) VT-d still haven't provide a correct replay() mechanism (e.g.,
> > when IOMMU domain switches, things will broke).
> >
> > This series should have solved the above two issues.
> >
> > Online repo:
> >
> > https://github.com/xzpeter/qemu/tree/vtd-vfio-enablement-v2
> >
> > I would be glad to hear about any review comments for above patches.
>
> This all looks good to me. The series needs to be
> rebased on top of latest bits.
> In particular, Jason made changes which conflict
> with this.
Michael,
Thanks for your positive feedback.
Could you provide me a branch so that I can rebase this work on? I was
trying to find a good base point in your repo here:
git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git
but failed to find a suitable branch. Thanks,
-- peterx
- [Qemu-devel] [PATCH RFC v2 10/17] memory: add section range info for IOMMU notifier, (continued)
- [Qemu-devel] [PATCH RFC v2 10/17] memory: add section range info for IOMMU notifier, Peter Xu, 2017/01/03
- [Qemu-devel] [PATCH RFC v2 11/17] memory: provide iommu_replay_all(), Peter Xu, 2017/01/03
- [Qemu-devel] [PATCH RFC v2 13/17] memory: add MemoryRegionIOMMUOps.replay() callback, Peter Xu, 2017/01/03
- [Qemu-devel] [PATCH RFC v2 12/17] memory: introduce memory_region_notify_one(), Peter Xu, 2017/01/03
- [Qemu-devel] [PATCH RFC v2 14/17] intel_iommu: provide its own replay() callback, Peter Xu, 2017/01/03
- [Qemu-devel] [PATCH RFC v2 15/17] intel_iommu: do replay when context invalidate, Peter Xu, 2017/01/03
- [Qemu-devel] [PATCH RFC v2 16/17] intel_iommu: allow dynamic switch of IOMMU region, Peter Xu, 2017/01/03
- [Qemu-devel] [PATCH RFC v2 17/17] intel_iommu: enable vfio devices, Peter Xu, 2017/01/03
- Re: [Qemu-devel] [PATCH RFC v2 00/17] VT-d: vfio enablement and misc enhances, no-reply, 2017/01/03
- Re: [Qemu-devel] [PATCH RFC v2 00/17] VT-d: vfio enablement and misc enhances, Michael S. Tsirkin, 2017/01/12
- Re: [Qemu-devel] [PATCH RFC v2 00/17] VT-d: vfio enablement and misc enhances,
Peter Xu <=