From 2a3c4bce3edb0d54983384aa8a88c0da330638f4 Mon Sep 17 00:00:00 2001 From: Miaohe Lin Date: Wed, 16 Mar 2022 16:15:28 +0800 Subject: mm/damon: minor cleanup for damon_pa_young if need_lock is true but folio_trylock fails, we should return false instead of NULL to match the return value type exactly. No functional change intended. Signed-off-by: Miaohe Lin Reviewed-by: SeongJae Park Signed-off-by: Matthew Wilcox (Oracle) --- mm/damon/paddr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'mm/damon/paddr.c') diff --git a/mm/damon/paddr.c b/mm/damon/paddr.c index ae24549921e2..74c2b6e1ca48 100644 --- a/mm/damon/paddr.c +++ b/mm/damon/paddr.c @@ -152,7 +152,7 @@ static bool damon_pa_young(unsigned long paddr, unsigned long *page_sz) need_lock = !folio_test_anon(folio) || folio_test_ksm(folio); if (need_lock && !folio_trylock(folio)) { folio_put(folio); - return NULL; + return false; } rmap_walk(folio, &rwc); -- cgit v1.2.3