diff options
| author | Petr Vandrovec <vandrove@vc.cvut.cz> | 2003-05-09 05:07:44 -0700 |
|---|---|---|
| committer | Petr Vandrovec <vandrove@vc.cvut.cz> | 2003-05-09 05:07:44 -0700 |
| commit | b36c92e78f12c942cf1d8b5e60003da7281749a2 (patch) | |
| tree | 7657f9304e0ff7a819144a3d65b5625cbf44e0b8 /include/linux | |
| parent | 55f3c6b7f64b96b5826f513a23b7d75da69bdd7a (diff) | |
[PATCH] Fix potential runqueue deadlock
send_sig_info() has been broken since 2.5.60.
The function can be invoked from a the time interrupt (timer_interrpt ->
do_timer -> update_process_times -> -> update_one_process -> (
do_process_times, do_it_prof, do_it_virt ) -> -> send_sig ->
send_sig_info) but it uses spin_unlock_irq instead of the correct
spin_unlock_irqrestore.
This enables interrupts, and later scheduler_tick() locks runqueue
(without disabling interrupts). And if we are unlucky, a new interrupt
comes at this point. And if this interrupt tries to do wake_up() (like
RTC interrupt does), we will deadlock on runqueue lock :-(
The bug was introduced by signal-fixes-2.5.59-A4, which split the
original send_sig_info into two functions, and in one branch it started
using these unsafe spinlock variants (while the "group" variant uses
irqsave/restore correctly).
Diffstat (limited to 'include/linux')
0 files changed, 0 insertions, 0 deletions
