aboutsummaryrefslogtreecommitdiffstats
path: root/kernel/futex.c
diff options
context:
space:
mode:
authorThomas Gleixner <tglx@linutronix.de>2009-05-20 03:22:40 -0400
committerThomas Gleixner <tglx@linutronix.de>2009-05-20 04:28:45 -0400
commit1c840c14906d4ddf66c1f4f5daea059aad951c82 (patch)
tree3c71d3d67c590df598098f5cdb79151359ff2ce4 /kernel/futex.c
parentc8b15a706d921baed3195407e4f55270112bb3c6 (diff)
futex: fix restart for early wakeup in futex_wait_requeue_pi()
The futex_wait_requeue_pi op should restart unconditionally like futex_lock_pi. The user of that function e.g. pthread_cond_wait can not be interrupted so we do not care about the SA_RESTART flag of the signal. Clean up the FIXMEs. Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'kernel/futex.c')
-rw-r--r--kernel/futex.c13
1 files changed, 3 insertions, 10 deletions
diff --git a/kernel/futex.c b/kernel/futex.c
index 381125a9f1e0..2aa216e5b594 100644
--- a/kernel/futex.c
+++ b/kernel/futex.c
@@ -2060,7 +2060,7 @@ pi_faulted:
2060 * 2060 *
2061 * Returns 2061 * Returns
2062 * 0 - no early wakeup detected 2062 * 0 - no early wakeup detected
2063 * <0 - -ETIMEDOUT or -ERESTARTSYS (FIXME: or ERESTARTNOINTR?) 2063 * <0 - -ETIMEDOUT or -ERESTARTNOINTR
2064 */ 2064 */
2065static inline 2065static inline
2066int handle_early_requeue_pi_wakeup(struct futex_hash_bucket *hb, 2066int handle_early_requeue_pi_wakeup(struct futex_hash_bucket *hb,
@@ -2087,15 +2087,8 @@ int handle_early_requeue_pi_wakeup(struct futex_hash_bucket *hb,
2087 2087
2088 if (timeout && !timeout->task) 2088 if (timeout && !timeout->task)
2089 ret = -ETIMEDOUT; 2089 ret = -ETIMEDOUT;
2090 else { 2090 else
2091 /* 2091 ret = -ERESTARTNOINTR;
2092 * We expect signal_pending(current), but another
2093 * thread may have handled it for us already.
2094 */
2095 /* FIXME: ERESTARTSYS or ERESTARTNOINTR? Do we care if
2096 * the user specified SA_RESTART or not? */
2097 ret = -ERESTARTSYS;
2098 }
2099 } 2092 }
2100 return ret; 2093 return ret;
2101} 2094}