All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yong Wu <yong.wu@mediatek.com>
To: Doug Anderson <dianders@chromium.org>
Cc: Joerg Roedel <joro@8bytes.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will.deacon@arm.com>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	Robin Murphy <robin.murphy@arm.com>,
	Daniel Kurtz <djkurtz@google.com>, Tomasz Figa <tfiga@google.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Lucas Stach <l.stach@pengutronix.de>,
	"moderated list:ARM/Mediatek SoC support"
	<linux-mediatek@lists.infradead.org>,
	srv_heupstream@mediatek.com,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	iommu@lists.linux-foundation.org,
	Marek Szyprowski <m.szyprowski@samsung.com>
Subject: Re: [PATCH] arm64/dma-mapping: Add DMA_ATTR_ALLOC_SINGLE_PAGES support
Date: Fri, 4 Mar 2016 08:00:49 +0800	[thread overview]
Message-ID: <1457049649.26149.11.camel@mhfsdcap03> (raw)
In-Reply-To: <CAD=FV=VOJT_Nd67O21J2-rabXRQ2=iTJxP8yCAty2xFhcVupGw@mail.gmail.com>

On Thu, 2016-03-03 at 09:44 -0800, Doug Anderson wrote:
> Hi,
> 
> On Wed, Mar 2, 2016 at 10:54 AM, Yong Wu <yong.wu@mediatek.com> wrote:
> > Sometimes it is not worth for the iommu allocating big chunks.
> > Here we enable DMA_ATTR_ALLOC_SINGLE_PAGES which could help avoid to
> > allocate big chunks while iommu allocating buffer.
> >
> > More information about this attribute, please check Doug's commit[1].
> >
> > [1]: https://lkml.org/lkml/2016/1/11/720
> >
> > Cc: Robin Murphy <robin.murphy@arm.com>
> > Suggested-by: Douglas Anderson <dianders@chromium.org>
> > Signed-off-by: Yong Wu <yong.wu@mediatek.com>
> > ---
> >
> > Our video drivers may soon use this.
> >
> >  arch/arm64/mm/dma-mapping.c |  4 ++--
> >  drivers/iommu/dma-iommu.c   | 14 ++++++++++----
> >  include/linux/dma-iommu.h   |  4 ++--
> >  3 files changed, 14 insertions(+), 8 deletions(-)
> 
> It should also be mentioned that this depends on commit df05c6f6e0bb
> ("ARM: 8506/1: common: DMA-mapping: add DMA_ATTR_ALLOC_SINGLE_PAGES
> attribute") which is not in mainline quite yet.

Thanks. I will add it in the commit message.

> 
> Also please CC Marek Szyprowski on any future patches in this area
> since I saw a patch series from him that was touching this same area.

Will do.

> 
> Reviewed-by: Douglas Anderson <dianders@chromium.org>

Thanks.

WARNING: multiple messages have this Message-ID (diff)
From: yong.wu@mediatek.com (Yong Wu)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm64/dma-mapping: Add DMA_ATTR_ALLOC_SINGLE_PAGES support
Date: Fri, 4 Mar 2016 08:00:49 +0800	[thread overview]
Message-ID: <1457049649.26149.11.camel@mhfsdcap03> (raw)
In-Reply-To: <CAD=FV=VOJT_Nd67O21J2-rabXRQ2=iTJxP8yCAty2xFhcVupGw@mail.gmail.com>

On Thu, 2016-03-03 at 09:44 -0800, Doug Anderson wrote:
> Hi,
> 
> On Wed, Mar 2, 2016 at 10:54 AM, Yong Wu <yong.wu@mediatek.com> wrote:
> > Sometimes it is not worth for the iommu allocating big chunks.
> > Here we enable DMA_ATTR_ALLOC_SINGLE_PAGES which could help avoid to
> > allocate big chunks while iommu allocating buffer.
> >
> > More information about this attribute, please check Doug's commit[1].
> >
> > [1]: https://lkml.org/lkml/2016/1/11/720
> >
> > Cc: Robin Murphy <robin.murphy@arm.com>
> > Suggested-by: Douglas Anderson <dianders@chromium.org>
> > Signed-off-by: Yong Wu <yong.wu@mediatek.com>
> > ---
> >
> > Our video drivers may soon use this.
> >
> >  arch/arm64/mm/dma-mapping.c |  4 ++--
> >  drivers/iommu/dma-iommu.c   | 14 ++++++++++----
> >  include/linux/dma-iommu.h   |  4 ++--
> >  3 files changed, 14 insertions(+), 8 deletions(-)
> 
> It should also be mentioned that this depends on commit df05c6f6e0bb
> ("ARM: 8506/1: common: DMA-mapping: add DMA_ATTR_ALLOC_SINGLE_PAGES
> attribute") which is not in mainline quite yet.

Thanks. I will add it in the commit message.

> 
> Also please CC Marek Szyprowski on any future patches in this area
> since I saw a patch series from him that was touching this same area.

Will do.

> 
> Reviewed-by: Douglas Anderson <dianders@chromium.org>

Thanks.

WARNING: multiple messages have this Message-ID (diff)
From: Yong Wu <yong.wu@mediatek.com>
To: Doug Anderson <dianders@chromium.org>
Cc: Joerg Roedel <joro@8bytes.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will.deacon@arm.com>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	Robin Murphy <robin.murphy@arm.com>,
	Daniel Kurtz <djkurtz@google.com>, Tomasz Figa <tfiga@google.com>,
	"Arnd Bergmann" <arnd@arndb.de>,
	Lucas Stach <l.stach@pengutronix.de>,
	"moderated list:ARM/Mediatek SoC support" 
	<linux-mediatek@lists.infradead.org>,
	<srv_heupstream@mediatek.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	<iommu@lists.linux-foundation.org>,
	Marek Szyprowski <m.szyprowski@samsung.com>
Subject: Re: [PATCH] arm64/dma-mapping: Add DMA_ATTR_ALLOC_SINGLE_PAGES support
Date: Fri, 4 Mar 2016 08:00:49 +0800	[thread overview]
Message-ID: <1457049649.26149.11.camel@mhfsdcap03> (raw)
In-Reply-To: <CAD=FV=VOJT_Nd67O21J2-rabXRQ2=iTJxP8yCAty2xFhcVupGw@mail.gmail.com>

On Thu, 2016-03-03 at 09:44 -0800, Doug Anderson wrote:
> Hi,
> 
> On Wed, Mar 2, 2016 at 10:54 AM, Yong Wu <yong.wu@mediatek.com> wrote:
> > Sometimes it is not worth for the iommu allocating big chunks.
> > Here we enable DMA_ATTR_ALLOC_SINGLE_PAGES which could help avoid to
> > allocate big chunks while iommu allocating buffer.
> >
> > More information about this attribute, please check Doug's commit[1].
> >
> > [1]: https://lkml.org/lkml/2016/1/11/720
> >
> > Cc: Robin Murphy <robin.murphy@arm.com>
> > Suggested-by: Douglas Anderson <dianders@chromium.org>
> > Signed-off-by: Yong Wu <yong.wu@mediatek.com>
> > ---
> >
> > Our video drivers may soon use this.
> >
> >  arch/arm64/mm/dma-mapping.c |  4 ++--
> >  drivers/iommu/dma-iommu.c   | 14 ++++++++++----
> >  include/linux/dma-iommu.h   |  4 ++--
> >  3 files changed, 14 insertions(+), 8 deletions(-)
> 
> It should also be mentioned that this depends on commit df05c6f6e0bb
> ("ARM: 8506/1: common: DMA-mapping: add DMA_ATTR_ALLOC_SINGLE_PAGES
> attribute") which is not in mainline quite yet.

Thanks. I will add it in the commit message.

> 
> Also please CC Marek Szyprowski on any future patches in this area
> since I saw a patch series from him that was touching this same area.

Will do.

> 
> Reviewed-by: Douglas Anderson <dianders@chromium.org>

Thanks.

  reply	other threads:[~2016-03-04  0:00 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-02 18:54 [PATCH] arm64/dma-mapping: Add DMA_ATTR_ALLOC_SINGLE_PAGES support Yong Wu
2016-03-02 18:54 ` Yong Wu
2016-03-02 18:54 ` Yong Wu
     [not found] ` <1456944866-15990-1-git-send-email-yong.wu-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2016-03-03 17:44   ` Doug Anderson
2016-03-03 17:44     ` Doug Anderson
2016-03-03 17:44     ` Doug Anderson
2016-03-04  0:00     ` Yong Wu [this message]
2016-03-04  0:00       ` Yong Wu
2016-03-04  0:00       ` Yong Wu
2016-03-21 18:01   ` Will Deacon
2016-03-21 18:01     ` Will Deacon
2016-03-21 18:01     ` Will Deacon
     [not found]     ` <20160321180147.GR23397-5wv7dgnIgG8@public.gmane.org>
2016-03-22 17:37       ` Doug Anderson
2016-03-22 17:37         ` Doug Anderson
2016-03-22 17:37         ` Doug Anderson
     [not found]         ` <CAD=FV=WL-OgE6D+8m3A8ZSJ8HgVZJw_+7uBce=qPh80LaxQKbg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-03-24 11:50           ` Will Deacon
2016-03-24 11:50             ` Will Deacon
     [not found]             ` <20160324115008.GE9323-5wv7dgnIgG8@public.gmane.org>
2016-03-25  4:25               ` Doug Anderson
2016-03-25  4:25                 ` Doug Anderson
2016-03-25  4:25                 ` Doug Anderson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1457049649.26149.11.camel@mhfsdcap03 \
    --to=yong.wu@mediatek.com \
    --cc=arnd@arndb.de \
    --cc=catalin.marinas@arm.com \
    --cc=dianders@chromium.org \
    --cc=djkurtz@google.com \
    --cc=iommu@lists.linux-foundation.org \
    --cc=joro@8bytes.org \
    --cc=l.stach@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=m.szyprowski@samsung.com \
    --cc=matthias.bgg@gmail.com \
    --cc=robin.murphy@arm.com \
    --cc=srv_heupstream@mediatek.com \
    --cc=tfiga@google.com \
    --cc=will.deacon@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.