The patch titled
Subject: mm/dmapool.c: add WARN_ON() in dma_pool_destroy
has been added to the -mm tree. Its filename is
mm-dmapoolc-add-warn_on-in-dma_pool_destroy.patch
This patch should soon appear at
http://ozlabs.org/~akpm/mmots/broken-out/mm-dmapoolc-add-warn_on-in-dma_pool_destroy.patch
and later at
http://ozlabs.org/~akpm/mmotm/broken-out/mm-dmapoolc-add-warn_on-in-dma_pool_destroy.patch
Before you just go and hit "reply", please:
a) Consider who else should be cc'ed
b) Prefer to cc a suitable mailing list as well
c) Ideally: find the original patch on the mailing list and do a
reply-to-all to that, adding suitable additional cc's
*** Remember to use Documentation/process/submit-checklist.rst when testing your code ***
The -mm tree is included into linux-next and is updated
there every 3-4 working days
------------------------------------------------------
From: Zhang Qiang <qiang.zhang@xxxxxxxxxxxxx>
Subject: mm/dmapool.c: add WARN_ON() in dma_pool_destroy
The pool is being destroyed so all pages which are in the pool should be
free. If some page is still in use by somebody, we should not just output
error logs, also should also add a WARN message so the stack backtrace may
be used to identify the caller.
Link: http://lkml.kernel.org/r/20200731023939.19206-1-qiang.zhang@xxxxxxxxxxxxx
Signed-off-by: Zhang Qiang <qiang.zhang@xxxxxxxxxxxxx>
Cc: Matthew Wilcox <willy@xxxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
mm/dmapool.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/mm/dmapool.c~mm-dmapoolc-add-warn_on-in-dma_pool_destroy
+++ a/mm/dmapool.c
@@ -285,7 +285,7 @@ void dma_pool_destroy(struct dma_pool *p
struct dma_page *page;
page = list_entry(pool->page_list.next,
struct dma_page, page_list);
- if (is_page_busy(page)) {
+ if (WARN_ON(is_page_busy(page))) {
if (pool->dev)
dev_err(pool->dev,
"dma_pool_destroy %s, %p busy\n",
_
Patches currently in -mm which might be from qiang.zhang@xxxxxxxxxxxxx are
mm-dmapoolc-add-warn_on-in-dma_pool_destroy.patch
