Re: [PATCH 12/21] dma-iommu: factor atomic pool allocations into helpers

[Robin Murphy]

On 17/04/2019 07:33, Christoph Hellwig wrote:
> On Wed, Apr 10, 2019 at 08:11:57AM +0200, Christoph Hellwig wrote:
> > On Tue, Apr 09, 2019 at 06:59:32PM +0100, Robin Murphy wrote:
> > > On 27/03/2019 08:04, Christoph Hellwig wrote:
> > > > This keeps the code together and will simplify compiling the code
> > > > out on architectures that are always dma coherent.
> > >
> > > And this is where things take a turn in the direction I just can't get on
> > > with - I'm looking at the final result and the twisty maze of little
> > > disjoint helpers all overlapping each other in functionality is really
> > > difficult to follow. And I would *much* rather have things rely on
> > > compile-time constant optimisation than spend the future having to fix the
> > > #ifdefed parts for arm64 whenever x86-centric changes fail to test them.
> >
> > Can you draft up a patch on top of my series to show me what you
> > want?  I can take care of finishing it up and moving the changes
> > into the right patches in the series.
>
> Any chance to make some progress on this?  Or at least a better
> description of what you want?

OK, I'm still looking at mmap and get_sgtable, but for now I've pushed 
out a partial branch that consolidates alloc and free in a way which 
makes sense to me:

  git://linux-arm.org/linux-rm  dma/rework

Please let me know what you think.

Robin.