2019-05-19 20:08:55 +08:00
|
|
|
// SPDX-License-Identifier: GPL-2.0-only
|
2006-06-27 17:54:53 +08:00
|
|
|
/*
|
|
|
|
* RT-Mutexes: simple blocking mutual exclusion locks with PI support
|
|
|
|
*
|
|
|
|
* started by Ingo Molnar and Thomas Gleixner.
|
|
|
|
*
|
|
|
|
* Copyright (C) 2004-2006 Red Hat, Inc., Ingo Molnar <mingo@redhat.com>
|
|
|
|
* Copyright (C) 2005-2006 Timesys Corp., Thomas Gleixner <tglx@timesys.com>
|
|
|
|
* Copyright (C) 2005 Kihon Technologies Inc., Steven Rostedt
|
|
|
|
* Copyright (C) 2006 Esben Nielsen
|
2006-07-30 18:04:03 +08:00
|
|
|
*
|
2019-04-10 19:32:41 +08:00
|
|
|
* See Documentation/locking/rt-mutex-design.rst for details.
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
2021-08-16 05:27:57 +08:00
|
|
|
#include <linux/sched.h>
|
|
|
|
#include <linux/sched/debug.h>
|
|
|
|
#include <linux/sched/deadline.h>
|
2017-02-03 02:15:33 +08:00
|
|
|
#include <linux/sched/signal.h>
|
2013-02-07 23:47:07 +08:00
|
|
|
#include <linux/sched/rt.h>
|
2017-02-01 23:36:40 +08:00
|
|
|
#include <linux/sched/wake_q.h>
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
#include "rtmutex_common.h"
|
|
|
|
|
|
|
|
/*
|
|
|
|
* lock->owner state tracking:
|
|
|
|
*
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
* lock->owner holds the task_struct pointer of the owner. Bit 0
|
|
|
|
* is used to keep track of the "lock has waiters" state.
|
2006-06-27 17:54:53 +08:00
|
|
|
*
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
* owner bit0
|
|
|
|
* NULL 0 lock is free (fast acquire possible)
|
|
|
|
* NULL 1 lock is free and has waiters and the top waiter
|
|
|
|
* is going to take the lock*
|
|
|
|
* taskpointer 0 lock is held (fast release possible)
|
|
|
|
* taskpointer 1 lock is held and has waiters**
|
2006-06-27 17:54:53 +08:00
|
|
|
*
|
|
|
|
* The fast atomic compare exchange based acquire and release is only
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
* possible when bit 0 of lock->owner is 0.
|
|
|
|
*
|
|
|
|
* (*) It also can be a transitional state when grabbing the lock
|
|
|
|
* with ->wait_lock is held. To prevent any fast path cmpxchg to the lock,
|
|
|
|
* we need to set the bit0 before looking at the lock, and the owner may be
|
|
|
|
* NULL in this small time, hence this can be a transitional state.
|
2006-06-27 17:54:53 +08:00
|
|
|
*
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
* (**) There is a small time when bit 0 is set but there are no
|
|
|
|
* waiters. This can happen when grabbing the lock in the slow path.
|
|
|
|
* To prevent a cmpxchg of the owner releasing the lock, we need to
|
|
|
|
* set this bit before looking at the lock.
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline void
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
rt_mutex_set_owner(struct rt_mutex *lock, struct task_struct *owner)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
unsigned long val = (unsigned long)owner;
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
if (rt_mutex_has_waiters(lock))
|
|
|
|
val |= RT_MUTEX_HAS_WAITERS;
|
|
|
|
|
2020-01-04 07:59:12 +08:00
|
|
|
WRITE_ONCE(lock->owner, (struct task_struct *)val);
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline void clear_rt_mutex_waiters(struct rt_mutex *lock)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
|
|
|
lock->owner = (struct task_struct *)
|
|
|
|
((unsigned long)lock->owner & ~RT_MUTEX_HAS_WAITERS);
|
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline void fixup_rt_mutex_waiters(struct rt_mutex *lock)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
2016-12-01 05:04:41 +08:00
|
|
|
unsigned long owner, *p = (unsigned long *) &lock->owner;
|
|
|
|
|
|
|
|
if (rt_mutex_has_waiters(lock))
|
|
|
|
return;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* The rbtree has no waiters enqueued, now make sure that the
|
|
|
|
* lock->owner still has the waiters bit set, otherwise the
|
|
|
|
* following can happen:
|
|
|
|
*
|
|
|
|
* CPU 0 CPU 1 CPU2
|
|
|
|
* l->owner=T1
|
|
|
|
* rt_mutex_lock(l)
|
|
|
|
* lock(l->lock)
|
|
|
|
* l->owner = T1 | HAS_WAITERS;
|
|
|
|
* enqueue(T2)
|
|
|
|
* boost()
|
|
|
|
* unlock(l->lock)
|
|
|
|
* block()
|
|
|
|
*
|
|
|
|
* rt_mutex_lock(l)
|
|
|
|
* lock(l->lock)
|
|
|
|
* l->owner = T1 | HAS_WAITERS;
|
|
|
|
* enqueue(T3)
|
|
|
|
* boost()
|
|
|
|
* unlock(l->lock)
|
|
|
|
* block()
|
|
|
|
* signal(->T2) signal(->T3)
|
|
|
|
* lock(l->lock)
|
|
|
|
* dequeue(T2)
|
|
|
|
* deboost()
|
|
|
|
* unlock(l->lock)
|
|
|
|
* lock(l->lock)
|
|
|
|
* dequeue(T3)
|
|
|
|
* ==> wait list is empty
|
|
|
|
* deboost()
|
|
|
|
* unlock(l->lock)
|
|
|
|
* lock(l->lock)
|
|
|
|
* fixup_rt_mutex_waiters()
|
|
|
|
* if (wait_list_empty(l) {
|
|
|
|
* l->owner = owner
|
|
|
|
* owner = l->owner & ~HAS_WAITERS;
|
|
|
|
* ==> l->owner = T1
|
|
|
|
* }
|
|
|
|
* lock(l->lock)
|
|
|
|
* rt_mutex_unlock(l) fixup_rt_mutex_waiters()
|
|
|
|
* if (wait_list_empty(l) {
|
|
|
|
* owner = l->owner & ~HAS_WAITERS;
|
|
|
|
* cmpxchg(l->owner, T1, NULL)
|
|
|
|
* ===> Success (l->owner = NULL)
|
|
|
|
*
|
|
|
|
* l->owner = owner
|
|
|
|
* ==> l->owner = T1
|
|
|
|
* }
|
|
|
|
*
|
|
|
|
* With the check for the waiter bit in place T3 on CPU2 will not
|
|
|
|
* overwrite. All tasks fiddling with the waiters bit are
|
|
|
|
* serialized by l->lock, so nothing else can modify the waiters
|
|
|
|
* bit. If the bit is set then nothing can change l->owner either
|
|
|
|
* so the simple RMW is safe. The cmpxchg() will simply fail if it
|
|
|
|
* happens in the middle of the RMW because the waiters bit is
|
|
|
|
* still set.
|
|
|
|
*/
|
|
|
|
owner = READ_ONCE(*p);
|
|
|
|
if (owner & RT_MUTEX_HAS_WAITERS)
|
|
|
|
WRITE_ONCE(*p, owner & ~RT_MUTEX_HAS_WAITERS);
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
|
|
|
|
2007-06-18 03:11:10 +08:00
|
|
|
/*
|
2015-02-26 01:56:13 +08:00
|
|
|
* We can speed up the acquire/release, if there's no debugging state to be
|
|
|
|
* set up.
|
2007-06-18 03:11:10 +08:00
|
|
|
*/
|
2015-02-26 01:56:13 +08:00
|
|
|
#ifndef CONFIG_DEBUG_RT_MUTEXES
|
2021-08-16 05:27:54 +08:00
|
|
|
static __always_inline bool rt_mutex_cmpxchg_acquire(struct rt_mutex *lock,
|
|
|
|
struct task_struct *old,
|
|
|
|
struct task_struct *new)
|
|
|
|
{
|
2021-08-16 05:27:55 +08:00
|
|
|
return try_cmpxchg_acquire(&lock->owner, &old, new);
|
2021-08-16 05:27:54 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static __always_inline bool rt_mutex_cmpxchg_release(struct rt_mutex *lock,
|
|
|
|
struct task_struct *old,
|
|
|
|
struct task_struct *new)
|
|
|
|
{
|
2021-08-16 05:27:55 +08:00
|
|
|
return try_cmpxchg_release(&lock->owner, &old, new);
|
2021-08-16 05:27:54 +08:00
|
|
|
}
|
2015-10-01 04:03:13 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Callers must hold the ->wait_lock -- which is the whole purpose as we force
|
|
|
|
* all future threads that attempt to [Rmw] the lock to the slowpath. As such
|
|
|
|
* relaxed semantics suffice.
|
|
|
|
*/
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline void mark_rt_mutex_waiters(struct rt_mutex *lock)
|
2007-06-18 03:11:10 +08:00
|
|
|
{
|
|
|
|
unsigned long owner, *p = (unsigned long *) &lock->owner;
|
|
|
|
|
|
|
|
do {
|
|
|
|
owner = *p;
|
2015-10-01 04:03:13 +08:00
|
|
|
} while (cmpxchg_relaxed(p, owner,
|
|
|
|
owner | RT_MUTEX_HAS_WAITERS) != owner);
|
2007-06-18 03:11:10 +08:00
|
|
|
}
|
2014-06-12 02:44:04 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Safe fastpath aware unlock:
|
|
|
|
* 1) Clear the waiters bit
|
|
|
|
* 2) Drop lock->wait_lock
|
|
|
|
* 3) Try to unlock the lock with cmpxchg
|
|
|
|
*/
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline bool unlock_rt_mutex_safe(struct rt_mutex *lock,
|
|
|
|
unsigned long flags)
|
2014-06-12 02:44:04 +08:00
|
|
|
__releases(lock->wait_lock)
|
|
|
|
{
|
|
|
|
struct task_struct *owner = rt_mutex_owner(lock);
|
|
|
|
|
|
|
|
clear_rt_mutex_waiters(lock);
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irqrestore(&lock->wait_lock, flags);
|
2014-06-12 02:44:04 +08:00
|
|
|
/*
|
|
|
|
* If a new waiter comes in between the unlock and the cmpxchg
|
|
|
|
* we have two situations:
|
|
|
|
*
|
|
|
|
* unlock(wait_lock);
|
|
|
|
* lock(wait_lock);
|
|
|
|
* cmpxchg(p, owner, 0) == owner
|
|
|
|
* mark_rt_mutex_waiters(lock);
|
|
|
|
* acquire(lock);
|
|
|
|
* or:
|
|
|
|
*
|
|
|
|
* unlock(wait_lock);
|
|
|
|
* lock(wait_lock);
|
|
|
|
* mark_rt_mutex_waiters(lock);
|
|
|
|
*
|
|
|
|
* cmpxchg(p, owner, 0) != owner
|
|
|
|
* enqueue_waiter();
|
|
|
|
* unlock(wait_lock);
|
|
|
|
* lock(wait_lock);
|
|
|
|
* wake waiter();
|
|
|
|
* unlock(wait_lock);
|
|
|
|
* lock(wait_lock);
|
|
|
|
* acquire(lock);
|
|
|
|
*/
|
2015-10-01 04:03:13 +08:00
|
|
|
return rt_mutex_cmpxchg_release(lock, owner, NULL);
|
2014-06-12 02:44:04 +08:00
|
|
|
}
|
|
|
|
|
2007-06-18 03:11:10 +08:00
|
|
|
#else
|
2021-08-16 05:27:54 +08:00
|
|
|
static __always_inline bool rt_mutex_cmpxchg_acquire(struct rt_mutex *lock,
|
|
|
|
struct task_struct *old,
|
|
|
|
struct task_struct *new)
|
|
|
|
{
|
|
|
|
return false;
|
|
|
|
|
|
|
|
}
|
|
|
|
|
|
|
|
static __always_inline bool rt_mutex_cmpxchg_release(struct rt_mutex *lock,
|
|
|
|
struct task_struct *old,
|
|
|
|
struct task_struct *new)
|
|
|
|
{
|
|
|
|
return false;
|
|
|
|
}
|
2015-10-01 04:03:13 +08:00
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline void mark_rt_mutex_waiters(struct rt_mutex *lock)
|
2007-06-18 03:11:10 +08:00
|
|
|
{
|
|
|
|
lock->owner = (struct task_struct *)
|
|
|
|
((unsigned long)lock->owner | RT_MUTEX_HAS_WAITERS);
|
|
|
|
}
|
2014-06-12 02:44:04 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Simple slow path only version: lock->owner is protected by lock->wait_lock.
|
|
|
|
*/
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline bool unlock_rt_mutex_safe(struct rt_mutex *lock,
|
|
|
|
unsigned long flags)
|
2014-06-12 02:44:04 +08:00
|
|
|
__releases(lock->wait_lock)
|
|
|
|
{
|
|
|
|
lock->owner = NULL;
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irqrestore(&lock->wait_lock, flags);
|
2014-06-12 02:44:04 +08:00
|
|
|
return true;
|
|
|
|
}
|
2007-06-18 03:11:10 +08:00
|
|
|
#endif
|
|
|
|
|
2017-03-23 22:56:14 +08:00
|
|
|
/*
|
|
|
|
* Only use with rt_mutex_waiter_{less,equal}()
|
|
|
|
*/
|
|
|
|
#define task_to_waiter(p) \
|
|
|
|
&(struct rt_mutex_waiter){ .prio = (p)->prio, .deadline = (p)->dl.deadline }
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline int rt_mutex_waiter_less(struct rt_mutex_waiter *left,
|
|
|
|
struct rt_mutex_waiter *right)
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
{
|
sched/deadline: Add SCHED_DEADLINE inheritance logic
Some method to deal with rt-mutexes and make sched_dl interact with
the current PI-coded is needed, raising all but trivial issues, that
needs (according to us) to be solved with some restructuring of
the pi-code (i.e., going toward a proxy execution-ish implementation).
This is under development, in the meanwhile, as a temporary solution,
what this commits does is:
- ensure a pi-lock owner with waiters is never throttled down. Instead,
when it runs out of runtime, it immediately gets replenished and it's
deadline is postponed;
- the scheduling parameters (relative deadline and default runtime)
used for that replenishments --during the whole period it holds the
pi-lock-- are the ones of the waiting task with earliest deadline.
Acting this way, we provide some kind of boosting to the lock-owner,
still by using the existing (actually, slightly modified by the previous
commit) pi-architecture.
We would stress the fact that this is only a surely needed, all but
clean solution to the problem. In the end it's only a way to re-start
discussion within the community. So, as always, comments, ideas, rants,
etc.. are welcome! :-)
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
[ Added !RT_MUTEXES build fix. ]
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-11-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:44 +08:00
|
|
|
if (left->prio < right->prio)
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
return 1;
|
|
|
|
|
|
|
|
/*
|
sched/deadline: Add SCHED_DEADLINE inheritance logic
Some method to deal with rt-mutexes and make sched_dl interact with
the current PI-coded is needed, raising all but trivial issues, that
needs (according to us) to be solved with some restructuring of
the pi-code (i.e., going toward a proxy execution-ish implementation).
This is under development, in the meanwhile, as a temporary solution,
what this commits does is:
- ensure a pi-lock owner with waiters is never throttled down. Instead,
when it runs out of runtime, it immediately gets replenished and it's
deadline is postponed;
- the scheduling parameters (relative deadline and default runtime)
used for that replenishments --during the whole period it holds the
pi-lock-- are the ones of the waiting task with earliest deadline.
Acting this way, we provide some kind of boosting to the lock-owner,
still by using the existing (actually, slightly modified by the previous
commit) pi-architecture.
We would stress the fact that this is only a surely needed, all but
clean solution to the problem. In the end it's only a way to re-start
discussion within the community. So, as always, comments, ideas, rants,
etc.. are welcome! :-)
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
[ Added !RT_MUTEXES build fix. ]
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-11-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:44 +08:00
|
|
|
* If both waiters have dl_prio(), we check the deadlines of the
|
|
|
|
* associated tasks.
|
|
|
|
* If left waiter has a dl_prio(), and we didn't return 1 above,
|
|
|
|
* then right waiter has a dl_prio() too.
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
*/
|
sched/deadline: Add SCHED_DEADLINE inheritance logic
Some method to deal with rt-mutexes and make sched_dl interact with
the current PI-coded is needed, raising all but trivial issues, that
needs (according to us) to be solved with some restructuring of
the pi-code (i.e., going toward a proxy execution-ish implementation).
This is under development, in the meanwhile, as a temporary solution,
what this commits does is:
- ensure a pi-lock owner with waiters is never throttled down. Instead,
when it runs out of runtime, it immediately gets replenished and it's
deadline is postponed;
- the scheduling parameters (relative deadline and default runtime)
used for that replenishments --during the whole period it holds the
pi-lock-- are the ones of the waiting task with earliest deadline.
Acting this way, we provide some kind of boosting to the lock-owner,
still by using the existing (actually, slightly modified by the previous
commit) pi-architecture.
We would stress the fact that this is only a surely needed, all but
clean solution to the problem. In the end it's only a way to re-start
discussion within the community. So, as always, comments, ideas, rants,
etc.. are welcome! :-)
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
[ Added !RT_MUTEXES build fix. ]
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-11-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:44 +08:00
|
|
|
if (dl_prio(left->prio))
|
2017-03-23 22:56:13 +08:00
|
|
|
return dl_time_before(left->deadline, right->deadline);
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline int rt_mutex_waiter_equal(struct rt_mutex_waiter *left,
|
|
|
|
struct rt_mutex_waiter *right)
|
2017-03-23 22:56:14 +08:00
|
|
|
{
|
|
|
|
if (left->prio != right->prio)
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* If both waiters have dl_prio(), we check the deadlines of the
|
|
|
|
* associated tasks.
|
|
|
|
* If left waiter has a dl_prio(), and we didn't return 0 above,
|
|
|
|
* then right waiter has a dl_prio() too.
|
|
|
|
*/
|
|
|
|
if (dl_prio(left->prio))
|
|
|
|
return left->deadline == right->deadline;
|
|
|
|
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
|
2020-04-29 23:29:58 +08:00
|
|
|
#define __node_2_waiter(node) \
|
|
|
|
rb_entry((node), struct rt_mutex_waiter, tree_entry)
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline bool __waiter_less(struct rb_node *a, const struct rb_node *b)
|
2020-04-29 23:29:58 +08:00
|
|
|
{
|
|
|
|
return rt_mutex_waiter_less(__node_2_waiter(a), __node_2_waiter(b));
|
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline void
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_enqueue(struct rt_mutex *lock, struct rt_mutex_waiter *waiter)
|
|
|
|
{
|
2020-04-29 23:29:58 +08:00
|
|
|
rb_add_cached(&waiter->tree_entry, &lock->waiters, __waiter_less);
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline void
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_dequeue(struct rt_mutex *lock, struct rt_mutex_waiter *waiter)
|
|
|
|
{
|
|
|
|
if (RB_EMPTY_NODE(&waiter->tree_entry))
|
|
|
|
return;
|
|
|
|
|
2017-09-09 07:15:01 +08:00
|
|
|
rb_erase_cached(&waiter->tree_entry, &lock->waiters);
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
RB_CLEAR_NODE(&waiter->tree_entry);
|
|
|
|
}
|
|
|
|
|
2020-04-29 23:29:58 +08:00
|
|
|
#define __node_2_pi_waiter(node) \
|
|
|
|
rb_entry((node), struct rt_mutex_waiter, pi_tree_entry)
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline bool
|
|
|
|
__pi_waiter_less(struct rb_node *a, const struct rb_node *b)
|
2020-04-29 23:29:58 +08:00
|
|
|
{
|
|
|
|
return rt_mutex_waiter_less(__node_2_pi_waiter(a), __node_2_pi_waiter(b));
|
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline void
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_enqueue_pi(struct task_struct *task, struct rt_mutex_waiter *waiter)
|
|
|
|
{
|
2020-04-29 23:29:58 +08:00
|
|
|
rb_add_cached(&waiter->pi_tree_entry, &task->pi_waiters, __pi_waiter_less);
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline void
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_dequeue_pi(struct task_struct *task, struct rt_mutex_waiter *waiter)
|
|
|
|
{
|
|
|
|
if (RB_EMPTY_NODE(&waiter->pi_tree_entry))
|
|
|
|
return;
|
|
|
|
|
2017-09-09 07:15:01 +08:00
|
|
|
rb_erase_cached(&waiter->pi_tree_entry, &task->pi_waiters);
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
RB_CLEAR_NODE(&waiter->pi_tree_entry);
|
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline void rt_mutex_adjust_prio(struct task_struct *p)
|
2014-02-08 03:58:42 +08:00
|
|
|
{
|
2017-03-23 22:56:11 +08:00
|
|
|
struct task_struct *pi_task = NULL;
|
2017-03-23 22:56:08 +08:00
|
|
|
|
2017-03-23 22:56:11 +08:00
|
|
|
lockdep_assert_held(&p->pi_lock);
|
2014-02-08 03:58:42 +08:00
|
|
|
|
2017-03-23 22:56:11 +08:00
|
|
|
if (task_has_pi_waiters(p))
|
|
|
|
pi_task = task_top_pi_waiter(p)->task;
|
2014-02-08 03:58:42 +08:00
|
|
|
|
2017-03-23 22:56:11 +08:00
|
|
|
rt_mutex_setprio(p, pi_task);
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
|
|
|
|
2014-05-22 11:25:47 +08:00
|
|
|
/*
|
|
|
|
* Deadlock detection is conditional:
|
|
|
|
*
|
|
|
|
* If CONFIG_DEBUG_RT_MUTEXES=n, deadlock detection is only conducted
|
|
|
|
* if the detect argument is == RT_MUTEX_FULL_CHAINWALK.
|
|
|
|
*
|
|
|
|
* If CONFIG_DEBUG_RT_MUTEXES=y, deadlock detection is always
|
|
|
|
* conducted independent of the detect argument.
|
|
|
|
*
|
|
|
|
* If the waiter argument is NULL this indicates the deboost path and
|
|
|
|
* deadlock detection is disabled independent of the detect argument
|
|
|
|
* and the config settings.
|
|
|
|
*/
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline bool
|
|
|
|
rt_mutex_cond_detect_deadlock(struct rt_mutex_waiter *waiter,
|
|
|
|
enum rtmutex_chainwalk chwalk)
|
2014-05-22 11:25:47 +08:00
|
|
|
{
|
2021-07-31 20:30:11 +08:00
|
|
|
if (IS_ENABLED(CONFIG_DEBUG_RT_MUTEXES))
|
2021-03-26 23:29:36 +08:00
|
|
|
return waiter != NULL;
|
|
|
|
return chwalk == RT_MUTEX_FULL_CHAINWALK;
|
2014-05-22 11:25:47 +08:00
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static __always_inline struct rt_mutex *task_blocked_on_lock(struct task_struct *p)
|
2014-06-05 17:16:12 +08:00
|
|
|
{
|
|
|
|
return p->pi_blocked_on ? p->pi_blocked_on->lock : NULL;
|
|
|
|
}
|
|
|
|
|
2006-06-27 17:54:53 +08:00
|
|
|
/*
|
|
|
|
* Adjust the priority chain. Also used for deadlock detection.
|
|
|
|
* Decreases task's usage by one - may thus free the task.
|
2013-05-15 17:04:10 +08:00
|
|
|
*
|
2014-06-05 17:16:12 +08:00
|
|
|
* @task: the task owning the mutex (owner) for which a chain walk is
|
|
|
|
* probably needed
|
2015-03-18 13:03:30 +08:00
|
|
|
* @chwalk: do we have to carry out deadlock detection?
|
2014-06-05 17:16:12 +08:00
|
|
|
* @orig_lock: the mutex (can be NULL if we are walking the chain to recheck
|
|
|
|
* things for a task that has just got its priority adjusted, and
|
|
|
|
* is waiting on a mutex)
|
|
|
|
* @next_lock: the mutex on which the owner of @orig_lock was blocked before
|
|
|
|
* we dropped its pi_lock. Is never dereferenced, only used for
|
|
|
|
* comparison to detect lock chain changes.
|
2013-05-15 17:04:10 +08:00
|
|
|
* @orig_waiter: rt_mutex_waiter struct for the task that has just donated
|
2014-06-05 17:16:12 +08:00
|
|
|
* its priority to the mutex owner (can be NULL in the case
|
|
|
|
* depicted above or if the top waiter is gone away and we are
|
|
|
|
* actually deboosting the owner)
|
|
|
|
* @top_task: the current top waiter
|
2013-05-15 17:04:10 +08:00
|
|
|
*
|
2006-06-27 17:54:53 +08:00
|
|
|
* Returns 0 or -EDEADLK.
|
2014-06-10 01:40:34 +08:00
|
|
|
*
|
|
|
|
* Chain walk basics and protection scope
|
|
|
|
*
|
|
|
|
* [R] refcount on task
|
|
|
|
* [P] task->pi_lock held
|
|
|
|
* [L] rtmutex->wait_lock held
|
|
|
|
*
|
|
|
|
* Step Description Protected by
|
|
|
|
* function arguments:
|
|
|
|
* @task [R]
|
|
|
|
* @orig_lock if != NULL @top_task is blocked on it
|
|
|
|
* @next_lock Unprotected. Cannot be
|
|
|
|
* dereferenced. Only used for
|
|
|
|
* comparison.
|
|
|
|
* @orig_waiter if != NULL @top_task is blocked on it
|
|
|
|
* @top_task current, or in case of proxy
|
|
|
|
* locking protected by calling
|
|
|
|
* code
|
|
|
|
* again:
|
|
|
|
* loop_sanity_check();
|
|
|
|
* retry:
|
|
|
|
* [1] lock(task->pi_lock); [R] acquire [P]
|
|
|
|
* [2] waiter = task->pi_blocked_on; [P]
|
|
|
|
* [3] check_exit_conditions_1(); [P]
|
|
|
|
* [4] lock = waiter->lock; [P]
|
|
|
|
* [5] if (!try_lock(lock->wait_lock)) { [P] try to acquire [L]
|
|
|
|
* unlock(task->pi_lock); release [P]
|
|
|
|
* goto retry;
|
|
|
|
* }
|
|
|
|
* [6] check_exit_conditions_2(); [P] + [L]
|
|
|
|
* [7] requeue_lock_waiter(lock, waiter); [P] + [L]
|
|
|
|
* [8] unlock(task->pi_lock); release [P]
|
|
|
|
* put_task_struct(task); release [R]
|
|
|
|
* [9] check_exit_conditions_3(); [L]
|
|
|
|
* [10] task = owner(lock); [L]
|
|
|
|
* get_task_struct(task); [L] acquire [R]
|
|
|
|
* lock(task->pi_lock); [L] acquire [P]
|
|
|
|
* [11] requeue_pi_waiter(tsk, waiters(lock));[P] + [L]
|
|
|
|
* [12] check_exit_conditions_4(); [P] + [L]
|
|
|
|
* [13] unlock(task->pi_lock); release [P]
|
|
|
|
* unlock(lock->wait_lock); release [L]
|
|
|
|
* goto again;
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
2021-03-26 23:29:40 +08:00
|
|
|
static int __sched rt_mutex_adjust_prio_chain(struct task_struct *task,
|
|
|
|
enum rtmutex_chainwalk chwalk,
|
|
|
|
struct rt_mutex *orig_lock,
|
|
|
|
struct rt_mutex *next_lock,
|
|
|
|
struct rt_mutex_waiter *orig_waiter,
|
|
|
|
struct task_struct *top_task)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
|
|
|
struct rt_mutex_waiter *waiter, *top_waiter = orig_waiter;
|
2014-05-22 11:25:54 +08:00
|
|
|
struct rt_mutex_waiter *prerequeue_top_waiter;
|
2014-05-22 11:25:47 +08:00
|
|
|
int ret = 0, depth = 0;
|
2014-05-22 11:25:54 +08:00
|
|
|
struct rt_mutex *lock;
|
2014-05-22 11:25:47 +08:00
|
|
|
bool detect_deadlock;
|
2014-05-22 11:25:57 +08:00
|
|
|
bool requeue = true;
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-05-22 11:25:47 +08:00
|
|
|
detect_deadlock = rt_mutex_cond_detect_deadlock(orig_waiter, chwalk);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* The (de)boosting is a step by step approach with a lot of
|
|
|
|
* pitfalls. We want this to be preemptible and we want hold a
|
|
|
|
* maximum of two locks per step. So we have to check
|
|
|
|
* carefully whether things change under us.
|
|
|
|
*/
|
|
|
|
again:
|
2014-06-10 01:40:34 +08:00
|
|
|
/*
|
|
|
|
* We limit the lock chain length for each invocation.
|
|
|
|
*/
|
2006-06-27 17:54:53 +08:00
|
|
|
if (++depth > max_lock_depth) {
|
|
|
|
static int prev_max;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Print this only once. If the admin changes the limit,
|
|
|
|
* print a new message when reaching the limit again.
|
|
|
|
*/
|
|
|
|
if (prev_max != max_lock_depth) {
|
|
|
|
prev_max = max_lock_depth;
|
|
|
|
printk(KERN_WARNING "Maximum lock depth %d reached "
|
|
|
|
"task: %s (%d)\n", max_lock_depth,
|
2007-10-19 14:40:40 +08:00
|
|
|
top_task->comm, task_pid_nr(top_task));
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
|
|
|
put_task_struct(task);
|
|
|
|
|
2014-06-05 18:34:23 +08:00
|
|
|
return -EDEADLK;
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
2014-06-10 01:40:34 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* We are fully preemptible here and only hold the refcount on
|
|
|
|
* @task. So everything can have changed under us since the
|
|
|
|
* caller or our own code below (goto retry/again) dropped all
|
|
|
|
* locks.
|
|
|
|
*/
|
2006-06-27 17:54:53 +08:00
|
|
|
retry:
|
|
|
|
/*
|
2014-06-10 01:40:34 +08:00
|
|
|
* [1] Task cannot go away as we did a get_task() before !
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock_irq(&task->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-10 01:40:34 +08:00
|
|
|
/*
|
|
|
|
* [2] Get the waiter on which @task is blocked on.
|
|
|
|
*/
|
2006-06-27 17:54:53 +08:00
|
|
|
waiter = task->pi_blocked_on;
|
2014-06-10 01:40:34 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* [3] check_exit_conditions_1() protected by task->pi_lock.
|
|
|
|
*/
|
|
|
|
|
2006-06-27 17:54:53 +08:00
|
|
|
/*
|
|
|
|
* Check whether the end of the boosting chain has been
|
|
|
|
* reached or the state of the chain has changed while we
|
|
|
|
* dropped the locks.
|
|
|
|
*/
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
if (!waiter)
|
2006-06-27 17:54:53 +08:00
|
|
|
goto out_unlock_pi;
|
|
|
|
|
2007-06-09 04:46:58 +08:00
|
|
|
/*
|
|
|
|
* Check the orig_waiter state. After we dropped the locks,
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
* the previous owner of the lock might have released the lock.
|
2007-06-09 04:46:58 +08:00
|
|
|
*/
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
if (orig_waiter && !rt_mutex_owner(orig_lock))
|
2007-06-09 04:46:58 +08:00
|
|
|
goto out_unlock_pi;
|
|
|
|
|
2014-06-05 17:16:12 +08:00
|
|
|
/*
|
|
|
|
* We dropped all locks after taking a refcount on @task, so
|
|
|
|
* the task might have moved on in the lock chain or even left
|
|
|
|
* the chain completely and blocks now on an unrelated lock or
|
|
|
|
* on @orig_lock.
|
|
|
|
*
|
|
|
|
* We stored the lock on which @task was blocked in @next_lock,
|
|
|
|
* so we can detect the chain change.
|
|
|
|
*/
|
|
|
|
if (next_lock != waiter->lock)
|
|
|
|
goto out_unlock_pi;
|
|
|
|
|
2007-06-09 04:46:58 +08:00
|
|
|
/*
|
|
|
|
* Drop out, when the task has no waiters. Note,
|
|
|
|
* top_waiter can be NULL, when we are in the deboosting
|
|
|
|
* mode!
|
|
|
|
*/
|
2014-05-22 11:25:39 +08:00
|
|
|
if (top_waiter) {
|
|
|
|
if (!task_has_pi_waiters(task))
|
|
|
|
goto out_unlock_pi;
|
|
|
|
/*
|
|
|
|
* If deadlock detection is off, we stop here if we
|
2014-05-22 11:25:57 +08:00
|
|
|
* are not the top pi waiter of the task. If deadlock
|
|
|
|
* detection is enabled we continue, but stop the
|
|
|
|
* requeueing in the chain walk.
|
2014-05-22 11:25:39 +08:00
|
|
|
*/
|
2014-05-22 11:25:57 +08:00
|
|
|
if (top_waiter != task_top_pi_waiter(task)) {
|
|
|
|
if (!detect_deadlock)
|
|
|
|
goto out_unlock_pi;
|
|
|
|
else
|
|
|
|
requeue = false;
|
|
|
|
}
|
2014-05-22 11:25:39 +08:00
|
|
|
}
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
/*
|
2014-05-22 11:25:57 +08:00
|
|
|
* If the waiter priority is the same as the task priority
|
|
|
|
* then there is no further priority adjustment necessary. If
|
|
|
|
* deadlock detection is off, we stop the chain walk. If its
|
|
|
|
* enabled we continue, but stop the requeueing in the chain
|
|
|
|
* walk.
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
2017-03-23 22:56:14 +08:00
|
|
|
if (rt_mutex_waiter_equal(waiter, task_to_waiter(task))) {
|
2014-05-22 11:25:57 +08:00
|
|
|
if (!detect_deadlock)
|
|
|
|
goto out_unlock_pi;
|
|
|
|
else
|
|
|
|
requeue = false;
|
|
|
|
}
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-10 01:40:34 +08:00
|
|
|
/*
|
|
|
|
* [4] Get the next lock
|
|
|
|
*/
|
2006-06-27 17:54:53 +08:00
|
|
|
lock = waiter->lock;
|
2014-06-10 01:40:34 +08:00
|
|
|
/*
|
|
|
|
* [5] We need to trylock here as we are holding task->pi_lock,
|
|
|
|
* which is the reverse lock order versus the other rtmutex
|
|
|
|
* operations.
|
|
|
|
*/
|
2009-11-18 01:22:11 +08:00
|
|
|
if (!raw_spin_trylock(&lock->wait_lock)) {
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irq(&task->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
cpu_relax();
|
|
|
|
goto retry;
|
|
|
|
}
|
|
|
|
|
2014-05-22 11:25:39 +08:00
|
|
|
/*
|
2014-06-10 01:40:34 +08:00
|
|
|
* [6] check_exit_conditions_2() protected by task->pi_lock and
|
|
|
|
* lock->wait_lock.
|
|
|
|
*
|
2014-05-22 11:25:39 +08:00
|
|
|
* Deadlock detection. If the lock is the same as the original
|
|
|
|
* lock which caused us to walk the lock chain or if the
|
|
|
|
* current lock is owned by the task which initiated the chain
|
|
|
|
* walk, we detected a deadlock.
|
|
|
|
*/
|
2006-06-27 17:55:02 +08:00
|
|
|
if (lock == orig_lock || rt_mutex_owner(lock) == top_task) {
|
2009-11-18 01:22:11 +08:00
|
|
|
raw_spin_unlock(&lock->wait_lock);
|
2014-06-05 18:34:23 +08:00
|
|
|
ret = -EDEADLK;
|
2006-06-27 17:54:53 +08:00
|
|
|
goto out_unlock_pi;
|
|
|
|
}
|
|
|
|
|
2014-05-22 11:25:57 +08:00
|
|
|
/*
|
|
|
|
* If we just follow the lock chain for deadlock detection, no
|
|
|
|
* need to do all the requeue operations. To avoid a truckload
|
|
|
|
* of conditionals around the various places below, just do the
|
|
|
|
* minimum chain walk checks.
|
|
|
|
*/
|
|
|
|
if (!requeue) {
|
|
|
|
/*
|
|
|
|
* No requeue[7] here. Just release @task [8]
|
|
|
|
*/
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock(&task->pi_lock);
|
2014-05-22 11:25:57 +08:00
|
|
|
put_task_struct(task);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* [9] check_exit_conditions_3 protected by lock->wait_lock.
|
|
|
|
* If there is no owner of the lock, end of chain.
|
|
|
|
*/
|
|
|
|
if (!rt_mutex_owner(lock)) {
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irq(&lock->wait_lock);
|
2014-05-22 11:25:57 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* [10] Grab the next task, i.e. owner of @lock */
|
2019-07-05 06:13:23 +08:00
|
|
|
task = get_task_struct(rt_mutex_owner(lock));
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock(&task->pi_lock);
|
2014-05-22 11:25:57 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* No requeue [11] here. We just do deadlock detection.
|
|
|
|
*
|
|
|
|
* [12] Store whether owner is blocked
|
|
|
|
* itself. Decision is made after dropping the locks
|
|
|
|
*/
|
|
|
|
next_lock = task_blocked_on_lock(task);
|
|
|
|
/*
|
|
|
|
* Get the top waiter for the next iteration
|
|
|
|
*/
|
|
|
|
top_waiter = rt_mutex_top_waiter(lock);
|
|
|
|
|
|
|
|
/* [13] Drop locks */
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock(&task->pi_lock);
|
|
|
|
raw_spin_unlock_irq(&lock->wait_lock);
|
2014-05-22 11:25:57 +08:00
|
|
|
|
|
|
|
/* If owner is not blocked, end of chain. */
|
|
|
|
if (!next_lock)
|
|
|
|
goto out_put_task;
|
|
|
|
goto again;
|
|
|
|
}
|
|
|
|
|
2014-05-22 11:25:54 +08:00
|
|
|
/*
|
|
|
|
* Store the current top waiter before doing the requeue
|
|
|
|
* operation on @lock. We need it for the boost/deboost
|
|
|
|
* decision below.
|
|
|
|
*/
|
|
|
|
prerequeue_top_waiter = rt_mutex_top_waiter(lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2015-05-20 01:24:57 +08:00
|
|
|
/* [7] Requeue the waiter in the lock waiter tree. */
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_dequeue(lock, waiter);
|
2017-03-23 22:56:13 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Update the waiter prio fields now that we're dequeued.
|
|
|
|
*
|
|
|
|
* These values can have changed through either:
|
|
|
|
*
|
|
|
|
* sys_sched_set_scheduler() / sys_sched_setattr()
|
|
|
|
*
|
|
|
|
* or
|
|
|
|
*
|
|
|
|
* DL CBS enforcement advancing the effective deadline.
|
|
|
|
*
|
|
|
|
* Even though pi_waiters also uses these fields, and that tree is only
|
|
|
|
* updated in [11], we can do this here, since we hold [L], which
|
|
|
|
* serializes all pi_waiters access and rb_erase() does not care about
|
|
|
|
* the values of the node being removed.
|
|
|
|
*/
|
sched/deadline: Add SCHED_DEADLINE inheritance logic
Some method to deal with rt-mutexes and make sched_dl interact with
the current PI-coded is needed, raising all but trivial issues, that
needs (according to us) to be solved with some restructuring of
the pi-code (i.e., going toward a proxy execution-ish implementation).
This is under development, in the meanwhile, as a temporary solution,
what this commits does is:
- ensure a pi-lock owner with waiters is never throttled down. Instead,
when it runs out of runtime, it immediately gets replenished and it's
deadline is postponed;
- the scheduling parameters (relative deadline and default runtime)
used for that replenishments --during the whole period it holds the
pi-lock-- are the ones of the waiting task with earliest deadline.
Acting this way, we provide some kind of boosting to the lock-owner,
still by using the existing (actually, slightly modified by the previous
commit) pi-architecture.
We would stress the fact that this is only a surely needed, all but
clean solution to the problem. In the end it's only a way to re-start
discussion within the community. So, as always, comments, ideas, rants,
etc.. are welcome! :-)
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
[ Added !RT_MUTEXES build fix. ]
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-11-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:44 +08:00
|
|
|
waiter->prio = task->prio;
|
2017-03-23 22:56:13 +08:00
|
|
|
waiter->deadline = task->dl.deadline;
|
|
|
|
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_enqueue(lock, waiter);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-10 01:40:34 +08:00
|
|
|
/* [8] Release the task */
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock(&task->pi_lock);
|
2014-06-07 18:10:36 +08:00
|
|
|
put_task_struct(task);
|
|
|
|
|
2014-05-22 11:25:54 +08:00
|
|
|
/*
|
2014-06-10 01:40:34 +08:00
|
|
|
* [9] check_exit_conditions_3 protected by lock->wait_lock.
|
|
|
|
*
|
2014-05-22 11:25:54 +08:00
|
|
|
* We must abort the chain walk if there is no lock owner even
|
|
|
|
* in the dead lock detection case, as we have nothing to
|
|
|
|
* follow here. This is the end of the chain we are walking.
|
|
|
|
*/
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
if (!rt_mutex_owner(lock)) {
|
|
|
|
/*
|
2014-06-10 01:40:34 +08:00
|
|
|
* If the requeue [7] above changed the top waiter,
|
|
|
|
* then we need to wake the new top waiter up to try
|
|
|
|
* to get the lock.
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
*/
|
2014-05-22 11:25:54 +08:00
|
|
|
if (prerequeue_top_waiter != rt_mutex_top_waiter(lock))
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
wake_up_process(rt_mutex_top_waiter(lock)->task);
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irq(&lock->wait_lock);
|
2014-06-07 18:10:36 +08:00
|
|
|
return 0;
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
}
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-10 01:40:34 +08:00
|
|
|
/* [10] Grab the next task, i.e. the owner of @lock */
|
2019-07-05 06:13:23 +08:00
|
|
|
task = get_task_struct(rt_mutex_owner(lock));
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock(&task->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-10 01:40:34 +08:00
|
|
|
/* [11] requeue the pi waiters if necessary */
|
2006-06-27 17:54:53 +08:00
|
|
|
if (waiter == rt_mutex_top_waiter(lock)) {
|
2014-05-22 11:25:54 +08:00
|
|
|
/*
|
|
|
|
* The waiter became the new top (highest priority)
|
|
|
|
* waiter on the lock. Replace the previous top waiter
|
2015-05-20 01:24:57 +08:00
|
|
|
* in the owner tasks pi waiters tree with this waiter
|
2014-05-22 11:25:54 +08:00
|
|
|
* and adjust the priority of the owner.
|
|
|
|
*/
|
|
|
|
rt_mutex_dequeue_pi(task, prerequeue_top_waiter);
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_enqueue_pi(task, waiter);
|
2017-03-23 22:56:11 +08:00
|
|
|
rt_mutex_adjust_prio(task);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-05-22 11:25:54 +08:00
|
|
|
} else if (prerequeue_top_waiter == waiter) {
|
|
|
|
/*
|
|
|
|
* The waiter was the top waiter on the lock, but is
|
2021-03-22 09:35:05 +08:00
|
|
|
* no longer the top priority waiter. Replace waiter in
|
2015-05-20 01:24:57 +08:00
|
|
|
* the owner tasks pi waiters tree with the new top
|
2014-05-22 11:25:54 +08:00
|
|
|
* (highest priority) waiter and adjust the priority
|
|
|
|
* of the owner.
|
|
|
|
* The new top waiter is stored in @waiter so that
|
|
|
|
* @waiter == @top_waiter evaluates to true below and
|
|
|
|
* we continue to deboost the rest of the chain.
|
|
|
|
*/
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_dequeue_pi(task, waiter);
|
2006-06-27 17:54:53 +08:00
|
|
|
waiter = rt_mutex_top_waiter(lock);
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_enqueue_pi(task, waiter);
|
2017-03-23 22:56:11 +08:00
|
|
|
rt_mutex_adjust_prio(task);
|
2014-05-22 11:25:54 +08:00
|
|
|
} else {
|
|
|
|
/*
|
|
|
|
* Nothing changed. No need to do any priority
|
|
|
|
* adjustment.
|
|
|
|
*/
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
|
|
|
|
2014-06-05 17:16:12 +08:00
|
|
|
/*
|
2014-06-10 01:40:34 +08:00
|
|
|
* [12] check_exit_conditions_4() protected by task->pi_lock
|
|
|
|
* and lock->wait_lock. The actual decisions are made after we
|
|
|
|
* dropped the locks.
|
|
|
|
*
|
2014-06-05 17:16:12 +08:00
|
|
|
* Check whether the task which owns the current lock is pi
|
|
|
|
* blocked itself. If yes we store a pointer to the lock for
|
|
|
|
* the lock chain change detection above. After we dropped
|
|
|
|
* task->pi_lock next_lock cannot be dereferenced anymore.
|
|
|
|
*/
|
|
|
|
next_lock = task_blocked_on_lock(task);
|
2014-05-22 11:25:54 +08:00
|
|
|
/*
|
|
|
|
* Store the top waiter of @lock for the end of chain walk
|
|
|
|
* decision below.
|
|
|
|
*/
|
2006-06-27 17:54:53 +08:00
|
|
|
top_waiter = rt_mutex_top_waiter(lock);
|
2014-06-10 01:40:34 +08:00
|
|
|
|
|
|
|
/* [13] Drop the locks */
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock(&task->pi_lock);
|
|
|
|
raw_spin_unlock_irq(&lock->wait_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-05 17:16:12 +08:00
|
|
|
/*
|
2014-06-10 01:40:34 +08:00
|
|
|
* Make the actual exit decisions [12], based on the stored
|
|
|
|
* values.
|
|
|
|
*
|
2014-06-05 17:16:12 +08:00
|
|
|
* We reached the end of the lock chain. Stop right here. No
|
|
|
|
* point to go back just to figure that out.
|
|
|
|
*/
|
|
|
|
if (!next_lock)
|
|
|
|
goto out_put_task;
|
|
|
|
|
2014-05-22 11:25:54 +08:00
|
|
|
/*
|
|
|
|
* If the current waiter is not the top waiter on the lock,
|
|
|
|
* then we can stop the chain walk here if we are not in full
|
|
|
|
* deadlock detection mode.
|
|
|
|
*/
|
2006-06-27 17:54:53 +08:00
|
|
|
if (!detect_deadlock && waiter != top_waiter)
|
|
|
|
goto out_put_task;
|
|
|
|
|
|
|
|
goto again;
|
|
|
|
|
|
|
|
out_unlock_pi:
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irq(&task->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
out_put_task:
|
|
|
|
put_task_struct(task);
|
2006-07-03 15:25:41 +08:00
|
|
|
|
2006-06-27 17:54:53 +08:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Try to take an rt-mutex
|
|
|
|
*
|
2016-01-13 18:25:38 +08:00
|
|
|
* Must be called with lock->wait_lock held and interrupts disabled
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
*
|
2014-06-11 07:01:13 +08:00
|
|
|
* @lock: The lock to be acquired.
|
|
|
|
* @task: The task which wants to acquire the lock
|
2015-05-20 01:24:57 +08:00
|
|
|
* @waiter: The waiter that is queued to the lock's wait tree if the
|
2014-06-11 07:01:13 +08:00
|
|
|
* callsite called task_blocked_on_lock(), otherwise NULL
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
2021-03-26 23:29:40 +08:00
|
|
|
static int __sched
|
|
|
|
try_to_take_rt_mutex(struct rt_mutex *lock, struct task_struct *task,
|
|
|
|
struct rt_mutex_waiter *waiter)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
2017-03-23 22:56:13 +08:00
|
|
|
lockdep_assert_held(&lock->wait_lock);
|
|
|
|
|
2006-06-27 17:54:53 +08:00
|
|
|
/*
|
2014-06-11 07:01:13 +08:00
|
|
|
* Before testing whether we can acquire @lock, we set the
|
|
|
|
* RT_MUTEX_HAS_WAITERS bit in @lock->owner. This forces all
|
|
|
|
* other tasks which try to modify @lock into the slow path
|
|
|
|
* and they serialize on @lock->wait_lock.
|
2006-06-27 17:54:53 +08:00
|
|
|
*
|
2014-06-11 07:01:13 +08:00
|
|
|
* The RT_MUTEX_HAS_WAITERS bit can have a transitional state
|
|
|
|
* as explained at the top of this file if and only if:
|
2006-06-27 17:54:53 +08:00
|
|
|
*
|
2014-06-11 07:01:13 +08:00
|
|
|
* - There is a lock owner. The caller must fixup the
|
|
|
|
* transient state if it does a trylock or leaves the lock
|
|
|
|
* function due to a signal or timeout.
|
|
|
|
*
|
|
|
|
* - @task acquires the lock and there are no other
|
|
|
|
* waiters. This is undone in rt_mutex_set_owner(@task) at
|
|
|
|
* the end of this function.
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
|
|
|
mark_rt_mutex_waiters(lock);
|
|
|
|
|
2014-06-11 07:01:13 +08:00
|
|
|
/*
|
|
|
|
* If @lock has an owner, give up.
|
|
|
|
*/
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
if (rt_mutex_owner(lock))
|
2006-06-27 17:54:53 +08:00
|
|
|
return 0;
|
|
|
|
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
/*
|
2014-06-11 07:01:13 +08:00
|
|
|
* If @waiter != NULL, @task has already enqueued the waiter
|
2015-05-20 01:24:57 +08:00
|
|
|
* into @lock waiter tree. If @waiter == NULL then this is a
|
2014-06-11 07:01:13 +08:00
|
|
|
* trylock attempt.
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
*/
|
2014-06-11 07:01:13 +08:00
|
|
|
if (waiter) {
|
|
|
|
/*
|
|
|
|
* If waiter is not the highest priority waiter of
|
|
|
|
* @lock, give up.
|
|
|
|
*/
|
|
|
|
if (waiter != rt_mutex_top_waiter(lock))
|
|
|
|
return 0;
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
|
2014-06-11 07:01:13 +08:00
|
|
|
/*
|
|
|
|
* We can acquire the lock. Remove the waiter from the
|
2015-05-20 01:24:57 +08:00
|
|
|
* lock waiters tree.
|
2014-06-11 07:01:13 +08:00
|
|
|
*/
|
|
|
|
rt_mutex_dequeue(lock, waiter);
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
|
2014-06-11 07:01:13 +08:00
|
|
|
} else {
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
/*
|
2014-06-11 07:01:13 +08:00
|
|
|
* If the lock has waiters already we check whether @task is
|
|
|
|
* eligible to take over the lock.
|
|
|
|
*
|
|
|
|
* If there are no other waiters, @task can acquire
|
|
|
|
* the lock. @task->pi_blocked_on is NULL, so it does
|
|
|
|
* not need to be dequeued.
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
*/
|
|
|
|
if (rt_mutex_has_waiters(lock)) {
|
2014-06-11 07:01:13 +08:00
|
|
|
/*
|
|
|
|
* If @task->prio is greater than or equal to
|
|
|
|
* the top waiter priority (kernel view),
|
|
|
|
* @task lost.
|
|
|
|
*/
|
2017-03-23 22:56:14 +08:00
|
|
|
if (!rt_mutex_waiter_less(task_to_waiter(task),
|
|
|
|
rt_mutex_top_waiter(lock)))
|
2014-06-11 07:01:13 +08:00
|
|
|
return 0;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* The current top waiter stays enqueued. We
|
|
|
|
* don't have to change anything in the lock
|
|
|
|
* waiters order.
|
|
|
|
*/
|
|
|
|
} else {
|
|
|
|
/*
|
|
|
|
* No waiters. Take the lock without the
|
|
|
|
* pi_lock dance.@task->pi_blocked_on is NULL
|
|
|
|
* and we have no waiters to enqueue in @task
|
2015-05-20 01:24:57 +08:00
|
|
|
* pi waiters tree.
|
2014-06-11 07:01:13 +08:00
|
|
|
*/
|
|
|
|
goto takeit;
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2014-06-11 07:01:13 +08:00
|
|
|
/*
|
|
|
|
* Clear @task->pi_blocked_on. Requires protection by
|
|
|
|
* @task->pi_lock. Redundant operation for the @waiter == NULL
|
|
|
|
* case, but conditionals are more expensive than a redundant
|
|
|
|
* store.
|
|
|
|
*/
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock(&task->pi_lock);
|
2014-06-11 07:01:13 +08:00
|
|
|
task->pi_blocked_on = NULL;
|
|
|
|
/*
|
|
|
|
* Finish the lock acquisition. @task is the new owner. If
|
|
|
|
* other waiters exist we have to insert the highest priority
|
2015-05-20 01:24:57 +08:00
|
|
|
* waiter into @task->pi_waiters tree.
|
2014-06-11 07:01:13 +08:00
|
|
|
*/
|
|
|
|
if (rt_mutex_has_waiters(lock))
|
|
|
|
rt_mutex_enqueue_pi(task, rt_mutex_top_waiter(lock));
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock(&task->pi_lock);
|
2014-06-11 07:01:13 +08:00
|
|
|
|
|
|
|
takeit:
|
|
|
|
/*
|
|
|
|
* This either preserves the RT_MUTEX_HAS_WAITERS bit if there
|
|
|
|
* are still waiters or clears it.
|
|
|
|
*/
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
rt_mutex_set_owner(lock, task);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Task blocks on lock.
|
|
|
|
*
|
|
|
|
* Prepare waiter and propagate pi chain
|
|
|
|
*
|
2016-01-13 18:25:38 +08:00
|
|
|
* This must be called with lock->wait_lock held and interrupts disabled
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
2021-03-26 23:29:40 +08:00
|
|
|
static int __sched task_blocks_on_rt_mutex(struct rt_mutex *lock,
|
|
|
|
struct rt_mutex_waiter *waiter,
|
|
|
|
struct task_struct *task,
|
|
|
|
enum rtmutex_chainwalk chwalk)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
2006-07-03 15:25:41 +08:00
|
|
|
struct task_struct *owner = rt_mutex_owner(lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
struct rt_mutex_waiter *top_waiter = waiter;
|
2014-06-05 17:16:12 +08:00
|
|
|
struct rt_mutex *next_lock;
|
2006-09-29 16:59:44 +08:00
|
|
|
int chain_walk = 0, res;
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2017-03-23 22:56:13 +08:00
|
|
|
lockdep_assert_held(&lock->wait_lock);
|
|
|
|
|
2014-05-22 11:25:39 +08:00
|
|
|
/*
|
|
|
|
* Early deadlock detection. We really don't want the task to
|
|
|
|
* enqueue on itself just to untangle the mess later. It's not
|
|
|
|
* only an optimization. We drop the locks, so another waiter
|
|
|
|
* can come in before the chain walk detects the deadlock. So
|
|
|
|
* the other will detect the deadlock and return -EDEADLOCK,
|
|
|
|
* which is wrong, as the other waiter is not in a deadlock
|
|
|
|
* situation.
|
|
|
|
*/
|
2014-06-05 18:34:23 +08:00
|
|
|
if (owner == task)
|
2014-05-22 11:25:39 +08:00
|
|
|
return -EDEADLK;
|
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock(&task->pi_lock);
|
2009-04-04 04:40:12 +08:00
|
|
|
waiter->task = task;
|
2006-06-27 17:54:53 +08:00
|
|
|
waiter->lock = lock;
|
sched/deadline: Add SCHED_DEADLINE inheritance logic
Some method to deal with rt-mutexes and make sched_dl interact with
the current PI-coded is needed, raising all but trivial issues, that
needs (according to us) to be solved with some restructuring of
the pi-code (i.e., going toward a proxy execution-ish implementation).
This is under development, in the meanwhile, as a temporary solution,
what this commits does is:
- ensure a pi-lock owner with waiters is never throttled down. Instead,
when it runs out of runtime, it immediately gets replenished and it's
deadline is postponed;
- the scheduling parameters (relative deadline and default runtime)
used for that replenishments --during the whole period it holds the
pi-lock-- are the ones of the waiting task with earliest deadline.
Acting this way, we provide some kind of boosting to the lock-owner,
still by using the existing (actually, slightly modified by the previous
commit) pi-architecture.
We would stress the fact that this is only a surely needed, all but
clean solution to the problem. In the end it's only a way to re-start
discussion within the community. So, as always, comments, ideas, rants,
etc.. are welcome! :-)
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
[ Added !RT_MUTEXES build fix. ]
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-11-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:44 +08:00
|
|
|
waiter->prio = task->prio;
|
2017-03-23 22:56:13 +08:00
|
|
|
waiter->deadline = task->dl.deadline;
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
/* Get the top priority waiter on the lock */
|
|
|
|
if (rt_mutex_has_waiters(lock))
|
|
|
|
top_waiter = rt_mutex_top_waiter(lock);
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_enqueue(lock, waiter);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2009-04-04 04:40:12 +08:00
|
|
|
task->pi_blocked_on = waiter;
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock(&task->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
if (!owner)
|
|
|
|
return 0;
|
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock(&owner->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
if (waiter == rt_mutex_top_waiter(lock)) {
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_dequeue_pi(owner, top_waiter);
|
|
|
|
rt_mutex_enqueue_pi(owner, waiter);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2017-03-23 22:56:11 +08:00
|
|
|
rt_mutex_adjust_prio(owner);
|
2006-09-29 16:59:44 +08:00
|
|
|
if (owner->pi_blocked_on)
|
|
|
|
chain_walk = 1;
|
2014-05-22 11:25:47 +08:00
|
|
|
} else if (rt_mutex_cond_detect_deadlock(waiter, chwalk)) {
|
2006-09-29 16:59:44 +08:00
|
|
|
chain_walk = 1;
|
2014-06-05 17:16:12 +08:00
|
|
|
}
|
2006-09-29 16:59:44 +08:00
|
|
|
|
2014-06-05 17:16:12 +08:00
|
|
|
/* Store the lock on which owner is blocked or NULL */
|
|
|
|
next_lock = task_blocked_on_lock(owner);
|
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock(&owner->pi_lock);
|
2014-06-05 17:16:12 +08:00
|
|
|
/*
|
|
|
|
* Even if full deadlock detection is on, if the owner is not
|
|
|
|
* blocked itself, we can avoid finding this out in the chain
|
|
|
|
* walk.
|
|
|
|
*/
|
|
|
|
if (!chain_walk || !next_lock)
|
2006-06-27 17:54:53 +08:00
|
|
|
return 0;
|
|
|
|
|
2006-09-29 16:59:44 +08:00
|
|
|
/*
|
|
|
|
* The owner can't disappear while holding a lock,
|
|
|
|
* so the owner struct is protected by wait_lock.
|
|
|
|
* Gets dropped in rt_mutex_adjust_prio_chain()!
|
|
|
|
*/
|
|
|
|
get_task_struct(owner);
|
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irq(&lock->wait_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-05-22 11:25:47 +08:00
|
|
|
res = rt_mutex_adjust_prio_chain(owner, chwalk, lock,
|
2014-06-05 17:16:12 +08:00
|
|
|
next_lock, waiter, task);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock_irq(&lock->wait_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
return res;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
2015-05-20 01:24:57 +08:00
|
|
|
* Remove the top waiter from the current tasks pi waiter tree and
|
2015-05-20 01:24:55 +08:00
|
|
|
* queue it up.
|
2006-06-27 17:54:53 +08:00
|
|
|
*
|
2016-01-13 18:25:38 +08:00
|
|
|
* Called with lock->wait_lock held and interrupts disabled.
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
2021-03-26 23:29:40 +08:00
|
|
|
static void __sched mark_wakeup_next_waiter(struct wake_q_head *wake_q,
|
|
|
|
struct rt_mutex *lock)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
|
|
|
struct rt_mutex_waiter *waiter;
|
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock(¤t->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
waiter = rt_mutex_top_waiter(lock);
|
|
|
|
|
|
|
|
/*
|
2017-03-23 22:56:11 +08:00
|
|
|
* Remove it from current->pi_waiters and deboost.
|
|
|
|
*
|
|
|
|
* We must in fact deboost here in order to ensure we call
|
|
|
|
* rt_mutex_setprio() to update p->pi_top_task before the
|
|
|
|
* task unblocks.
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_dequeue_pi(current, waiter);
|
2017-03-23 22:56:11 +08:00
|
|
|
rt_mutex_adjust_prio(current);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-12 02:44:04 +08:00
|
|
|
/*
|
|
|
|
* As we are waking up the top waiter, and the waiter stays
|
|
|
|
* queued on the lock until it gets the lock, this lock
|
|
|
|
* obviously has waiters. Just set the bit here and this has
|
|
|
|
* the added benefit of forcing all new tasks into the
|
|
|
|
* slow path making sure no task of lower priority than
|
|
|
|
* the top waiter can steal this lock.
|
|
|
|
*/
|
|
|
|
lock->owner = (void *) RT_MUTEX_HAS_WAITERS;
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2017-03-23 22:56:11 +08:00
|
|
|
/*
|
|
|
|
* We deboosted before waking the top waiter task such that we don't
|
|
|
|
* run two tasks with the 'same' priority (and ensure the
|
|
|
|
* p->pi_top_task pointer points to a blocked task). This however can
|
|
|
|
* lead to priority inversion if we would get preempted after the
|
|
|
|
* deboost but before waking our donor task, hence the preempt_disable()
|
|
|
|
* before unlock.
|
|
|
|
*
|
|
|
|
* Pairs with preempt_enable() in rt_mutex_postunlock();
|
|
|
|
*/
|
|
|
|
preempt_disable();
|
2015-05-20 01:24:55 +08:00
|
|
|
wake_q_add(wake_q, waiter->task);
|
2017-03-23 22:56:11 +08:00
|
|
|
raw_spin_unlock(¤t->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
* Remove a waiter from a lock and give up
|
2006-06-27 17:54:53 +08:00
|
|
|
*
|
2016-01-13 18:25:38 +08:00
|
|
|
* Must be called with lock->wait_lock held and interrupts disabled. I must
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
* have just failed to try_to_take_rt_mutex().
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
2021-03-26 23:29:40 +08:00
|
|
|
static void __sched remove_waiter(struct rt_mutex *lock,
|
|
|
|
struct rt_mutex_waiter *waiter)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
2014-06-07 15:36:13 +08:00
|
|
|
bool is_top_waiter = (waiter == rt_mutex_top_waiter(lock));
|
2006-07-03 15:25:41 +08:00
|
|
|
struct task_struct *owner = rt_mutex_owner(lock);
|
2014-06-07 15:36:13 +08:00
|
|
|
struct rt_mutex *next_lock;
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2017-03-23 22:56:13 +08:00
|
|
|
lockdep_assert_held(&lock->wait_lock);
|
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock(¤t->pi_lock);
|
rtmutex: Turn the plist into an rb-tree
Turn the pi-chains from plist to rb-tree, in the rt_mutex code,
and provide a proper comparison function for -deadline and
-priority tasks.
This is done mainly because:
- classical prio field of the plist is just an int, which might
not be enough for representing a deadline;
- manipulating such a list would become O(nr_deadline_tasks),
which might be to much, as the number of -deadline task increases.
Therefore, an rb-tree is used, and tasks are queued in it according
to the following logic:
- among two -priority (i.e., SCHED_BATCH/OTHER/RR/FIFO) tasks, the
one with the higher (lower, actually!) prio wins;
- among a -priority and a -deadline task, the latter always wins;
- among two -deadline tasks, the one with the earliest deadline
wins.
Queueing and dequeueing functions are changed accordingly, for both
the list of a task's pi-waiters and the list of tasks blocked on
a pi-lock.
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Dario Faggioli <raistlin@linux.it>
Signed-off-by: Juri Lelli <juri.lelli@gmail.com>
Signed-off-again-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1383831828-15501-10-git-send-email-juri.lelli@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
2013-11-07 21:43:43 +08:00
|
|
|
rt_mutex_dequeue(lock, waiter);
|
2006-06-27 17:54:53 +08:00
|
|
|
current->pi_blocked_on = NULL;
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock(¤t->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-07 15:36:13 +08:00
|
|
|
/*
|
|
|
|
* Only update priority if the waiter was the highest priority
|
|
|
|
* waiter of the lock and there is an owner to update.
|
|
|
|
*/
|
|
|
|
if (!owner || !is_top_waiter)
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
return;
|
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock(&owner->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-07 15:36:13 +08:00
|
|
|
rt_mutex_dequeue_pi(owner, waiter);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-07 15:36:13 +08:00
|
|
|
if (rt_mutex_has_waiters(lock))
|
|
|
|
rt_mutex_enqueue_pi(owner, rt_mutex_top_waiter(lock));
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2017-03-23 22:56:11 +08:00
|
|
|
rt_mutex_adjust_prio(owner);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-07 15:36:13 +08:00
|
|
|
/* Store the lock on which owner is blocked or NULL */
|
|
|
|
next_lock = task_blocked_on_lock(owner);
|
2006-09-29 16:59:44 +08:00
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock(&owner->pi_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-07 15:36:13 +08:00
|
|
|
/*
|
|
|
|
* Don't walk the chain, if the owner task is not blocked
|
|
|
|
* itself.
|
|
|
|
*/
|
2014-06-05 17:16:12 +08:00
|
|
|
if (!next_lock)
|
2006-06-27 17:54:53 +08:00
|
|
|
return;
|
|
|
|
|
2006-09-29 16:59:44 +08:00
|
|
|
/* gets dropped in rt_mutex_adjust_prio_chain()! */
|
|
|
|
get_task_struct(owner);
|
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irq(&lock->wait_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-05-22 11:25:47 +08:00
|
|
|
rt_mutex_adjust_prio_chain(owner, RT_MUTEX_MIN_CHAINWALK, lock,
|
|
|
|
next_lock, NULL, current);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock_irq(&lock->wait_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
|
|
|
|
2009-04-04 04:40:12 +08:00
|
|
|
/**
|
|
|
|
* __rt_mutex_slowlock() - Perform the wait-wake-try-to-take loop
|
|
|
|
* @lock: the rt_mutex to take
|
|
|
|
* @state: the state the task should block in (TASK_INTERRUPTIBLE
|
2016-01-13 18:25:38 +08:00
|
|
|
* or TASK_UNINTERRUPTIBLE)
|
2009-04-04 04:40:12 +08:00
|
|
|
* @timeout: the pre-initialized and started timer, or NULL for none
|
|
|
|
* @waiter: the pre-initialized rt_mutex_waiter
|
|
|
|
*
|
2016-01-13 18:25:38 +08:00
|
|
|
* Must be called with lock->wait_lock held and interrupts disabled
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
2021-06-11 16:28:17 +08:00
|
|
|
static int __sched __rt_mutex_slowlock(struct rt_mutex *lock, unsigned int state,
|
2021-03-26 23:29:40 +08:00
|
|
|
struct hrtimer_sleeper *timeout,
|
|
|
|
struct rt_mutex_waiter *waiter)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
|
|
|
int ret = 0;
|
|
|
|
|
|
|
|
for (;;) {
|
|
|
|
/* Try to acquire the lock: */
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
if (try_to_take_rt_mutex(lock, current, waiter))
|
2006-06-27 17:54:53 +08:00
|
|
|
break;
|
|
|
|
|
2021-03-26 23:29:44 +08:00
|
|
|
if (timeout && !timeout->task) {
|
|
|
|
ret = -ETIMEDOUT;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
if (signal_pending_state(state, current)) {
|
|
|
|
ret = -EINTR;
|
|
|
|
break;
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irq(&lock->wait_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2015-07-02 04:29:48 +08:00
|
|
|
schedule();
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock_irq(&lock->wait_lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
set_current_state(state);
|
|
|
|
}
|
|
|
|
|
2015-02-02 14:16:24 +08:00
|
|
|
__set_current_state(TASK_RUNNING);
|
2009-04-04 04:40:12 +08:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static void __sched rt_mutex_handle_deadlock(int res, int detect_deadlock,
|
|
|
|
struct rt_mutex_waiter *w)
|
2014-06-05 18:34:23 +08:00
|
|
|
{
|
|
|
|
/*
|
|
|
|
* If the result is not -EDEADLOCK or the caller requested
|
|
|
|
* deadlock detection, nothing to do here.
|
|
|
|
*/
|
|
|
|
if (res != -EDEADLOCK || detect_deadlock)
|
|
|
|
return;
|
|
|
|
|
|
|
|
/*
|
2021-03-22 09:35:05 +08:00
|
|
|
* Yell loudly and stop the task right here.
|
2014-06-05 18:34:23 +08:00
|
|
|
*/
|
2021-03-26 23:29:32 +08:00
|
|
|
WARN(1, "rtmutex deadlock detected\n");
|
2014-06-05 18:34:23 +08:00
|
|
|
while (1) {
|
|
|
|
set_current_state(TASK_INTERRUPTIBLE);
|
|
|
|
schedule();
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2009-04-04 04:40:12 +08:00
|
|
|
/*
|
|
|
|
* Slow path lock function:
|
|
|
|
*/
|
2021-06-11 16:28:17 +08:00
|
|
|
static int __sched rt_mutex_slowlock(struct rt_mutex *lock, unsigned int state,
|
2021-03-26 23:29:40 +08:00
|
|
|
struct hrtimer_sleeper *timeout,
|
|
|
|
enum rtmutex_chainwalk chwalk)
|
2009-04-04 04:40:12 +08:00
|
|
|
{
|
|
|
|
struct rt_mutex_waiter waiter;
|
2016-01-13 18:25:38 +08:00
|
|
|
unsigned long flags;
|
2009-04-04 04:40:12 +08:00
|
|
|
int ret = 0;
|
|
|
|
|
2017-03-22 18:35:56 +08:00
|
|
|
rt_mutex_init_waiter(&waiter);
|
2009-04-04 04:40:12 +08:00
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
/*
|
|
|
|
* Technically we could use raw_spin_[un]lock_irq() here, but this can
|
|
|
|
* be called in early boot if the cmpxchg() fast path is disabled
|
|
|
|
* (debug, no architecture support). In this case we will acquire the
|
|
|
|
* rtmutex with lock->wait_lock held. But we cannot unconditionally
|
|
|
|
* enable interrupts in that early boot case. So we need to use the
|
|
|
|
* irqsave/restore variants.
|
|
|
|
*/
|
|
|
|
raw_spin_lock_irqsave(&lock->wait_lock, flags);
|
2009-04-04 04:40:12 +08:00
|
|
|
|
|
|
|
/* Try to acquire the lock again: */
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
if (try_to_take_rt_mutex(lock, current, NULL)) {
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irqrestore(&lock->wait_lock, flags);
|
2009-04-04 04:40:12 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
set_current_state(state);
|
|
|
|
|
|
|
|
/* Setup the timer, when timeout != NULL */
|
2015-04-15 05:09:15 +08:00
|
|
|
if (unlikely(timeout))
|
2009-04-04 04:40:12 +08:00
|
|
|
hrtimer_start_expires(&timeout->timer, HRTIMER_MODE_ABS);
|
|
|
|
|
2014-05-22 11:25:47 +08:00
|
|
|
ret = task_blocks_on_rt_mutex(lock, &waiter, current, chwalk);
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
|
|
|
|
if (likely(!ret))
|
2015-02-02 14:16:24 +08:00
|
|
|
/* sleep on the mutex */
|
rtmutex: Simplify PI algorithm and make highest prio task get lock
In current rtmutex, the pending owner may be boosted by the tasks
in the rtmutex's waitlist when the pending owner is deboosted
or a task in the waitlist is boosted. This boosting is unrelated,
because the pending owner does not really take the rtmutex.
It is not reasonable.
Example.
time1:
A(high prio) onwers the rtmutex.
B(mid prio) and C (low prio) in the waitlist.
time2
A release the lock, B becomes the pending owner
A(or other high prio task) continues to run. B's prio is lower
than A, so B is just queued at the runqueue.
time3
A or other high prio task sleeps, but we have passed some time
The B and C's prio are changed in the period (time2 ~ time3)
due to boosting or deboosting. Now C has the priority higher
than B. ***Is it reasonable that C has to boost B and help B to
get the rtmutex?
NO!! I think, it is unrelated/unneed boosting before B really
owns the rtmutex. We should give C a chance to beat B and
win the rtmutex.
This is the motivation of this patch. This patch *ensures*
only the top waiter or higher priority task can take the lock.
How?
1) we don't dequeue the top waiter when unlock, if the top waiter
is changed, the old top waiter will fail and go to sleep again.
2) when requiring lock, it will get the lock when the lock is not taken and:
there is no waiter OR higher priority than waiters OR it is top waiter.
3) In any time, the top waiter is changed, the top waiter will be woken up.
The algorithm is much simpler than before, no pending owner, no
boosting for pending owner.
Other advantage of this patch:
1) The states of a rtmutex are reduced a half, easier to read the code.
2) the codes become shorter.
3) top waiter is not dequeued until it really take the lock:
they will retain FIFO when it is stolen.
Not advantage nor disadvantage
1) Even we may wakeup multiple waiters(any time when top waiter changed),
we hardly cause "thundering herd",
the number of wokenup task is likely 1 or very little.
2) two APIs are changed.
rt_mutex_owner() will not return pending owner, it will return NULL when
the top waiter is going to take the lock.
rt_mutex_next_owner() always return the top waiter.
will not return NULL if we have waiters
because the top waiter is not dequeued.
I have fixed the code that use these APIs.
need updated after this patch is accepted
1) Document/*
2) the testcase scripts/rt-tester/t4-l2-pi-deboost.tst
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
LKML-Reference: <4D3012D5.4060709@cn.fujitsu.com>
Reviewed-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
2011-01-14 17:09:41 +08:00
|
|
|
ret = __rt_mutex_slowlock(lock, state, timeout, &waiter);
|
2009-04-04 04:40:12 +08:00
|
|
|
|
2014-06-05 18:34:23 +08:00
|
|
|
if (unlikely(ret)) {
|
2015-02-28 00:57:09 +08:00
|
|
|
__set_current_state(TASK_RUNNING);
|
2018-03-27 20:14:38 +08:00
|
|
|
remove_waiter(lock, &waiter);
|
2014-05-22 11:25:47 +08:00
|
|
|
rt_mutex_handle_deadlock(ret, chwalk, &waiter);
|
2014-06-05 18:34:23 +08:00
|
|
|
}
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* try_to_take_rt_mutex() sets the waiter bit
|
|
|
|
* unconditionally. We might have to fix that up.
|
|
|
|
*/
|
|
|
|
fixup_rt_mutex_waiters(lock);
|
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irqrestore(&lock->wait_lock, flags);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
/* Remove pending timer: */
|
|
|
|
if (unlikely(timeout))
|
|
|
|
hrtimer_cancel(&timeout->timer);
|
|
|
|
|
|
|
|
debug_rt_mutex_free_waiter(&waiter);
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2021-08-16 05:27:57 +08:00
|
|
|
static __always_inline int __rt_mutex_lock(struct rt_mutex *lock,
|
|
|
|
unsigned int state)
|
|
|
|
{
|
|
|
|
if (likely(rt_mutex_cmpxchg_acquire(lock, NULL, current)))
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
return rt_mutex_slowlock(lock, state, NULL, RT_MUTEX_MIN_CHAINWALK);
|
|
|
|
}
|
|
|
|
|
2021-03-26 23:29:40 +08:00
|
|
|
static int __sched __rt_mutex_slowtrylock(struct rt_mutex *lock)
|
futex: Avoid violating the 10th rule of futex
Julia reported futex state corruption in the following scenario:
waiter waker stealer (prio > waiter)
futex(WAIT_REQUEUE_PI, uaddr, uaddr2,
timeout=[N ms])
futex_wait_requeue_pi()
futex_wait_queue_me()
freezable_schedule()
<scheduled out>
futex(LOCK_PI, uaddr2)
futex(CMP_REQUEUE_PI, uaddr,
uaddr2, 1, 0)
/* requeues waiter to uaddr2 */
futex(UNLOCK_PI, uaddr2)
wake_futex_pi()
cmp_futex_value_locked(uaddr2, waiter)
wake_up_q()
<woken by waker>
<hrtimer_wakeup() fires,
clears sleeper->task>
futex(LOCK_PI, uaddr2)
__rt_mutex_start_proxy_lock()
try_to_take_rt_mutex() /* steals lock */
rt_mutex_set_owner(lock, stealer)
<preempted>
<scheduled in>
rt_mutex_wait_proxy_lock()
__rt_mutex_slowlock()
try_to_take_rt_mutex() /* fails, lock held by stealer */
if (timeout && !timeout->task)
return -ETIMEDOUT;
fixup_owner()
/* lock wasn't acquired, so,
fixup_pi_state_owner skipped */
return -ETIMEDOUT;
/* At this point, we've returned -ETIMEDOUT to userspace, but the
* futex word shows waiter to be the owner, and the pi_mutex has
* stealer as the owner */
futex_lock(LOCK_PI, uaddr2)
-> bails with EDEADLK, futex word says we're owner.
And suggested that what commit:
73d786bd043e ("futex: Rework inconsistent rt_mutex/futex_q state")
removes from fixup_owner() looks to be just what is needed. And indeed
it is -- I completely missed that requeue_pi could also result in this
case. So we need to restore that, except that subsequent patches, like
commit:
16ffa12d7425 ("futex: Pull rt_mutex_futex_unlock() out from under hb->lock")
changed all the locking rules. Even without that, the sequence:
- if (rt_mutex_futex_trylock(&q->pi_state->pi_mutex)) {
- locked = 1;
- goto out;
- }
- raw_spin_lock_irq(&q->pi_state->pi_mutex.wait_lock);
- owner = rt_mutex_owner(&q->pi_state->pi_mutex);
- if (!owner)
- owner = rt_mutex_next_owner(&q->pi_state->pi_mutex);
- raw_spin_unlock_irq(&q->pi_state->pi_mutex.wait_lock);
- ret = fixup_pi_state_owner(uaddr, q, owner);
already suggests there were races; otherwise we'd never have to look
at next_owner.
So instead of doing 3 consecutive wait_lock sections with who knows
what races, we do it all in a single section. Additionally, the usage
of pi_state->owner in fixup_owner() was only safe because only the
rt_mutex owner would modify it, which this additional case wrecks.
Luckily the values can only change away and not to the value we're
testing, this means we can do a speculative test and double check once
we have the wait_lock.
Fixes: 73d786bd043e ("futex: Rework inconsistent rt_mutex/futex_q state")
Reported-by: Julia Cartwright <julia@ni.com>
Reported-by: Gratian Crisan <gratian.crisan@ni.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Tested-by: Julia Cartwright <julia@ni.com>
Tested-by: Gratian Crisan <gratian.crisan@ni.com>
Cc: Darren Hart <dvhart@infradead.org>
Cc: stable@vger.kernel.org
Link: https://lkml.kernel.org/r/20171208124939.7livp7no2ov65rrc@hirez.programming.kicks-ass.net
2017-12-08 20:49:39 +08:00
|
|
|
{
|
|
|
|
int ret = try_to_take_rt_mutex(lock, current, NULL);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* try_to_take_rt_mutex() sets the lock waiters bit
|
|
|
|
* unconditionally. Clean this up.
|
|
|
|
*/
|
|
|
|
fixup_rt_mutex_waiters(lock);
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2006-06-27 17:54:53 +08:00
|
|
|
/*
|
|
|
|
* Slow path try-lock function:
|
|
|
|
*/
|
2021-03-26 23:29:40 +08:00
|
|
|
static int __sched rt_mutex_slowtrylock(struct rt_mutex *lock)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
2016-01-13 18:25:38 +08:00
|
|
|
unsigned long flags;
|
2014-06-11 04:53:40 +08:00
|
|
|
int ret;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* If the lock already has an owner we fail to get the lock.
|
|
|
|
* This can be done without taking the @lock->wait_lock as
|
|
|
|
* it is only being read, and this is a trylock anyway.
|
|
|
|
*/
|
|
|
|
if (rt_mutex_owner(lock))
|
|
|
|
return 0;
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2014-06-11 04:53:40 +08:00
|
|
|
/*
|
2016-01-13 18:25:38 +08:00
|
|
|
* The mutex has currently no owner. Lock the wait lock and try to
|
|
|
|
* acquire the lock. We use irqsave here to support early boot calls.
|
2014-06-11 04:53:40 +08:00
|
|
|
*/
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock_irqsave(&lock->wait_lock, flags);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
futex: Avoid violating the 10th rule of futex
Julia reported futex state corruption in the following scenario:
waiter waker stealer (prio > waiter)
futex(WAIT_REQUEUE_PI, uaddr, uaddr2,
timeout=[N ms])
futex_wait_requeue_pi()
futex_wait_queue_me()
freezable_schedule()
<scheduled out>
futex(LOCK_PI, uaddr2)
futex(CMP_REQUEUE_PI, uaddr,
uaddr2, 1, 0)
/* requeues waiter to uaddr2 */
futex(UNLOCK_PI, uaddr2)
wake_futex_pi()
cmp_futex_value_locked(uaddr2, waiter)
wake_up_q()
<woken by waker>
<hrtimer_wakeup() fires,
clears sleeper->task>
futex(LOCK_PI, uaddr2)
__rt_mutex_start_proxy_lock()
try_to_take_rt_mutex() /* steals lock */
rt_mutex_set_owner(lock, stealer)
<preempted>
<scheduled in>
rt_mutex_wait_proxy_lock()
__rt_mutex_slowlock()
try_to_take_rt_mutex() /* fails, lock held by stealer */
if (timeout && !timeout->task)
return -ETIMEDOUT;
fixup_owner()
/* lock wasn't acquired, so,
fixup_pi_state_owner skipped */
return -ETIMEDOUT;
/* At this point, we've returned -ETIMEDOUT to userspace, but the
* futex word shows waiter to be the owner, and the pi_mutex has
* stealer as the owner */
futex_lock(LOCK_PI, uaddr2)
-> bails with EDEADLK, futex word says we're owner.
And suggested that what commit:
73d786bd043e ("futex: Rework inconsistent rt_mutex/futex_q state")
removes from fixup_owner() looks to be just what is needed. And indeed
it is -- I completely missed that requeue_pi could also result in this
case. So we need to restore that, except that subsequent patches, like
commit:
16ffa12d7425 ("futex: Pull rt_mutex_futex_unlock() out from under hb->lock")
changed all the locking rules. Even without that, the sequence:
- if (rt_mutex_futex_trylock(&q->pi_state->pi_mutex)) {
- locked = 1;
- goto out;
- }
- raw_spin_lock_irq(&q->pi_state->pi_mutex.wait_lock);
- owner = rt_mutex_owner(&q->pi_state->pi_mutex);
- if (!owner)
- owner = rt_mutex_next_owner(&q->pi_state->pi_mutex);
- raw_spin_unlock_irq(&q->pi_state->pi_mutex.wait_lock);
- ret = fixup_pi_state_owner(uaddr, q, owner);
already suggests there were races; otherwise we'd never have to look
at next_owner.
So instead of doing 3 consecutive wait_lock sections with who knows
what races, we do it all in a single section. Additionally, the usage
of pi_state->owner in fixup_owner() was only safe because only the
rt_mutex owner would modify it, which this additional case wrecks.
Luckily the values can only change away and not to the value we're
testing, this means we can do a speculative test and double check once
we have the wait_lock.
Fixes: 73d786bd043e ("futex: Rework inconsistent rt_mutex/futex_q state")
Reported-by: Julia Cartwright <julia@ni.com>
Reported-by: Gratian Crisan <gratian.crisan@ni.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Tested-by: Julia Cartwright <julia@ni.com>
Tested-by: Gratian Crisan <gratian.crisan@ni.com>
Cc: Darren Hart <dvhart@infradead.org>
Cc: stable@vger.kernel.org
Link: https://lkml.kernel.org/r/20171208124939.7livp7no2ov65rrc@hirez.programming.kicks-ass.net
2017-12-08 20:49:39 +08:00
|
|
|
ret = __rt_mutex_slowtrylock(lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irqrestore(&lock->wait_lock, flags);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2021-08-16 05:27:57 +08:00
|
|
|
static __always_inline int __rt_mutex_trylock(struct rt_mutex *lock)
|
2021-03-26 23:29:41 +08:00
|
|
|
{
|
2021-08-16 05:27:57 +08:00
|
|
|
if (likely(rt_mutex_cmpxchg_acquire(lock, NULL, current)))
|
|
|
|
return 1;
|
2021-03-26 23:29:41 +08:00
|
|
|
|
2021-08-16 05:27:57 +08:00
|
|
|
return rt_mutex_slowtrylock(lock);
|
2021-03-26 23:29:41 +08:00
|
|
|
}
|
|
|
|
|
2006-06-27 17:54:53 +08:00
|
|
|
/*
|
2015-06-17 16:33:50 +08:00
|
|
|
* Slow path to release a rt-mutex.
|
2006-06-27 17:54:53 +08:00
|
|
|
*/
|
2021-03-26 23:29:41 +08:00
|
|
|
static void __sched rt_mutex_slowunlock(struct rt_mutex *lock)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
2021-03-26 23:29:41 +08:00
|
|
|
DEFINE_WAKE_Q(wake_q);
|
2016-01-13 18:25:38 +08:00
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
/* irqsave required to support early boot calls */
|
|
|
|
raw_spin_lock_irqsave(&lock->wait_lock, flags);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
|
|
|
debug_rt_mutex_unlock(lock);
|
|
|
|
|
2014-06-12 02:44:04 +08:00
|
|
|
/*
|
|
|
|
* We must be careful here if the fast path is enabled. If we
|
|
|
|
* have no waiters queued we cannot set owner to NULL here
|
|
|
|
* because of:
|
|
|
|
*
|
|
|
|
* foo->lock->owner = NULL;
|
|
|
|
* rtmutex_lock(foo->lock); <- fast path
|
|
|
|
* free = atomic_dec_and_test(foo->refcnt);
|
|
|
|
* rtmutex_unlock(foo->lock); <- fast path
|
|
|
|
* if (free)
|
|
|
|
* kfree(foo);
|
|
|
|
* raw_spin_unlock(foo->lock->wait_lock);
|
|
|
|
*
|
|
|
|
* So for the fastpath enabled kernel:
|
|
|
|
*
|
|
|
|
* Nothing can set the waiters bit as long as we hold
|
|
|
|
* lock->wait_lock. So we do the following sequence:
|
|
|
|
*
|
|
|
|
* owner = rt_mutex_owner(lock);
|
|
|
|
* clear_rt_mutex_waiters(lock);
|
|
|
|
* raw_spin_unlock(&lock->wait_lock);
|
|
|
|
* if (cmpxchg(&lock->owner, owner, 0) == owner)
|
|
|
|
* return;
|
|
|
|
* goto retry;
|
|
|
|
*
|
|
|
|
* The fastpath disabled variant is simple as all access to
|
|
|
|
* lock->owner is serialized by lock->wait_lock:
|
|
|
|
*
|
|
|
|
* lock->owner = NULL;
|
|
|
|
* raw_spin_unlock(&lock->wait_lock);
|
|
|
|
*/
|
|
|
|
while (!rt_mutex_has_waiters(lock)) {
|
|
|
|
/* Drops lock->wait_lock ! */
|
2016-01-13 18:25:38 +08:00
|
|
|
if (unlock_rt_mutex_safe(lock, flags) == true)
|
2021-03-26 23:29:41 +08:00
|
|
|
return;
|
2014-06-12 02:44:04 +08:00
|
|
|
/* Relock the rtmutex and try again */
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_lock_irqsave(&lock->wait_lock, flags);
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
|
|
|
|
2014-06-12 02:44:04 +08:00
|
|
|
/*
|
|
|
|
* The wakeup next waiter path does not suffer from the above
|
|
|
|
* race. See the comments there.
|
2015-05-20 01:24:55 +08:00
|
|
|
*
|
|
|
|
* Queue the next waiter for wakeup once we release the wait_lock.
|
2014-06-12 02:44:04 +08:00
|
|
|
*/
|
2021-03-26 23:29:41 +08:00
|
|
|
mark_wakeup_next_waiter(&wake_q, lock);
|
2016-01-13 18:25:38 +08:00
|
|
|
raw_spin_unlock_irqrestore(&lock->wait_lock, flags);
|
2006-06-27 17:54:53 +08:00
|
|
|
|
2021-03-26 23:29:41 +08:00
|
|
|
rt_mutex_postunlock(&wake_q);
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|
|
|
|
|
2021-08-16 05:27:57 +08:00
|
|
|
static __always_inline void __rt_mutex_unlock(struct rt_mutex *lock)
|
2006-06-27 17:54:53 +08:00
|
|
|
{
|
2021-03-26 23:29:41 +08:00
|
|
|
if (likely(rt_mutex_cmpxchg_release(lock, current, NULL)))
|
|
|
|
return;
|
|
|
|
|
|
|
|
rt_mutex_slowunlock(lock);
|
2006-06-27 17:54:53 +08:00
|
|
|
}
|