summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKeren Sun <kerensun@google.com>2024-11-15 15:57:44 -0800
committerAndrew Morton <akpm@linux-foundation.org>2025-01-13 22:40:40 -0800
commit3472f639c692b28e7de345e90519430ad6ce2362 (patch)
tree4b50b659061d93126ee2033e7d68d6e36fc6b19a
parent91478b238e584d0ec1ea8c58ef46e4e14d58e526 (diff)
mm: remove the non-useful else after a break in a if statement
Remove the else block since there is already a break in the statement of if (iter->oom_lock), just set iter->oom_lock true after the if block ends. Link: https://lkml.kernel.org/r/20241115235744.1419580-4-kerensun@google.com Signed-off-by: Keren Sun <kerensun@google.com> Reviewed-by: Roman Gushchin <roman.gushchin@linux.dev> Cc: Johannes Weiner <hannes@cmpxchg.org> Cc: Michal Hocko <mhocko@kernel.org> Cc: Muchun Song <muchun.song@linux.dev> Cc: Shakeel Butt <shakeel.butt@linux.dev> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
-rw-r--r--mm/memcontrol-v1.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/mm/memcontrol-v1.c b/mm/memcontrol-v1.c
index c9fe524d341a..2be6b9112808 100644
--- a/mm/memcontrol-v1.c
+++ b/mm/memcontrol-v1.c
@@ -1134,8 +1134,8 @@ static bool mem_cgroup_oom_trylock(struct mem_cgroup *memcg)
failed = iter;
mem_cgroup_iter_break(memcg, iter);
break;
- } else
- iter->oom_lock = true;
+ }
+ iter->oom_lock = true;
}
if (failed) {