diff options
author | Thomas Gleixner <tglx@linutronix.de> | 2009-05-20 03:22:40 -0400 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2009-05-20 04:28:45 -0400 |
commit | 1c840c14906d4ddf66c1f4f5daea059aad951c82 (patch) | |
tree | 3c71d3d67c590df598098f5cdb79151359ff2ce4 /kernel/futex.c | |
parent | c8b15a706d921baed3195407e4f55270112bb3c6 (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.c | 13 |
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 | */ |
2065 | static inline | 2065 | static inline |
2066 | int handle_early_requeue_pi_wakeup(struct futex_hash_bucket *hb, | 2066 | int 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 | } |