Skip to content

Commit

Permalink
futex: fix restart for early wakeup in futex_wait_requeue_pi()
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Thomas Gleixner committed May 20, 2009
1 parent c8b15a7 commit 1c840c1
Showing 1 changed file with 3 additions and 10 deletions.
13 changes: 3 additions & 10 deletions kernel/futex.c
Original file line number Diff line number Diff line change
Expand Up @@ -2060,7 +2060,7 @@ static int futex_unlock_pi(u32 __user *uaddr, int fshared)
*
* Returns
* 0 - no early wakeup detected
* <0 - -ETIMEDOUT or -ERESTARTSYS (FIXME: or ERESTARTNOINTR?)
* <0 - -ETIMEDOUT or -ERESTARTNOINTR
*/
static inline
int handle_early_requeue_pi_wakeup(struct futex_hash_bucket *hb,
Expand All @@ -2087,15 +2087,8 @@ int handle_early_requeue_pi_wakeup(struct futex_hash_bucket *hb,

if (timeout && !timeout->task)
ret = -ETIMEDOUT;
else {
/*
* We expect signal_pending(current), but another
* thread may have handled it for us already.
*/
/* FIXME: ERESTARTSYS or ERESTARTNOINTR? Do we care if
* the user specified SA_RESTART or not? */
ret = -ERESTARTSYS;
}
else
ret = -ERESTARTNOINTR;
}
return ret;
}
Expand Down

0 comments on commit 1c840c1

Please sign in to comment.