Messages in this thread Patch in this message | | | From | xiakaixu1987@gmail ... | Subject | [PATCH v2] mm/damon/vaddr: add a comment for 'default' case in damon_va_apply_scheme() | Date | Thu, 8 Sep 2022 11:13:17 +0800 |
| |
From: Kaixu Xia <kaixuxia@tencent.com>
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@tencent.com> --- 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
| |