Messages in this thread | | | Subject | Re: [PATCH 12/21] dma-iommu: factor atomic pool allocations into helpers | From | Robin Murphy <> | Date | Thu, 18 Apr 2019 16:06:56 +0100 |
| |
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.
| |