mbox series

[RESEND,0/2] PCI DMA alias support

Message ID 157187055395.5439.6693401317571144578.stgit@gimli.home
Headers show
Series PCI DMA alias support | expand

Message

Alex Williamson Oct. 23, 2019, 10:47 p.m. UTC
Previous posting:
https://lists.gnu.org/archive/html/qemu-devel/2019-07/msg06182.html

Rebased (no change) and added Peter's R-b.  Please apply for QEMU 4.2.

Previous cover letter:

Please see patch 1/ for the motivation and utility of this series.
This v1 submission improves on the previous RFC with revised commit
logs, comments, and more testing, and the missing IVRS support for DMA
alias ranges is now included.  Testing has been done with Linux guests
with both SeaBIOS and OVMF with configurations of intel-iommu and
amd-iommu.  Intel-iommu testing includes device assignment, amd-iommu
is necessarily limited to emulated devices with interrupt remapping
disabled and iommu=pt in the guest (enabling interrupt remapping or
disabling guest passthrough mode fails to work regardless of this
series).  This series is NOT intended for QEMU v4.1.  Thanks,

Alex

---

Alex Williamson (2):
      pci: Use PCI aliases when determining device IOMMU address space
      hw/i386: AMD-Vi IVRS DMA alias support


 hw/i386/acpi-build.c |  127 +++++++++++++++++++++++++++++++++++++++++++++++---
 hw/pci/pci.c         |   43 ++++++++++++++++-
 2 files changed, 160 insertions(+), 10 deletions(-)

Comments

Alex Williamson Nov. 4, 2019, 10:03 p.m. UTC | #1
On Wed, 23 Oct 2019 16:47:02 -0600
Alex Williamson <alex.williamson@redhat.com> wrote:

> Previous posting:
> https://lists.gnu.org/archive/html/qemu-devel/2019-07/msg06182.html
> 
> Rebased (no change) and added Peter's R-b.  Please apply for QEMU 4.2.

Anyone?  I suppose this has missed another release despite pings from
me, Peter, and a resend :-(

Alex
 
> Previous cover letter:
> 
> Please see patch 1/ for the motivation and utility of this series.
> This v1 submission improves on the previous RFC with revised commit
> logs, comments, and more testing, and the missing IVRS support for DMA
> alias ranges is now included.  Testing has been done with Linux guests
> with both SeaBIOS and OVMF with configurations of intel-iommu and
> amd-iommu.  Intel-iommu testing includes device assignment, amd-iommu
> is necessarily limited to emulated devices with interrupt remapping
> disabled and iommu=pt in the guest (enabling interrupt remapping or
> disabling guest passthrough mode fails to work regardless of this
> series).  This series is NOT intended for QEMU v4.1.  Thanks,
> 
> Alex
> 
> ---
> 
> Alex Williamson (2):
>       pci: Use PCI aliases when determining device IOMMU address space
>       hw/i386: AMD-Vi IVRS DMA alias support
> 
> 
>  hw/i386/acpi-build.c |  127 +++++++++++++++++++++++++++++++++++++++++++++++---
>  hw/pci/pci.c         |   43 ++++++++++++++++-
>  2 files changed, 160 insertions(+), 10 deletions(-)
Michael S. Tsirkin Nov. 5, 2019, 9:03 a.m. UTC | #2
On Mon, Nov 04, 2019 at 03:03:53PM -0700, Alex Williamson wrote:
> On Wed, 23 Oct 2019 16:47:02 -0600
> Alex Williamson <alex.williamson@redhat.com> wrote:
> 
> > Previous posting:
> > https://lists.gnu.org/archive/html/qemu-devel/2019-07/msg06182.html
> > 
> > Rebased (no change) and added Peter's R-b.  Please apply for QEMU 4.2.
> 
> Anyone?  I suppose this has missed another release despite pings from
> me, Peter, and a resend :-(
> 
> Alex

Yes but I put it in the tree for the next release, will merge
right after the release.


> > Previous cover letter:
> > 
> > Please see patch 1/ for the motivation and utility of this series.
> > This v1 submission improves on the previous RFC with revised commit
> > logs, comments, and more testing, and the missing IVRS support for DMA
> > alias ranges is now included.  Testing has been done with Linux guests
> > with both SeaBIOS and OVMF with configurations of intel-iommu and
> > amd-iommu.  Intel-iommu testing includes device assignment, amd-iommu
> > is necessarily limited to emulated devices with interrupt remapping
> > disabled and iommu=pt in the guest (enabling interrupt remapping or
> > disabling guest passthrough mode fails to work regardless of this
> > series).  This series is NOT intended for QEMU v4.1.  Thanks,
> > 
> > Alex
> > 
> > ---
> > 
> > Alex Williamson (2):
> >       pci: Use PCI aliases when determining device IOMMU address space
> >       hw/i386: AMD-Vi IVRS DMA alias support
> > 
> > 
> >  hw/i386/acpi-build.c |  127 +++++++++++++++++++++++++++++++++++++++++++++++---
> >  hw/pci/pci.c         |   43 ++++++++++++++++-
> >  2 files changed, 160 insertions(+), 10 deletions(-)
Michael S. Tsirkin Nov. 6, 2019, 11:34 a.m. UTC | #3
On Mon, Nov 04, 2019 at 03:03:53PM -0700, Alex Williamson wrote:
> On Wed, 23 Oct 2019 16:47:02 -0600
> Alex Williamson <alex.williamson@redhat.com> wrote:
> 
> > Previous posting:
> > https://lists.gnu.org/archive/html/qemu-devel/2019-07/msg06182.html
> > 
> > Rebased (no change) and added Peter's R-b.  Please apply for QEMU 4.2.
> 
> Anyone?  I suppose this has missed another release despite pings from
> me, Peter, and a resend :-(
> 
> Alex

It can be considered a bugfix so I think ok after freeze.

> > Previous cover letter:
> > 
> > Please see patch 1/ for the motivation and utility of this series.
> > This v1 submission improves on the previous RFC with revised commit
> > logs, comments, and more testing, and the missing IVRS support for DMA
> > alias ranges is now included.  Testing has been done with Linux guests
> > with both SeaBIOS and OVMF with configurations of intel-iommu and
> > amd-iommu.  Intel-iommu testing includes device assignment, amd-iommu
> > is necessarily limited to emulated devices with interrupt remapping
> > disabled and iommu=pt in the guest (enabling interrupt remapping or
> > disabling guest passthrough mode fails to work regardless of this
> > series).  This series is NOT intended for QEMU v4.1.  Thanks,
> > 
> > Alex
> > 
> > ---
> > 
> > Alex Williamson (2):
> >       pci: Use PCI aliases when determining device IOMMU address space
> >       hw/i386: AMD-Vi IVRS DMA alias support
> > 
> > 
> >  hw/i386/acpi-build.c |  127 +++++++++++++++++++++++++++++++++++++++++++++++---
> >  hw/pci/pci.c         |   43 ++++++++++++++++-
> >  2 files changed, 160 insertions(+), 10 deletions(-)