Re: [PATCH v6 03/25] powerpc/iommu: Setup a default domain and remove set_platform_dma_ops

Baolu Lu
2023-08-14T02:14:33Z

On 2023/8/3 8:07, Jason Gunthorpe wrote:
> POWER is using the set_platform_dma_ops() callback to hook up its private
> dma_ops, but this is buired under some indirection and is weirdly
> happening for a BLOCKED domain as well.
>
> For better documentation create a PLATFORM domain to manage the dma_ops,
> since that is what it is for, and make the BLOCKED domain an alias for
> it. BLOCKED is required for VFIO.
>
> Also removes the leaky allocation of the BLOCKED domain by using a global
> static.
>
> Signed-off-by: Jason Gunthorpe<jgg@nvidia.com>
> ---
> arch/powerpc/kernel/iommu.c | 38 +++++++++++++++++--------------------
> 1 file changed, 17 insertions(+), 21 deletions(-)

Not sure how to fix the fake blocking domain in this driver.

But it's not the purpose of this patch, so

Reviewed-by: Lu Baolu <baolu.lu@linux.intel.com>

Best regards,
baolu