| From 2bd89bc5c36b86d07274856f6d308c7945f27991 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Wed, 1 Jun 2022 07:51:16 -0700 |
| Subject: dma-debug: make things less spammy under memory pressure |
| |
| From: Rob Clark <robdclark@chromium.org> |
| |
| [ Upstream commit e19f8fa6ce1ca9b8b934ba7d2e8f34c95abc6e60 ] |
| |
| Limit the error msg to avoid flooding the console. If you have a lot of |
| threads hitting this at once, they could have already gotten passed the |
| dma_debug_disabled() check before they get to the point of allocation |
| failure, resulting in quite a lot of this error message spamming the |
| log. Use pr_err_once() to limit that. |
| |
| Signed-off-by: Rob Clark <robdclark@chromium.org> |
| Signed-off-by: Christoph Hellwig <hch@lst.de> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| kernel/dma/debug.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| diff --git a/kernel/dma/debug.c b/kernel/dma/debug.c |
| index ee7da1f2462f..ae9fc1ee6d20 100644 |
| --- a/kernel/dma/debug.c |
| +++ b/kernel/dma/debug.c |
| @@ -564,7 +564,7 @@ static void add_dma_entry(struct dma_debug_entry *entry) |
| |
| rc = active_cacheline_insert(entry); |
| if (rc == -ENOMEM) { |
| - pr_err("cacheline tracking ENOMEM, dma-debug disabled\n"); |
| + pr_err_once("cacheline tracking ENOMEM, dma-debug disabled\n"); |
| global_disable = true; |
| } |
| |
| -- |
| 2.35.1 |
| |
