diff mbox series

[04/11] mm/damon/vaddr: Remove an unnecessary warning message

Message ID 20211201150440.1088-5-sj@kernel.org (mailing list archive)
State New
Headers show
Series mm/damon: Trivial fixups and improvements | expand

Commit Message

SeongJae Park Dec. 1, 2021, 3:04 p.m. UTC
The DAMON virtual address space monitoring primitive prints a warning
message for wrong DAMOS action.  However, it is not essential as the
code returns appropriate failure in the case.  This commit removes the
message to make the log clean.

Fixes: 6dea8add4d28 ("mm/damon/vaddr: support DAMON-based Operation Schemes")
Signed-off-by: SeongJae Park <sj@kernel.org>
---
 mm/damon/vaddr.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Muchun Song Dec. 3, 2021, 3:01 a.m. UTC | #1
On Wed, Dec 1, 2021 at 11:05 PM SeongJae Park <sj@kernel.org> wrote:
>
> The DAMON virtual address space monitoring primitive prints a warning
> message for wrong DAMOS action.  However, it is not essential as the
> code returns appropriate failure in the case.  This commit removes the
> message to make the log clean.
>
> Fixes: 6dea8add4d28 ("mm/damon/vaddr: support DAMON-based Operation Schemes")

I don't think there should be a Fixes tag since it's not a serious bug.

Without this:

Reviewed-by: Muchun Song <songmuchun@bytedance.com>

Thanks.
Andrew Morton Dec. 3, 2021, 8:44 p.m. UTC | #2
On Fri, 3 Dec 2021 11:01:36 +0800 Muchun Song <songmuchun@bytedance.com> wrote:

> On Wed, Dec 1, 2021 at 11:05 PM SeongJae Park <sj@kernel.org> wrote:
> >
> > The DAMON virtual address space monitoring primitive prints a warning
> > message for wrong DAMOS action.  However, it is not essential as the
> > code returns appropriate failure in the case.  This commit removes the
> > message to make the log clean.
> >
> > Fixes: 6dea8add4d28 ("mm/damon/vaddr: support DAMON-based Operation Schemes")
> 
> I don't think there should be a Fixes tag since it's not a serious bug.

"Fixes:" doesn't mean "backport to stable".  At least, not for MM
patches.  Other subsystems do get their Fixes:-tagged patches
automatically backported.
Muchun Song Dec. 4, 2021, 2:37 a.m. UTC | #3
On Sat, Dec 4, 2021 at 4:44 AM Andrew Morton <akpm@linux-foundation.org> wrote:
>
> On Fri, 3 Dec 2021 11:01:36 +0800 Muchun Song <songmuchun@bytedance.com> wrote:
>
> > On Wed, Dec 1, 2021 at 11:05 PM SeongJae Park <sj@kernel.org> wrote:
> > >
> > > The DAMON virtual address space monitoring primitive prints a warning
> > > message for wrong DAMOS action.  However, it is not essential as the
> > > code returns appropriate failure in the case.  This commit removes the
> > > message to make the log clean.
> > >
> > > Fixes: 6dea8add4d28 ("mm/damon/vaddr: support DAMON-based Operation Schemes")
> >
> > I don't think there should be a Fixes tag since it's not a serious bug.
>
> "Fixes:" doesn't mean "backport to stable".  At least, not for MM
> patches.  Other subsystems do get their Fixes:-tagged patches
> automatically backported.
>

Got it. Thanks.
diff mbox series

Patch

diff --git a/mm/damon/vaddr.c b/mm/damon/vaddr.c
index 79481f0c2838..a65b1a4d236c 100644
--- a/mm/damon/vaddr.c
+++ b/mm/damon/vaddr.c
@@ -617,7 +617,6 @@  static int damon_va_apply_scheme(struct damon_ctx *ctx, struct damon_target *t,
 	case DAMOS_STAT:
 		return 0;
 	default:
-		pr_warn("Wrong action %d\n", scheme->action);
 		return -EINVAL;
 	}