Re: [PATCH v4 0/3] Replace private domain with per-group default domain

From: Derrick, Jonathan
Date: Wed May 06 2020 - 13:50:58 EST


On Wed, 2020-05-06 at 10:09 +0800, Daniel Drake wrote:
> On Wed, May 6, 2020 at 10:03 AM Lu Baolu <baolu.lu@xxxxxxxxxxxxxxx> wrote:
> > https://lkml.org/lkml/2020/4/14/616
> > [This has been applied in iommu/next.]
> >
> > Hence, there is no need to keep the private domain implementation
> > in the Intel IOMMU driver. This patch series aims to remove it.
>
> I applied these patches on top of Joerg's branch and confirmed that
> they fix the issue discussed in the thread:
>
> [PATCH v2] iommu/vt-d: consider real PCI device when checking if
> mapping is needed
> (the patch there is no longer needed)
>
> Tested-by: Daniel Drake <drake@xxxxxxxxxxxx>
>
> Thanks!

Looks like the key to the real DMA dev fix was removing
identity_mapping() paths that led to dev->archdata.iommu == NULL -> DMA
domain

Works great for me as well

Reviewed-by: Jon Derrick <jonathan.derrick@xxxxxxxxx>