Re: [PATCH v2] iommu/dma: Rationalise types for DMA masks

From: Joerg Roedel
Date: Tue Dec 17 2019 - 05:18:05 EST


On Wed, Dec 11, 2019 at 08:02:35PM +0100, Christoph Hellwig wrote:
> On Wed, Dec 11, 2019 at 06:33:26PM +0000, Robin Murphy wrote:
> > Since iommu_dma_alloc_iova() combines incoming masks with the u64 bus
> > limit, it makes more sense to pass them around in their native u64
> > rather than converting to dma_addr_t early. Do that, and resolve the
> > remaining type discrepancy against the domain geometry with a cheeky
> > cast to keep things simple.
> >
> > Signed-off-by: Robin Murphy <robin.murphy@xxxxxxx>
>
> Looks good to me:
>
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
>
> Joerg, let me know if you want to pick this up through the iommu tree as
> it touches the iommu code, or through the dma-mapping tree that
> introduced the warning.

I'll take it through my tree, as I am about to collect fixes anyway.

Patch is now applied, thanks everyone.