aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPeter Zijlstra <peterz@infradead.org>2017-03-22 06:35:56 -0400
committerThomas Gleixner <tglx@linutronix.de>2017-03-23 14:10:09 -0400
commit50809358dd7199aa7ce232f6877dd09ec30ef374 (patch)
treea8f847cba791b1f9585330789fa230e7abfb762f
parent16ffa12d742534d4ff73e8b3a4e81c1de39196f0 (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.c5
-rw-r--r--kernel/locking/rtmutex.c12
-rw-r--r--kernel/locking/rtmutex_common.h1
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
1156void 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);
104extern void rt_mutex_proxy_unlock(struct rt_mutex *lock, 104extern void rt_mutex_proxy_unlock(struct rt_mutex *lock,
105 struct task_struct *proxy_owner); 105 struct task_struct *proxy_owner);
106extern void rt_mutex_init_waiter(struct rt_mutex_waiter *waiter);
106extern int rt_mutex_start_proxy_lock(struct rt_mutex *lock, 107extern 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);