From 54f47e40b48fa3d19f45301aac6fd58f3e7067bc Mon Sep 17 00:00:00 2001 From: Andrew Morton Date: Fri, 29 Sep 2023 11:43:48 -0700 Subject: [PATCH] mm-kmem-scoped-objcg-protection-checkpatch-fixes WARNING: else is not generally useful after a break or return #73: FILE: mm/memcontrol.c:3135: + return objcg; + } else { total: 0 errors, 1 warnings, 111 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. ./patches/mm-kmem-scoped-objcg-protection.patch has style problems, please review. NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. Please run checkpatch prior to sending patches Cc: David Rientjes Cc: Dennis Zhou Cc: Johannes Weiner Cc: Michal Hocko Cc: Muchun Song Cc: Roman Gushchin (Cruise) Cc: Shakeel Butt Cc: Vlastimil Babka Signed-off-by: Andrew Morton --- mm/memcontrol.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/mm/memcontrol.c b/mm/memcontrol.c index 74c3a345ea659d..245d51cdd9f0f3 100644 --- a/mm/memcontrol.c +++ b/mm/memcontrol.c @@ -3132,11 +3132,11 @@ __always_inline struct obj_cgroup *current_obj_cgroup(void) * to use the objcg by the current task. */ return objcg; - } else { - memcg = this_cpu_read(int_active_memcg); - if (unlikely(memcg)) - goto from_memcg; } + + memcg = this_cpu_read(int_active_memcg); + if (unlikely(memcg)) + goto from_memcg; return NULL; from_memcg: