Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS

From	Geert Uytterhoeven <geert@linux-m68k.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS
Date	2017-01-13 13:40 +0100
Message-ID	<sZ9jc-Rn-19@gated-at.bofh.it> (permalink)
References	<sZ7U6-7f-33@gated-at.bofh.it> <sZ8wO-kY-35@gated-at.bofh.it> <sZ8Gu-ol-23@gated-at.bofh.it> <sZ8ZR-L2-47@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Robin,

On Fri, Jan 13, 2017 at 1:17 PM, Robin Murphy <robin.murphy@arm.com> wrote:
> On 13/01/17 11:59, Geert Uytterhoeven wrote:
>> On Fri, Jan 13, 2017 at 12:32 PM, Robin Murphy <robin.murphy@arm.com> wrote:
>>> On 13/01/17 11:07, Geert Uytterhoeven wrote:
>>>> Add support for DMA_ATTR_FORCE_CONTIGUOUS to the generic IOMMU DMA code.
>>>> This allows to allocate physically contiguous DMA buffers on arm64
>>>> systems with an IOMMU.
>>>
>>> Can anyone explain what this attribute is actually used for? I've never
>>> quite figured it out.
>>
>> My understanding is that DMA_ATTR_FORCE_CONTIGUOUS is needed when using
>> an IOMMU but wanting the buffers to be both contiguous in IOVA space and
>> physically contiguous to allow passing to devices without IOMMU.
>>
>> Main users are graphic and remote processors.
>
> Sure, I assumed it must be to do with buffer sharing, but the systems
> I'm aware of which have IOMMUs in their media subsystems tend to have
> them in front of every IP block involved, so I was curious as to what
> bit of non-IOMMU hardware wanted to play too. The lone in-tree use in
> the Exynos DRM driver was never very revealing, and the new one I see in
> the Qualcomm PIL driver frankly looks redundant to me.

I'll let the GPU-literate people comment on that...

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Thread

[PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS Geert Uytterhoeven <geert+renesas@glider.be> - 2017-01-13 12:10 +0100
  Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS Robin Murphy <robin.murphy@arm.com> - 2017-01-13 12:50 +0100
    Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS Geert Uytterhoeven <geert@linux-m68k.org> - 2017-01-13 13:00 +0100
      Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS Robin Murphy <robin.murphy@arm.com> - 2017-01-13 13:20 +0100
        Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS Geert Uytterhoeven <geert@linux-m68k.org> - 2017-01-13 13:40 +0100
        Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2017-01-15 22:20 +0100

csiph-web