Re: [PATCH] sched/fair: remove the spin_lock operations

2020-11-02 Thread Hui Su
On Mon, Nov 02, 2020 at 08:53:41AM -0500, Phil Auld wrote: > On Fri, Oct 30, 2020 at 10:16:29PM + David Laight wrote: > > From: Benjamin Segall > > > Sent: 30 October 2020 18:48 > > > > > > Hui Su writes: > > > > > > > Since 'ab93a4bc955b ("sched/fair: Remove > > > > distribute_running

Re: [PATCH] sched/fair: remove the spin_lock operations

2020-11-02 Thread Phil Auld
On Fri, Oct 30, 2020 at 10:16:29PM + David Laight wrote: > From: Benjamin Segall > > Sent: 30 October 2020 18:48 > > > > Hui Su writes: > > > > > Since 'ab93a4bc955b ("sched/fair: Remove > > > distribute_running fromCFS bandwidth")',there is > > > nothing to protect between

RE: [PATCH] sched/fair: remove the spin_lock operations

2020-10-30 Thread David Laight
From: Benjamin Segall > Sent: 30 October 2020 18:48 > > Hui Su writes: > > > 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(). > > > > So remove it. > >

Re: [PATCH] sched/fair: remove the spin_lock operations

2020-10-30 Thread Benjamin Segall
Hui Su writes: > 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(). > > So remove it. Reviewed-by: Ben Segall (I might nitpick the subject to be clear that it

Re: [PATCH] sched/fair: remove the spin_lock operations

2020-10-30 Thread Phil Auld
On Fri, Oct 30, 2020 at 10:46:21PM +0800 Hui Su wrote: > 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(). > > So remove it. > > Signed-off-by: Hui Su > --- >

[PATCH] sched/fair: remove the spin_lock operations

2020-10-30 Thread Hui Su
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(). So remove it. Signed-off-by: Hui Su --- kernel/sched/fair.c | 3 --- 1 file changed, 3 deletions(-) diff --git