From: Ezequiel Garcia <ezequiel@collabora.com>
To: Christoph Hellwig <hch@lst.de>, iommu@lists.linux-foundation.org
Cc: Robin Murphy <robin.murphy@arm.com>,
Vineet Gupta <vgupta@synopsys.com>,
"Matwey V. Kornilov" <matwey@sai.msu.ru>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
linux-snps-arc@lists.infradead.org, linux-media@vger.kernel.org,
linux-arm-kernel@vger.kernel.org,
dri-devel@lists.freedesktop.org, sparclinux@vger.kernel.org,
openrisc@lists.librecores.org, linux-parisc@vger.kernel.org,
linux-mips@vger.kernel.org
Subject: Re: [PATCH 04/10] arm: implement DMA_ATTR_NON_CONSISTENT
Date: Sat, 08 Dec 2018 19:52:04 -0300 [thread overview]
Message-ID: <3aeb99c505fd1c33199fcc26fe5e0bf239dee57e.camel@collabora.com> (raw)
In-Reply-To: <20181208173702.15158-5-hch@lst.de>
On Sat, 2018-12-08 at 09:36 -0800, Christoph Hellwig wrote:
> For the iommu ops we can just use the implementaton for DMA coherent
> devices. For the regular ops we need mix and match a bit so that
> we either use the CMA allocator without remapping, but with a special
> error handling case for highmem pages, or the simple allocator.
>
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
> arch/arm/mm/dma-mapping.c | 49 ++++++++++++++++++++++++++++-----------
> 1 file changed, 35 insertions(+), 14 deletions(-)
>
> diff --git a/arch/arm/mm/dma-mapping.c b/arch/arm/mm/dma-mapping.c
> index 2cfb17bad1e6..b3b66b41c450 100644
> --- a/arch/arm/mm/dma-mapping.c
> +++ b/arch/arm/mm/dma-mapping.c
> @@ -49,6 +49,7 @@ struct arm_dma_alloc_args {
> const void *caller;
> bool want_vaddr;
> int coherent_flag;
> + bool nonconsistent_flag;
> };
>
> struct arm_dma_free_args {
> @@ -57,6 +58,7 @@ struct arm_dma_free_args {
> void *cpu_addr;
> struct page *page;
> bool want_vaddr;
> + bool nonconsistent_flag;
> };
>
> #define NORMAL 0
> @@ -348,7 +350,8 @@ static void __dma_free_buffer(struct page *page, size_t size)
> static void *__alloc_from_contiguous(struct device *dev, size_t size,
> pgprot_t prot, struct page **ret_page,
> const void *caller, bool want_vaddr,
> - int coherent_flag, gfp_t gfp);
> + int coherent_flag, bool nonconsistent_flag,
> + gfp_t gfp);
>
> static void *__alloc_remap_buffer(struct device *dev, size_t size, gfp_t gfp,
> pgprot_t prot, struct page **ret_page,
> @@ -405,7 +408,7 @@ static int __init atomic_pool_init(void)
> if (dev_get_cma_area(NULL))
> ptr = __alloc_from_contiguous(NULL, atomic_pool_size, prot,
> &page, atomic_pool_init, true, NORMAL,
> - GFP_KERNEL);
> + false, GFP_KERNEL);
> else
> ptr = __alloc_remap_buffer(NULL, atomic_pool_size, gfp, prot,
> &page, atomic_pool_init, true);
> @@ -579,7 +582,8 @@ static int __free_from_pool(void *start, size_t size)
> static void *__alloc_from_contiguous(struct device *dev, size_t size,
> pgprot_t prot, struct page **ret_page,
> const void *caller, bool want_vaddr,
> - int coherent_flag, gfp_t gfp)
> + int coherent_flag, bool nonconsistent_flag,
> + gfp_t gfp)
> {
> unsigned long order = get_order(size);
> size_t count = size >> PAGE_SHIFT;
> @@ -595,12 +599,16 @@ static void *__alloc_from_contiguous(struct device *dev, size_t size,
> if (!want_vaddr)
> goto out;
>
> + if (nonconsistent_flag) {
> + if (PageHighMem(page))
> + goto fail;
> + goto out;
> + }
> +
> if (PageHighMem(page)) {
> ptr = __dma_alloc_remap(page, size, GFP_KERNEL, prot, caller);
> - if (!ptr) {
> - dma_release_from_contiguous(dev, page, count);
> - return NULL;
> - }
> + if (!ptr)
> + goto fail;
> } else {
> __dma_remap(page, size, prot);
> ptr = page_address(page);
> @@ -609,12 +617,15 @@ static void *__alloc_from_contiguous(struct device *dev, size_t size,
> out:
> *ret_page = page;
> return ptr;
> + fail:
> + dma_release_from_contiguous(dev, page, count);
> + return NULL;
> }
>
> static void __free_from_contiguous(struct device *dev, struct page *page,
> - void *cpu_addr, size_t size, bool want_vaddr)
> + void *cpu_addr, size_t size, bool remapped)
> {
> - if (want_vaddr) {
> + if (remapped) {
> if (PageHighMem(page))
> __dma_free_remap(cpu_addr, size);
> else
> @@ -635,7 +646,11 @@ static void *__alloc_simple_buffer(struct device *dev, size_t size, gfp_t gfp,
> struct page **ret_page)
> {
> struct page *page;
> - /* __alloc_simple_buffer is only called when the device is coherent */
> + /*
> + * __alloc_simple_buffer is only called when the device is coherent,
> + * or if the caller explicitly asked for an allocation that is not
> + * consistent.
> + */
> page = __dma_alloc_buffer(dev, size, gfp, COHERENT);
> if (!page)
> return NULL;
> @@ -667,13 +682,15 @@ static void *cma_allocator_alloc(struct arm_dma_alloc_args *args,
> return __alloc_from_contiguous(args->dev, args->size, args->prot,
> ret_page, args->caller,
> args->want_vaddr, args->coherent_flag,
> + args->nonconsistent_flag,
> args->gfp);
> }
>
> static void cma_allocator_free(struct arm_dma_free_args *args)
> {
> __free_from_contiguous(args->dev, args->page, args->cpu_addr,
> - args->size, args->want_vaddr);
> + args->size,
> + args->want_vaddr || args->nonconsistent_flag);
> }
>
> static struct arm_dma_allocator cma_allocator = {
> @@ -735,6 +752,7 @@ static void *__dma_alloc(struct device *dev, size_t size, dma_addr_t *handle,
> .caller = caller,
> .want_vaddr = ((attrs & DMA_ATTR_NO_KERNEL_MAPPING) == 0),
> .coherent_flag = is_coherent ? COHERENT : NORMAL,
> + .nonconsistent_flag = (attrs & DMA_ATTR_NON_CONSISTENT),
> };
>
> #ifdef CONFIG_DMA_API_DEBUG
> @@ -773,7 +791,7 @@ static void *__dma_alloc(struct device *dev, size_t size, dma_addr_t *handle,
>
> if (cma)
> buf->allocator = &cma_allocator;
> - else if (is_coherent)
> + else if (is_coherent || (attrs & DMA_ATTR_NON_CONSISTENT))
> buf->allocator = &simple_allocator;
Reading through your code I can't really see where the pgprot is changed
for non-consistent requests. Namely, __get_dma_pgprot only
returns writecombine or coherent memory.
Regards,
Ezequiel
next prev parent reply other threads:[~2018-12-08 22:52 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-08 17:36 make the non-consistent DMA allocator more userful Christoph Hellwig
2018-12-08 17:36 ` [PATCH 01/10] dma-direct: provide a generic implementation of DMA_ATTR_NON_CONSISTENT Christoph Hellwig
2018-12-08 17:36 ` [PATCH 02/10] arm64/iommu: don't remap contiguous allocations for coherent devices Christoph Hellwig
2018-12-10 19:19 ` Robin Murphy
2018-12-10 19:25 ` Christoph Hellwig
2018-12-08 17:36 ` [PATCH 03/10] arm64/iommu: implement support for DMA_ATTR_NON_CONSISTENT Christoph Hellwig
2018-12-08 17:36 ` [PATCH 04/10] arm: implement DMA_ATTR_NON_CONSISTENT Christoph Hellwig
2018-12-08 22:52 ` Ezequiel Garcia [this message]
2018-12-10 19:16 ` Christoph Hellwig
2018-12-08 17:36 ` [PATCH 05/10] sparc64/iommu: move code around a bit Christoph Hellwig
2018-12-09 4:58 ` David Miller
2018-12-08 17:36 ` [PATCH 06/10] sparc64/iommu: implement DMA_ATTR_NON_CONSISTENT Christoph Hellwig
2018-12-09 4:58 ` David Miller
2018-12-08 17:36 ` [PATCH 07/10] sparc64/pci_sun4v: move code around a bit Christoph Hellwig
2018-12-09 4:58 ` David Miller
2018-12-08 17:37 ` [PATCH 08/10] sparc64/pci_sun4v: implement DMA_ATTR_NON_CONSISTENT Christoph Hellwig
2018-12-09 4:58 ` David Miller
2018-12-08 17:37 ` [PATCH 09/10] dma-mapping: skip declared coherent memory for DMA_ATTR_NON_CONSISTENT Christoph Hellwig
2018-12-08 17:37 ` [PATCH 10/10] Documentation: update the description " Christoph Hellwig
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=3aeb99c505fd1c33199fcc26fe5e0bf239dee57e.camel@collabora.com \
--to=ezequiel@collabora.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=hch@lst.de \
--cc=iommu@lists.linux-foundation.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-arm-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=linux-parisc@vger.kernel.org \
--cc=linux-snps-arc@lists.infradead.org \
--cc=matwey@sai.msu.ru \
--cc=openrisc@lists.librecores.org \
--cc=robin.murphy@arm.com \
--cc=sparclinux@vger.kernel.org \
--cc=vgupta@synopsys.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).