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

Path	csiph.com!1.us.feeder.erje.net!2.us.feeder.erje.net!feeder.erje.net!1.eu.feeder.erje.net!news2.arglkargh.de!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From	Geert Uytterhoeven <geert@linux-m68k.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH] iommu/dma: Add support for DMA_ATTR_FORCE_CONTIGUOUS
Date	Fri, 13 Jan 2017 13:40:02 +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>
X-Original-To	Robin Murphy <robin.murphy@arm.com>
Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc; bh=7stq2auRjjZjmPCnFEx/0QggydHKKMOwuil+h8XtifE=; b=XhoNoi5mgJbsrgRkfIDFdD/TlCVcGe1naakxUtdh/KvHBzoG4/kJkCMZpr73Afe39L YGPH/dOx09iIK/l80rAp/pr5e1wrnu7SbMzw2BCSa1L/5rB8S64G6XV7Qp0GYTf24ilf qG//qcZr087OW/dbqWygyLDvt/QLBd2138qSOmnVTkgpbbn/rdg+ZQMGQmATu0QMLRip Muta3qPKU4Q2TkQ0Inp9ymMylG+zKstWtR4at8QaIXFhQEv6uO47SuMq94gGaBV1Y08Q o4D31duBtfpnObpahWIQWqof9kdg/GM9XfZYmNAoi0hqT8gsEHB0vEXZ/EC9boVcUXvQ a65g==
X-Google-Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:sender:in-reply-to:references:from :date:message-id:subject:to:cc; bh=7stq2auRjjZjmPCnFEx/0QggydHKKMOwuil+h8XtifE=; b=AXy9wlmw9u9fp34HiY60YNKJ416d9OMfYmW+OqxgKioQIjdNjP0jX8XYaq0fVhVaR+ aT/8huywZgudO/dRGaPKpYIsW1zRrgR2xrfN87MCs0L7yUwbTovsv1CfwBw33FqrIrhL DrF3l1OUQgOtw3hspGUotY7eIXyCD50BouNXnpvfUQ0zQH6H/EJ03qCu8TrIRCGg03MW /24gIU4mmFJ+LGcUKSRakRE48BG7vzuNv9zYdxoJXlG8jNnv4qWxX+dRVn3ZbSsWt0Wd VTxhgVnh1fU4A9uYZFOuKX+/ZfLYSWA1wUsn4eKUlp62zOB6mdG8gXTqAFS6BPt9h3cW rNwQ==
X-Gm-Message-State	AIkVDXJWVBcPG2acMyZNtkgnQbSdfIBc5vKQ6ZxFqYC+vo0DrtCO6SPLjG0O4lcVl5ScbIdwVY9UVNA+OI9HNQ==
X-Received	by 10.36.216.67 with SMTP id b64mr2077702itg.104.1484310982573; Fri, 13 Jan 2017 04:36:22 -0800 (PST)
MIME-Version	1.0
X-Google-Sender-Auth	ivuuaByiPHxjUYkuvMwFNTyUqQ0
Content-Type	text/plain; charset=UTF-8
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	38
Organization	linux.* mail to news gateway
X-Original-Cc	Geert Uytterhoeven <geert+renesas@glider.be>, Joerg Roedel <joro@8bytes.org>, Catalin Marinas <catalin.marinas@arm.com>, Will Deacon <will.deacon@arm.com>, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, Marek Szyprowski <m.szyprowski@samsung.com>, Magnus Damm <magnus.damm@gmail.com>, iommu@lists.linux-foundation.org, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, Linux-Renesas <linux-renesas-soc@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
X-Original-Date	Fri, 13 Jan 2017 13:36:21 +0100
X-Original-Message-ID	<CAMuHMdVnteoor56EYG1Ka7t0o8+OKNehGnk6YUciTXdEQ5KKcQ@mail.gmail.com>
X-Original-References	<1484305674-15018-1-git-send-email-geert+renesas@glider.be> <d336ef3b-4fe6-5ebb-9fed-071fa79028df@arm.com> <CAMuHMdWBfTm_QRuB7x-ZDh0bgGmHokqm7=F2b0rjQTYzizC9hA@mail.gmail.com> <0e5c066f-ddf6-30ce-392e-79bd0f401a18@arm.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1558371

Show key headers only | 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