On 2022/4/1 6:26, Jason Gunthorpe wrote:
On Thu, Mar 31, 2022 at 01:59:22PM -0700, Jacob Pan wrote:

+       handle->dev = dev;
+       handle->domain = domain;
+       handle->pasid = mm->pasid;
why do we need to store pasid here? Conceptually, pasid is per sva domain
not per bind. You can get it from handle->domain->sva_cookie.
That is a mistake - SVA needs to follow the general PASID design - the
domain does not encode the PASID, the PASID comes from the device

You are right. We should not store any pasid information in the domain.

Best regards,
baolu
_______________________________________________
iommu mailing list
[email protected]
https://lists.linuxfoundation.org/mailman/listinfo/iommu

Reply via email to