diff options
| author | Alan Cox <alan@linux.intel.com> | 2009-11-30 13:17:46 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-12-11 15:18:07 -0800 |
| commit | 2493c0c166565e36831196446af594eb07892daf (patch) | |
| tree | 0175b121f39076dbd858fd7e475885e9d804a5f4 /include | |
| parent | 07f86c03fec711692121bf7f5316e0b3426acc05 (diff) | |
| download | op-kernel-dev-2493c0c166565e36831196446af594eb07892daf.zip op-kernel-dev-2493c0c166565e36831196446af594eb07892daf.tar.gz | |
tty: isicom: fix deadlock on shutdown
Alexander Strakh <strakh@ispras.ru> reported
KERNEL_VERSION: 2.6.31
DESCRIBE:
Driver drivers/char/isicom.c might sleep in atomic context, because it
calls
tty_port_xmit_buf under spin_lock.
./drivers/char/isicom.c:
1307 static void isicom_hangup(struct tty_struct *tty)
1308 {
...
1315 spin_lock_irqsave(&port->card->card_lock, flags);
1316 isicom_shutdown_port(port);
...
Path to might_sleep macro from isicom_hangup:
1. isicom_hangup calls spin_lock_irqsave (drivers/char/isicom.c:1315) and
then
calls isicom_shutdown_port.
2. isiscom_shutdown_port calls tty_port_free_xmit_buf at
drivers/char/isicom.c:906
3. tty_port_free_xmit_buf calls mutex_lock at drivers/char/tty_port:48
Found by Linux Driver Verification Project.
Reported-by: Alexander Strakh <strakh@ispras.ru>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions
