sched/ext: Remove redundant else after return in scx_bpf_dsq_move_to_local

[PATCH] sched/ext: Remove redundant else after return in scx_bpf_dsq_move_to_local

fangqiurong@kylinos.cn posted 1 patch 2 months, 1 week ago

Download mbox

kernel/sched/ext.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

Expand all Fold all

[PATCH] sched/ext: Remove redundant else after return in scx_bpf_dsq_move_to_local

Posted by fangqiurong@kylinos.cn 2 months, 1 week ago

From: fangqiurong <fangqiurong@kylinos.cn>

The else branch is unnecessary when the if-block already returns.

Signed-off-by: fangqiurong <fangqiurong@kylinos.cn>
---
 kernel/sched/ext.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/sched/ext.c b/kernel/sched/ext.c
index 064eaa76be4b..e8384b5f1b0f 100644
--- a/kernel/sched/ext.c
+++ b/kernel/sched/ext.c
@@ -6280,9 +6280,9 @@ __bpf_kfunc bool scx_bpf_dsq_move_to_local(u64 dsq_id)
 		 */
 		dspc->nr_tasks++;
 		return true;
-	} else {
-		return false;
 	}
+
+	return false;
 }
 
 /**
-- 
2.25.1