diff options
| author | 2006-06-15 20:11:15 +0400 | |
|---|---|---|
| committer | 2006-06-17 10:52:13 -0700 | |
| commit | 8f17fc20bfb75bcec4cfeda789738979c8338fdc (patch) | |
| tree | ec84c5222de58b4d26507c892d0b8f828a6dce7e /fs/jbd/commit.c | |
| parent | [PATCH] sky2: netconsole suspend/resume interaction (diff) | |
| download | wireguard-linux-8f17fc20bfb75bcec4cfeda789738979c8338fdc.tar.xz wireguard-linux-8f17fc20bfb75bcec4cfeda789738979c8338fdc.zip | |
[PATCH] check_process_timers: fix possible lockup
If the local timer interrupt happens just after do_exit() sets PF_EXITING
(and before it clears ->it_xxx_expires) run_posix_cpu_timers() will call
check_process_timers() with tasklist_lock + ->siglock held and
check_process_timers:
t = tsk;
do {
....
do {
t = next_thread(t);
} while (unlikely(t->flags & PF_EXITING));
} while (t != tsk);
the outer loop will never stop.
Actually, the window is bigger. Another process can attach the timer
after ->it_xxx_expires was cleared (see the next commit) and the 'if
(PF_EXITING)' check in arm_timer() is racy (see the one after that).
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/jbd/commit.c')
0 files changed, 0 insertions, 0 deletions
