* [PATCH] dma-coherent: fix rmem_dma_device_init regression
@ 2017-09-15 15:08 ` Arnd Bergmann
0 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2017-09-15 15:08 UTC (permalink / raw)
To: Christoph Hellwig, Marek Szyprowski, Greg Kroah-Hartman
Cc: Roy Pledge, Arnd Bergmann, Robin Murphy, Vladimir Murzin,
George G. Davis, Mark Craske, Bastian Hecht, iommu, linux-kernel
My recent bug fix introduced another bug, which caused rmem_dma_device_init
to always fail, as rmem->priv is never set to anything.
This restores the previous behavior, calling dma_init_coherent_memory()
whenever ->priv is NULL.
Fixes: d35b0996fef3 ("dma-coherent: fix dma_declare_coherent_memory() logic error")
Reported-by: Roy Pledge <roy.pledge@nxp.com>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
Roy, can you test this new fix?
---
drivers/base/dma-coherent.c | 19 +++++++++----------
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/drivers/base/dma-coherent.c b/drivers/base/dma-coherent.c
index a39b2166b145..744f64f43454 100644
--- a/drivers/base/dma-coherent.c
+++ b/drivers/base/dma-coherent.c
@@ -348,16 +348,15 @@ static int rmem_dma_device_init(struct reserved_mem *rmem, struct device *dev)
struct dma_coherent_mem *mem = rmem->priv;
int ret;
- if (!mem)
- return -ENODEV;
-
- ret = dma_init_coherent_memory(rmem->base, rmem->base, rmem->size,
- DMA_MEMORY_EXCLUSIVE, &mem);
-
- if (ret) {
- pr_err("Reserved memory: failed to init DMA memory pool at %pa, size %ld MiB\n",
- &rmem->base, (unsigned long)rmem->size / SZ_1M);
- return ret;
+ if (!mem) {
+ ret = dma_init_coherent_memory(rmem->base, rmem->base,
+ rmem->size,
+ DMA_MEMORY_EXCLUSIVE, &mem);
+ if (ret) {
+ pr_err("Reserved memory: failed to init DMA memory pool at %pa, size %ld MiB\n",
+ &rmem->base, (unsigned long)rmem->size / SZ_1M);
+ return ret;
+ }
}
mem->use_dev_dma_pfn_offset = true;
rmem->priv = mem;
--
2.9.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] dma-coherent: fix rmem_dma_device_init regression
@ 2017-09-15 15:08 ` Arnd Bergmann
0 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2017-09-15 15:08 UTC (permalink / raw)
To: Christoph Hellwig, Marek Szyprowski, Greg Kroah-Hartman
Cc: Vladimir Murzin, Arnd Bergmann, Roy Pledge,
linux-kernel-u79uwXL29TY76Z2rM5mHXA, Mark Craske, Bastian Hecht,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA,
George G. Davis
My recent bug fix introduced another bug, which caused rmem_dma_device_init
to always fail, as rmem->priv is never set to anything.
This restores the previous behavior, calling dma_init_coherent_memory()
whenever ->priv is NULL.
Fixes: d35b0996fef3 ("dma-coherent: fix dma_declare_coherent_memory() logic error")
Reported-by: Roy Pledge <roy.pledge-3arQi8VN3Tc@public.gmane.org>
Signed-off-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
---
Roy, can you test this new fix?
---
drivers/base/dma-coherent.c | 19 +++++++++----------
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/drivers/base/dma-coherent.c b/drivers/base/dma-coherent.c
index a39b2166b145..744f64f43454 100644
--- a/drivers/base/dma-coherent.c
+++ b/drivers/base/dma-coherent.c
@@ -348,16 +348,15 @@ static int rmem_dma_device_init(struct reserved_mem *rmem, struct device *dev)
struct dma_coherent_mem *mem = rmem->priv;
int ret;
- if (!mem)
- return -ENODEV;
-
- ret = dma_init_coherent_memory(rmem->base, rmem->base, rmem->size,
- DMA_MEMORY_EXCLUSIVE, &mem);
-
- if (ret) {
- pr_err("Reserved memory: failed to init DMA memory pool at %pa, size %ld MiB\n",
- &rmem->base, (unsigned long)rmem->size / SZ_1M);
- return ret;
+ if (!mem) {
+ ret = dma_init_coherent_memory(rmem->base, rmem->base,
+ rmem->size,
+ DMA_MEMORY_EXCLUSIVE, &mem);
+ if (ret) {
+ pr_err("Reserved memory: failed to init DMA memory pool at %pa, size %ld MiB\n",
+ &rmem->base, (unsigned long)rmem->size / SZ_1M);
+ return ret;
+ }
}
mem->use_dev_dma_pfn_offset = true;
rmem->priv = mem;
--
2.9.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] dma-coherent: fix rmem_dma_device_init regression
2017-09-15 15:08 ` Arnd Bergmann
@ 2017-09-17 12:35 ` Roy Pledge
-1 siblings, 0 replies; 6+ messages in thread
From: Roy Pledge @ 2017-09-17 12:35 UTC (permalink / raw)
To: Arnd Bergmann, Christoph Hellwig, Marek Szyprowski, Greg Kroah-Hartman
Cc: Robin Murphy, Vladimir Murzin, George G. Davis, Mark Craske,
Bastian Hecht, iommu, linux-kernel
On 9/15/2017 11:08 AM, Arnd Bergmann wrote:
> My recent bug fix introduced another bug, which caused rmem_dma_device_init
> to always fail, as rmem->priv is never set to anything.
>
> This restores the previous behavior, calling dma_init_coherent_memory()
> whenever ->priv is NULL.
>
> Fixes: d35b0996fef3 ("dma-coherent: fix dma_declare_coherent_memory() logic error")
> Reported-by: Roy Pledge <roy.pledge@nxp.com>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> Roy, can you test this new fix?
> ---
> drivers/base/dma-coherent.c | 19 +++++++++----------
> 1 file changed, 9 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/base/dma-coherent.c b/drivers/base/dma-coherent.c
> index a39b2166b145..744f64f43454 100644
> --- a/drivers/base/dma-coherent.c
> +++ b/drivers/base/dma-coherent.c
> @@ -348,16 +348,15 @@ static int rmem_dma_device_init(struct reserved_mem *rmem, struct device *dev)
> struct dma_coherent_mem *mem = rmem->priv;
> int ret;
>
> - if (!mem)
> - return -ENODEV;
> -
> - ret = dma_init_coherent_memory(rmem->base, rmem->base, rmem->size,
> - DMA_MEMORY_EXCLUSIVE, &mem);
> -
> - if (ret) {
> - pr_err("Reserved memory: failed to init DMA memory pool at %pa, size %ld MiB\n",
> - &rmem->base, (unsigned long)rmem->size / SZ_1M);
> - return ret;
> + if (!mem) {
> + ret = dma_init_coherent_memory(rmem->base, rmem->base,
> + rmem->size,
> + DMA_MEMORY_EXCLUSIVE, &mem);
> + if (ret) {
> + pr_err("Reserved memory: failed to init DMA memory pool at %pa, size %ld MiB\n",
> + &rmem->base, (unsigned long)rmem->size / SZ_1M);
> + return ret;
> + }
> }
> mem->use_dev_dma_pfn_offset = true;
> rmem->priv = mem;
>
I tried this and it solves the issue I was seeing. Thanks for the quick fix.
Tested-by: Roy Pledge <roy.pledge@nxp.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dma-coherent: fix rmem_dma_device_init regression
@ 2017-09-17 12:35 ` Roy Pledge
0 siblings, 0 replies; 6+ messages in thread
From: Roy Pledge @ 2017-09-17 12:35 UTC (permalink / raw)
To: Arnd Bergmann, Christoph Hellwig, Marek Szyprowski, Greg Kroah-Hartman
Cc: Vladimir Murzin, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
Mark Craske, Bastian Hecht,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA,
George G. Davis
On 9/15/2017 11:08 AM, Arnd Bergmann wrote:
> My recent bug fix introduced another bug, which caused rmem_dma_device_init
> to always fail, as rmem->priv is never set to anything.
>
> This restores the previous behavior, calling dma_init_coherent_memory()
> whenever ->priv is NULL.
>
> Fixes: d35b0996fef3 ("dma-coherent: fix dma_declare_coherent_memory() logic error")
> Reported-by: Roy Pledge <roy.pledge-3arQi8VN3Tc@public.gmane.org>
> Signed-off-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
> ---
> Roy, can you test this new fix?
> ---
> drivers/base/dma-coherent.c | 19 +++++++++----------
> 1 file changed, 9 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/base/dma-coherent.c b/drivers/base/dma-coherent.c
> index a39b2166b145..744f64f43454 100644
> --- a/drivers/base/dma-coherent.c
> +++ b/drivers/base/dma-coherent.c
> @@ -348,16 +348,15 @@ static int rmem_dma_device_init(struct reserved_mem *rmem, struct device *dev)
> struct dma_coherent_mem *mem = rmem->priv;
> int ret;
>
> - if (!mem)
> - return -ENODEV;
> -
> - ret = dma_init_coherent_memory(rmem->base, rmem->base, rmem->size,
> - DMA_MEMORY_EXCLUSIVE, &mem);
> -
> - if (ret) {
> - pr_err("Reserved memory: failed to init DMA memory pool at %pa, size %ld MiB\n",
> - &rmem->base, (unsigned long)rmem->size / SZ_1M);
> - return ret;
> + if (!mem) {
> + ret = dma_init_coherent_memory(rmem->base, rmem->base,
> + rmem->size,
> + DMA_MEMORY_EXCLUSIVE, &mem);
> + if (ret) {
> + pr_err("Reserved memory: failed to init DMA memory pool at %pa, size %ld MiB\n",
> + &rmem->base, (unsigned long)rmem->size / SZ_1M);
> + return ret;
> + }
> }
> mem->use_dev_dma_pfn_offset = true;
> rmem->priv = mem;
>
I tried this and it solves the issue I was seeing. Thanks for the quick fix.
Tested-by: Roy Pledge <roy.pledge-3arQi8VN3Tc@public.gmane.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dma-coherent: fix rmem_dma_device_init regression
@ 2017-09-17 15:20 ` Christoph Hellwig
0 siblings, 0 replies; 6+ messages in thread
From: Christoph Hellwig @ 2017-09-17 15:20 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Christoph Hellwig, Marek Szyprowski, Greg Kroah-Hartman,
Roy Pledge, Robin Murphy, Vladimir Murzin, George G. Davis,
Mark Craske, Bastian Hecht, iommu, linux-kernel
Thanks a lot Arnd,
applied to the dma-mapping tree for 4.14.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dma-coherent: fix rmem_dma_device_init regression
@ 2017-09-17 15:20 ` Christoph Hellwig
0 siblings, 0 replies; 6+ messages in thread
From: Christoph Hellwig @ 2017-09-17 15:20 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Vladimir Murzin, Greg Kroah-Hartman, Roy Pledge,
linux-kernel-u79uwXL29TY76Z2rM5mHXA, Mark Craske, Bastian Hecht,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA,
George G. Davis, Christoph Hellwig
Thanks a lot Arnd,
applied to the dma-mapping tree for 4.14.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-09-17 15:20 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-15 15:08 [PATCH] dma-coherent: fix rmem_dma_device_init regression Arnd Bergmann
2017-09-15 15:08 ` Arnd Bergmann
2017-09-17 12:35 ` Roy Pledge
2017-09-17 12:35 ` Roy Pledge
2017-09-17 15:20 ` Christoph Hellwig
2017-09-17 15:20 ` Christoph Hellwig
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.