Re: [PATCH 5/7] drm/nouveau/device: Get right pgsize_bitmap of iommu_domain

From: Christoph Hellwig
Date: Mon Jan 24 2022 - 04:30:09 EST


On Mon, Jan 24, 2022 at 03:11:00PM +0800, Lu Baolu wrote:
> The supported page sizes of an iommu_domain are saved in the pgsize_bitmap
> field. Retrieve the value from the right place.
>
> Signed-off-by: Lu Baolu <baolu.lu@xxxxxxxxxxxxxxx>
> Reviewed-by: Robin Murphy <robin.murphy@xxxxxxx>
> Link: https://lore.kernel.org/r/20211218074546.1772553-1-baolu.lu@xxxxxxxxxxxxxxx

Looks good:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

Wouldn't it mke sense to remove the pgsize_bitmap in struct iommu_ops
and initialize the domain field in the domain_alloc methods? Or am I
missing something?