Re: [PATCH v4 2/2] dma: add IOMMU static calls with clear default ops

[Christoph Hellwig <hch@lst.de>]

On Thu, Aug 15, 2024 at 05:54:49PM +0100, Robin Murphy wrote:
>>   +#ifdef CONFIG_DMA_OPS_HELPERS
>> +#include <asm/dma-mapping.h>
>
> What needs a definition of get_arch_dma_ops() in this scope?
>
>> +struct page *dma_common_alloc_pages(struct device *dev, size_t size,
>> +		dma_addr_t *dma_handle, enum dma_data_direction dir, gfp_t gfp);
>> +void dma_common_free_pages(struct device *dev, size_t size, struct page *vaddr,
>> +		dma_addr_t dma_handle, enum dma_data_direction dir);
>
> Why not bring the rest of the dma_common_* declarations in here as well? 
> (Or more literally, add the #ifdef around where they all are already)

We actually don't need the ifdef at all, the calls in mapping.c are
all keyed off compile time constants, so just leaving the stray
prorotypes for this code that won't ever be called around should be
just fine.

>>   	depends on !NO_DMA
>>   	default y
>>   +# DMA IOMMU uses common ops helpers for certain operations, so let's 
>> allow to build
>> +# ops_helpers.c even if DMA_OPS is not enabled
>
> Hmm, but actually dma-direct also uses dma_common_contiguous_remap(), so 
> something seems a little inconsistent here...

Yes, but that's not really new.  I'll look into a patch to select
the helpers based on the conditions that make dma-direct use it.

I'll fix up all style issues and will apply the patch with that over
the weekend so that we can get it into this merge window.