On 2022/5/4 22:38, Jason Gunthorpe wrote:
@@ -3180,7 +3181,9 @@ int iommu_group_claim_dma_owner(struct iommu_group *group, void *owner) ret = -EPERM; goto unlock_out; } else { - if (group->domain && group->domain != group->default_domain) { + if (group->domain && + group->domain != group->default_domain && + group->domain != group->blocking_domain) {Why do we need this hunk? This is just trying to check if there is some conflict with some other domain attach, group->domain can never be blocking_domain here.
This is *not* needed. Also verified with real hardware.
Sorry for the noise. Best regards, baolu _______________________________________________ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
