Commit 13f563ad authored by Hui Su's avatar Hui Su Committed by zhaoxiaoqiang11
Browse files

sched/fair: Remove superfluous lock section in do_sched_cfs_slack_timer() 

mainline inclusion
from mainline-v5.11-rc1
commit cdb31047
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/I6DDGM
CVE: NA

Reference: https://github.com/torvalds/linux/commit/cdb310474dece99985e4cdd2b96b1324e39c1c9d



--------------------------------

Since ab93a4bc ("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: default avatarHui Su <sh_def@163.com>
Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: default avatarPhil Auld <pauld@redhat.com>
Reviewed-by: default avatarBen Segall <bsegall@google.com>
Link: https://lkml.kernel.org/r/20201030144621.GA96974@rlk


Signed-off-by: default avatarzhaoxiaoqiang11 <zhaoxiaoqiang11@jd.com>
Reviewed-by: default avatarzhangwei <zhangwei123171@jd.com>
parent 598a1d66

kernel/sched/fair.c

+0 −3
Original line number Diff line number Diff line
@@ -5342,9 +5342,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);

}



/*