Re: [PATCH v2 3/5] iommu/virtio: Move to domain_alloc_paging()

[Jason Gunthorpe <jgg@nvidia.com>]

On Mon, Mar 03, 2025 at 03:21:43PM +0000, Jean-Philippe Brucker wrote:
> On Thu, Feb 27, 2025 at 08:20:01PM -0400, Jason Gunthorpe wrote:
> > virtio has the complication that it sometimes wants to return a paging
> > domain for IDENTITY which makes this conversion a little different than
> > other drivers.
> > 
> > Add a viommu_domain_alloc_paging() that combines viommu_domain_alloc() and
> > viommu_domain_finalise() to always return a fully initialized and
> > finalized paging domain.
> > 
> > Use viommu_domain_alloc_identity() to implement the special non-bypass
> > IDENTITY flow by calling viommu_domain_alloc_paging() then
> > viommu_domain_map_identity().
> > 
> > Remove support for deferred finalize and the vdomain->mutex.
> > 
> > Remove core support for domain_alloc() IDENTITY as virtio was the last
> > driver using it.
> > 
> > Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
> 
> Reviewed-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> 
> And my tests still pass (after fixing the build issue on patch 2)

Yeah, my mistake, my tree has a different fix for that in a followup
series :\ I will resend it

Thanks,
Jason