damon_next_region() and damon_prev_region() manually do container_of() type casting to fetch the next and previous region from the linked list. Use the dedicated list.h helpers, list_next_entry() and list_prev_entry(), as damon_first_region() and damon_last_region() do. Signed-off-by: Enze Li --- include/linux/damon.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/damon.h b/include/linux/damon.h index 380f9cf6ddd8..2ba43257e043 100644 --- a/include/linux/damon.h +++ b/include/linux/damon.h @@ -988,12 +988,12 @@ static inline unsigned long damon_rand(struct damon_ctx *ctx, static inline struct damon_region *damon_next_region(struct damon_region *r) { - return container_of(r->list.next, struct damon_region, list); + return list_next_entry(r, list); } static inline struct damon_region *damon_prev_region(struct damon_region *r) { - return container_of(r->list.prev, struct damon_region, list); + return list_prev_entry(r, list); } static inline struct damon_region *damon_last_region(struct damon_target *t) base-commit: 74d73ffcffe03804f97e9fb82301dadf75671282 -- 2.56.0