diff options
author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2008-02-01 11:45:13 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-02-01 11:45:13 -0500 |
commit | 3588a085cd52ef080bf72df772378e1ba6bb292f (patch) | |
tree | 9c013c37431eb3a0d0da3ab14cbc4985600e82a2 | |
parent | 24e1c13c93cbdd05e4b7ea921c0050b036555adc (diff) |
hrtimer: fix hrtimer_init_sleeper() users
this patch:
commit 37bb6cb4097e29ffee970065b74499cbf10603a3
Author: Peter Zijlstra <a.p.zijlstra@chello.nl>
Date: Fri Jan 25 21:08:32 2008 +0100
hrtimer: unlock hrtimer_wakeup
Broke hrtimer_init_sleeper() users. It forgot to fix up the futex
caller of this function to detect the failed queueing and messed up
the do_nanosleep() caller in that it could leak a TASK_INTERRUPTIBLE
state.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/futex.c | 2 | ||||
-rw-r--r-- | kernel/hrtimer.c | 2 |
2 files changed, 4 insertions, 0 deletions
diff --git a/kernel/futex.c b/kernel/futex.c index db9824de8bf0..0edd314798c0 100644 --- a/kernel/futex.c +++ b/kernel/futex.c | |||
@@ -1252,6 +1252,8 @@ static int futex_wait(u32 __user *uaddr, struct rw_semaphore *fshared, | |||
1252 | t.timer.expires = *abs_time; | 1252 | t.timer.expires = *abs_time; |
1253 | 1253 | ||
1254 | hrtimer_start(&t.timer, t.timer.expires, HRTIMER_MODE_ABS); | 1254 | hrtimer_start(&t.timer, t.timer.expires, HRTIMER_MODE_ABS); |
1255 | if (!hrtimer_active(&t.timer)) | ||
1256 | t.task = NULL; | ||
1255 | 1257 | ||
1256 | /* | 1258 | /* |
1257 | * the timer could have already expired, in which | 1259 | * the timer could have already expired, in which |
diff --git a/kernel/hrtimer.c b/kernel/hrtimer.c index bd5d6b5060bc..1069998fe25f 100644 --- a/kernel/hrtimer.c +++ b/kernel/hrtimer.c | |||
@@ -1315,6 +1315,8 @@ static int __sched do_nanosleep(struct hrtimer_sleeper *t, enum hrtimer_mode mod | |||
1315 | 1315 | ||
1316 | } while (t->task && !signal_pending(current)); | 1316 | } while (t->task && !signal_pending(current)); |
1317 | 1317 | ||
1318 | __set_current_state(TASK_RUNNING); | ||
1319 | |||
1318 | return t->task == NULL; | 1320 | return t->task == NULL; |
1319 | } | 1321 | } |
1320 | 1322 | ||