| From 55732d561d324bc366907b9b287764bfdaa5bf66 Mon Sep 17 00:00:00 2001 |
| From: Vladimir Murzin <vladimir.murzin@arm.com> |
| Date: Wed, 30 Oct 2019 10:13:13 +0000 |
| Subject: [PATCH] dma-mapping: fix handling of dma-ranges for reserved memory |
| (again) |
| |
| commit a445e940ea686fc60475564009821010eb213be3 upstream. |
| |
| Daniele reported that issue previously fixed in c41f9ea998f3 |
| ("drivers: dma-coherent: Account dma_pfn_offset when used with device |
| tree") reappear shortly after 43fc509c3efb ("dma-coherent: introduce |
| interface for default DMA pool") where fix was accidentally dropped. |
| |
| Lets put fix back in place and respect dma-ranges for reserved memory. |
| |
| Fixes: 43fc509c3efb ("dma-coherent: introduce interface for default DMA pool") |
| |
| Reported-by: Daniele Alessandrelli <daniele.alessandrelli@gmail.com> |
| Tested-by: Daniele Alessandrelli <daniele.alessandrelli@gmail.com> |
| Tested-by: Alexandre Torgue <alexandre.torgue@st.com> |
| Signed-off-by: Vladimir Murzin <vladimir.murzin@arm.com> |
| Signed-off-by: Christoph Hellwig <hch@lst.de> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/arch/arm/mm/dma-mapping-nommu.c b/arch/arm/mm/dma-mapping-nommu.c |
| index 1aea01ba1262..5dccc1d6a651 100644 |
| --- a/arch/arm/mm/dma-mapping-nommu.c |
| +++ b/arch/arm/mm/dma-mapping-nommu.c |
| @@ -46,7 +46,7 @@ static void *arm_nommu_dma_alloc(struct device *dev, size_t size, |
| return dma_direct_alloc_pages(dev, size, dma_handle, gfp, |
| attrs); |
| |
| - ret = dma_alloc_from_global_coherent(size, dma_handle); |
| + ret = dma_alloc_from_global_coherent(dev, size, dma_handle); |
| |
| /* |
| * dma_alloc_from_global_coherent() may fail because: |
| diff --git a/include/linux/dma-mapping.h b/include/linux/dma-mapping.h |
| index e42c4620ef32..9e62d11d8cca 100644 |
| --- a/include/linux/dma-mapping.h |
| +++ b/include/linux/dma-mapping.h |
| @@ -166,7 +166,7 @@ int dma_release_from_dev_coherent(struct device *dev, int order, void *vaddr); |
| int dma_mmap_from_dev_coherent(struct device *dev, struct vm_area_struct *vma, |
| void *cpu_addr, size_t size, int *ret); |
| |
| -void *dma_alloc_from_global_coherent(ssize_t size, dma_addr_t *dma_handle); |
| +void *dma_alloc_from_global_coherent(struct device *dev, ssize_t size, dma_addr_t *dma_handle); |
| int dma_release_from_global_coherent(int order, void *vaddr); |
| int dma_mmap_from_global_coherent(struct vm_area_struct *vma, void *cpu_addr, |
| size_t size, int *ret); |
| @@ -176,7 +176,7 @@ int dma_mmap_from_global_coherent(struct vm_area_struct *vma, void *cpu_addr, |
| #define dma_release_from_dev_coherent(dev, order, vaddr) (0) |
| #define dma_mmap_from_dev_coherent(dev, vma, vaddr, order, ret) (0) |
| |
| -static inline void *dma_alloc_from_global_coherent(ssize_t size, |
| +static inline void *dma_alloc_from_global_coherent(struct device *dev, ssize_t size, |
| dma_addr_t *dma_handle) |
| { |
| return NULL; |
| diff --git a/kernel/dma/coherent.c b/kernel/dma/coherent.c |
| index 29fd6590dc1e..99b21efe2649 100644 |
| --- a/kernel/dma/coherent.c |
| +++ b/kernel/dma/coherent.c |
| @@ -135,8 +135,9 @@ void dma_release_declared_memory(struct device *dev) |
| } |
| EXPORT_SYMBOL(dma_release_declared_memory); |
| |
| -static void *__dma_alloc_from_coherent(struct dma_coherent_mem *mem, |
| - ssize_t size, dma_addr_t *dma_handle) |
| +static void *__dma_alloc_from_coherent(struct device *dev, |
| + struct dma_coherent_mem *mem, |
| + ssize_t size, dma_addr_t *dma_handle) |
| { |
| int order = get_order(size); |
| unsigned long flags; |
| @@ -155,7 +156,7 @@ static void *__dma_alloc_from_coherent(struct dma_coherent_mem *mem, |
| /* |
| * Memory was found in the coherent area. |
| */ |
| - *dma_handle = mem->device_base + (pageno << PAGE_SHIFT); |
| + *dma_handle = dma_get_device_base(dev, mem) + (pageno << PAGE_SHIFT); |
| ret = mem->virt_base + (pageno << PAGE_SHIFT); |
| spin_unlock_irqrestore(&mem->spinlock, flags); |
| memset(ret, 0, size); |
| @@ -187,17 +188,18 @@ int dma_alloc_from_dev_coherent(struct device *dev, ssize_t size, |
| if (!mem) |
| return 0; |
| |
| - *ret = __dma_alloc_from_coherent(mem, size, dma_handle); |
| + *ret = __dma_alloc_from_coherent(dev, mem, size, dma_handle); |
| return 1; |
| } |
| |
| -void *dma_alloc_from_global_coherent(ssize_t size, dma_addr_t *dma_handle) |
| +void *dma_alloc_from_global_coherent(struct device *dev, ssize_t size, |
| + dma_addr_t *dma_handle) |
| { |
| if (!dma_coherent_default_memory) |
| return NULL; |
| |
| - return __dma_alloc_from_coherent(dma_coherent_default_memory, size, |
| - dma_handle); |
| + return __dma_alloc_from_coherent(dev, dma_coherent_default_memory, size, |
| + dma_handle); |
| } |
| |
| static int __dma_release_from_coherent(struct dma_coherent_mem *mem, |
| -- |
| 2.7.4 |
| |