From 780e0e6d3f3237f6a61957bfef0c8581993f6b1a Mon Sep 17 00:00:00 2001 From: Hui Su Date: Tue, 7 Feb 2023 16:32:45 +0800 Subject: [PATCH] sched/fair: Remove superfluous lock section in do_sched_cfs_slack_timer() mainline inclusion from mainline-v5.11-rc1 commit cdb310474dece99985e4cdd2b96b1324e39c1c9d category: bugfix bugzilla: https://gitee.com/openeuler/kernel/issues/I6DDGM CVE: NA Reference: https://github.com/torvalds/linux/commit/cdb310474dece99985e4cdd2b96b1324e39c1c9d -------------------------------- 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 Signed-off-by: Peter Zijlstra (Intel) Reviewed-by: Phil Auld Reviewed-by: Ben Segall Link: https://lkml.kernel.org/r/20201030144621.GA96974@rlk Signed-off-by: zhaoxiaoqiang11 Reviewed-by: zhangwei (cherry picked from commit 13f563ad128f4023e2899a6e3839ae62a8647c10) --- kernel/sched/fair.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 4485e985e35ae..e0d8f2e94c365 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -5512,9 +5512,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