From: Kaixu Xia <kaixuxia@xxxxxxxxxxx> The switch case 'DAMOS_STAT' and switch case 'default' have same return value in damon_va_apply_scheme(), and the 'default' case is for DAMOS actions that not supported by 'vaddr'. It might make sense to add a comment here. Signed-off-by: Kaixu Xia <kaixuxia@xxxxxxxxxxx> --- mm/damon/vaddr.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/mm/damon/vaddr.c b/mm/damon/vaddr.c index 3c7b9d6dca95..3814200e61e4 100644 --- a/mm/damon/vaddr.c +++ b/mm/damon/vaddr.c @@ -646,6 +646,9 @@ static unsigned long damon_va_apply_scheme(struct damon_ctx *ctx, case DAMOS_STAT: return 0; default: + /* + * DAMOS actions that not yet supported by 'vaddr'. + */ return 0; } -- 2.27.0
