diff options
author | Peter Zijlstra <peterz@infradead.org> | 2017-03-22 06:35:56 -0400 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2017-03-23 14:10:09 -0400 |
commit | 50809358dd7199aa7ce232f6877dd09ec30ef374 (patch) | |
tree | a8f847cba791b1f9585330789fa230e7abfb762f | |
parent | 16ffa12d742534d4ff73e8b3a4e81c1de39196f0 (diff) |
futex,rt_mutex: Introduce rt_mutex_init_waiter()
Since there's already two copies of this code, introduce a helper now
before adding a third one.
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: juri.lelli@arm.com
Cc: bigeasy@linutronix.de
Cc: xlpang@redhat.com
Cc: rostedt@goodmis.org
Cc: mathieu.desnoyers@efficios.com
Cc: jdesfossez@efficios.com
Cc: dvhart@infradead.org
Cc: bristot@redhat.com
Link: http://lkml.kernel.org/r/20170322104151.950039479@infradead.org
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
-rw-r--r-- | kernel/futex.c | 5 | ||||
-rw-r--r-- | kernel/locking/rtmutex.c | 12 | ||||
-rw-r--r-- | kernel/locking/rtmutex_common.h | 1 |
3 files changed, 11 insertions, 7 deletions
diff --git a/kernel/futex.c b/kernel/futex.c index 3b0aace334a8..f03ff63326d7 100644 --- a/kernel/futex.c +++ b/kernel/futex.c | |||
@@ -2956,10 +2956,7 @@ static int futex_wait_requeue_pi(u32 __user *uaddr, unsigned int flags, | |||
2956 | * The waiter is allocated on our stack, manipulated by the requeue | 2956 | * The waiter is allocated on our stack, manipulated by the requeue |
2957 | * code while we sleep on uaddr. | 2957 | * code while we sleep on uaddr. |
2958 | */ | 2958 | */ |
2959 | debug_rt_mutex_init_waiter(&rt_waiter); | 2959 | rt_mutex_init_waiter(&rt_waiter); |
2960 | RB_CLEAR_NODE(&rt_waiter.pi_tree_entry); | ||
2961 | RB_CLEAR_NODE(&rt_waiter.tree_entry); | ||
2962 | rt_waiter.task = NULL; | ||
2963 | 2960 | ||
2964 | ret = get_futex_key(uaddr2, flags & FLAGS_SHARED, &key2, VERIFY_WRITE); | 2961 | ret = get_futex_key(uaddr2, flags & FLAGS_SHARED, &key2, VERIFY_WRITE); |
2965 | if (unlikely(ret != 0)) | 2962 | if (unlikely(ret != 0)) |
diff --git a/kernel/locking/rtmutex.c b/kernel/locking/rtmutex.c index 7d63bc5dd9b2..d2fe4b472b13 100644 --- a/kernel/locking/rtmutex.c +++ b/kernel/locking/rtmutex.c | |||
@@ -1153,6 +1153,14 @@ void rt_mutex_adjust_pi(struct task_struct *task) | |||
1153 | next_lock, NULL, task); | 1153 | next_lock, NULL, task); |
1154 | } | 1154 | } |
1155 | 1155 | ||
1156 | void rt_mutex_init_waiter(struct rt_mutex_waiter *waiter) | ||
1157 | { | ||
1158 | debug_rt_mutex_init_waiter(waiter); | ||
1159 | RB_CLEAR_NODE(&waiter->pi_tree_entry); | ||
1160 | RB_CLEAR_NODE(&waiter->tree_entry); | ||
1161 | waiter->task = NULL; | ||
1162 | } | ||
1163 | |||
1156 | /** | 1164 | /** |
1157 | * __rt_mutex_slowlock() - Perform the wait-wake-try-to-take loop | 1165 | * __rt_mutex_slowlock() - Perform the wait-wake-try-to-take loop |
1158 | * @lock: the rt_mutex to take | 1166 | * @lock: the rt_mutex to take |
@@ -1235,9 +1243,7 @@ rt_mutex_slowlock(struct rt_mutex *lock, int state, | |||
1235 | unsigned long flags; | 1243 | unsigned long flags; |
1236 | int ret = 0; | 1244 | int ret = 0; |
1237 | 1245 | ||
1238 | debug_rt_mutex_init_waiter(&waiter); | 1246 | rt_mutex_init_waiter(&waiter); |
1239 | RB_CLEAR_NODE(&waiter.pi_tree_entry); | ||
1240 | RB_CLEAR_NODE(&waiter.tree_entry); | ||
1241 | 1247 | ||
1242 | /* | 1248 | /* |
1243 | * Technically we could use raw_spin_[un]lock_irq() here, but this can | 1249 | * Technically we could use raw_spin_[un]lock_irq() here, but this can |
diff --git a/kernel/locking/rtmutex_common.h b/kernel/locking/rtmutex_common.h index af667f61ebcb..10f57d688f17 100644 --- a/kernel/locking/rtmutex_common.h +++ b/kernel/locking/rtmutex_common.h | |||
@@ -103,6 +103,7 @@ extern void rt_mutex_init_proxy_locked(struct rt_mutex *lock, | |||
103 | struct task_struct *proxy_owner); | 103 | struct task_struct *proxy_owner); |
104 | extern void rt_mutex_proxy_unlock(struct rt_mutex *lock, | 104 | extern void rt_mutex_proxy_unlock(struct rt_mutex *lock, |
105 | struct task_struct *proxy_owner); | 105 | struct task_struct *proxy_owner); |
106 | extern void rt_mutex_init_waiter(struct rt_mutex_waiter *waiter); | ||
106 | extern int rt_mutex_start_proxy_lock(struct rt_mutex *lock, | 107 | extern int rt_mutex_start_proxy_lock(struct rt_mutex *lock, |
107 | struct rt_mutex_waiter *waiter, | 108 | struct rt_mutex_waiter *waiter, |
108 | struct task_struct *task); | 109 | struct task_struct *task); |