diff options
| author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2006-09-25 06:51:41 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-09-28 18:36:43 -0400 |
| commit | e81ee637e4aed723f71007c90a901268317ed6d6 (patch) | |
| tree | 25fec34b4577eb36b52b8e4e45a191138ac49404 /drivers/usb/serial/safe_serial.c | |
| parent | 2e3a43f0b6f16705ec76d3744b82a116965ebebe (diff) | |
usb-serial: possible irq lock inversion (PPP vs. usb/serial)
=========================================================
[ INFO: possible irq lock inversion dependency detected ]
Diffstat (limited to 'drivers/usb/serial/safe_serial.c')
| -rw-r--r-- | drivers/usb/serial/safe_serial.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/usb/serial/safe_serial.c b/drivers/usb/serial/safe_serial.c index 789771ecdb1..1e07dfad685 100644 --- a/drivers/usb/serial/safe_serial.c +++ b/drivers/usb/serial/safe_serial.c | |||
| @@ -298,14 +298,14 @@ static int safe_write (struct usb_serial_port *port, const unsigned char *buf, i | |||
| 298 | dbg ("%s - write request of 0 bytes", __FUNCTION__); | 298 | dbg ("%s - write request of 0 bytes", __FUNCTION__); |
| 299 | return (0); | 299 | return (0); |
| 300 | } | 300 | } |
| 301 | spin_lock(&port->lock); | 301 | spin_lock_bh(&port->lock); |
| 302 | if (port->write_urb_busy) { | 302 | if (port->write_urb_busy) { |
| 303 | spin_unlock(&port->lock); | 303 | spin_unlock_bh(&port->lock); |
| 304 | dbg("%s - already writing", __FUNCTION__); | 304 | dbg("%s - already writing", __FUNCTION__); |
| 305 | return 0; | 305 | return 0; |
| 306 | } | 306 | } |
| 307 | port->write_urb_busy = 1; | 307 | port->write_urb_busy = 1; |
| 308 | spin_unlock(&port->lock); | 308 | spin_unlock_bh(&port->lock); |
| 309 | 309 | ||
| 310 | packet_length = port->bulk_out_size; // get max packetsize | 310 | packet_length = port->bulk_out_size; // get max packetsize |
| 311 | 311 | ||