On 22/11/2018 08:44, Joerg Roedel wrote: > Hi Jean-Philippe, > > On Wed, Nov 21, 2018 at 11:16:07AM +0000, Jean-Philippe Brucker wrote: >> On 12/11/2018 14:40, Joerg Roedel wrote: >>> What is the intended use-case for this function? >> >> I'm using it in sva_bind(), to see if there already exists an io_mm >> associated to the mm_struct given as argument > > Can we store the pasid in the mm_struct directly? We still need a > reverse mapping for the page-fault path, but storing information in > mm_struct makes at least this path faster.
The bind() path doesn't really need to be fast. So on its own, I don't think it justifies changing the mm_struct until we have real-world numbers indicating that it's really too slow (as the worst case of iterating over 2**20 values will be) Thanks, Jean _______________________________________________ iommu mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/iommu
