From a63b954559ee2cfd163363eb3f9ae7203e6242c0 Mon Sep 17 00:00:00 2001
From: Hui Su <sh_def@163.com>
Date: Fri, 30 Oct 2020 22:46:21 +0800
Subject: [PATCH] sched/fair: Remove superfluous lock section in
 do_sched_cfs_slack_timer()

ANBZ: #4091

commit cdb310474dece99985e4cdd2b96b1324e39c1c9d upstream

Since ab93a4bc955b ("sched/fair: Remove distribute_running fromCFS
bandwidth"), there is nothing to protect between
raw_spin_lock_irqsave/store() in do_sched_cfs_slack_timer().

Signed-off-by: Hui Su <sh_def@163.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Phil Auld <pauld@redhat.com>
Reviewed-by: Ben Segall <bsegall@google.com>
Link: https://lkml.kernel.org/r/20201030144621.GA96974@rlk
Signed-off-by: wangkaiqiang <wangkaiqiang@inspur.com>
---
 kernel/sched/fair.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 98a3a482ac5c..942d6e84453d 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -6996,9 +6996,6 @@ static void do_sched_cfs_slack_timer(struct cfs_bandwidth *cfs_b)
 		return;
 
 	distribute_cfs_runtime(cfs_b);
-
-	raw_spin_lock_irqsave(&cfs_b->lock, flags);
-	raw_spin_unlock_irqrestore(&cfs_b->lock, flags);
 }
 
 /*
-- 
Gitee