do_signal_stop() already returns in the if branch so there's no need to
keep the else branch around.

Signed-off-by: Christian Brauner <christ...@brauner.io>
---
v0->v1:
* patch unchanged
---
 kernel/signal.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/kernel/signal.c b/kernel/signal.c
index a628b56415e6..d1914439f144 100644
--- a/kernel/signal.c
+++ b/kernel/signal.c
@@ -2214,14 +2214,14 @@ static bool do_signal_stop(int signr)
                /* Now we don't run again until woken by SIGCONT or SIGKILL */
                freezable_schedule();
                return true;
-       } else {
-               /*
-                * While ptraced, group stop is handled by STOP trap.
-                * Schedule it and let the caller deal with it.
-                */
-               task_set_jobctl_pending(current, JOBCTL_TRAP_STOP);
-               return false;
        }
+
+       /*
+        * While ptraced, group stop is handled by STOP trap.
+        * Schedule it and let the caller deal with it.
+        */
+       task_set_jobctl_pending(current, JOBCTL_TRAP_STOP);
+       return false;
 }
 
 /**
-- 
2.17.0

Reply via email to