On Tue, Oct 17, 2023 at 10:07:11AM +0100, Joao Martins wrote: > > static struct iommu_domain *do_iommu_domain_alloc(unsigned int type, > - struct amd_iommu *iommu, > struct device *dev, > u32 flags) > { > struct protection_domain *domain; > + struct amd_iommu *iommu = NULL; > + > + if (dev) { > + iommu = rlookup_amd_iommu(dev); > + if (!iommu) This really shouldn't be rlookup_amd_iommu, didn't the series fixing this get merged? Jason