diff options
| author |   Wei Yang <richard.weiyang@gmail.com> | 2021-12-25 00:09:31 +0000 |
|---|---|---|
| committer |   Tejun Heo <tj@kernel.org> | 2022-01-06 11:10:06 -1000 |
| commit | 0da41f7348fff193d01d031ce255088fa98324b7 (patch) | |
| tree | 5cbbda35d65f186116a02a998413a553f0f0fb9f /kernel | |
| parent | cgroup: return early if it is already on preloaded list (diff) | |
| download | linux-dev-0da41f7348fff193d01d031ce255088fa98324b7.tar.xz linux-dev-0da41f7348fff193d01d031ce255088fa98324b7.zip | |
cgroup: rstat: explicitly put loop variant in while
Instead of do while unconditionally, let's put the loop variant in
while.
Signed-off-by: Wei Yang <richard.weiyang@gmail.com>
Reviewed-by: Michal Koutný <mkoutny@suse.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/cgroup/rstat.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/kernel/cgroup/rstat.c b/kernel/cgroup/rstat.c index 1abe74114527..bc6993258271 100644 --- a/kernel/cgroup/rstat.c +++ b/kernel/cgroup/rstat.c @@ -124,12 +124,10 @@ static struct cgroup *cgroup_rstat_cpu_pop_updated(struct cgroup *pos, prstatc = cgroup_rstat_cpu(parent, cpu); nextp = &prstatc->updated_children; - while (true) { + while (*nextp != pos) { struct cgroup_rstat_cpu *nrstatc; nrstatc = cgroup_rstat_cpu(*nextp, cpu); - if (*nextp == pos) - break; WARN_ON_ONCE(*nextp == parent); nextp = &nrstatc->updated_next; } |