When the check, (ssid) < CGROUP_SUBSYS_COUNT, passed, it means cgroup_subsys[ssid] is defined to its proper value. It is not necessary to use a true to enter the loop. Signed-off-by: Wei Yang <richard.weiyang@xxxxxxxxx> --- kernel/cgroup/cgroup-internal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/cgroup/cgroup-internal.h b/kernel/cgroup/cgroup-internal.h index bfbeabc17a9d..0c5d1df6cdef 100644 --- a/kernel/cgroup/cgroup-internal.h +++ b/kernel/cgroup/cgroup-internal.h @@ -163,7 +163,7 @@ extern struct file_system_type cgroup_fs_type; */ #define for_each_subsys(ss, ssid) \ for ((ssid) = 0; (ssid) < CGROUP_SUBSYS_COUNT && \ - (((ss) = cgroup_subsys[ssid]) || true); (ssid)++) + ((ss) = cgroup_subsys[ssid]); (ssid)++) static inline bool cgroup_is_dead(const struct cgroup *cgrp) { -- 2.33.1