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/ir-usb.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/ir-usb.c')
| -rw-r--r-- | drivers/usb/serial/ir-usb.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/usb/serial/ir-usb.c b/drivers/usb/serial/ir-usb.c index 1738b0b6a376..1b348df388ed 100644 --- a/drivers/usb/serial/ir-usb.c +++ b/drivers/usb/serial/ir-usb.c | |||
| @@ -342,14 +342,14 @@ static int ir_write (struct usb_serial_port *port, const unsigned char *buf, int | |||
| 342 | if (count == 0) | 342 | if (count == 0) |
| 343 | return 0; | 343 | return 0; |
| 344 | 344 | ||
| 345 | spin_lock(&port->lock); | 345 | spin_lock_bh(&port->lock); |
| 346 | if (port->write_urb_busy) { | 346 | if (port->write_urb_busy) { |
| 347 | spin_unlock(&port->lock); | 347 | spin_unlock_bh(&port->lock); |
| 348 | dbg("%s - already writing", __FUNCTION__); | 348 | dbg("%s - already writing", __FUNCTION__); |
| 349 | return 0; | 349 | return 0; |
| 350 | } | 350 | } |
| 351 | port->write_urb_busy = 1; | 351 | port->write_urb_busy = 1; |
| 352 | spin_unlock(&port->lock); | 352 | spin_unlock_bh(&port->lock); |
| 353 | 353 | ||
| 354 | transfer_buffer = port->write_urb->transfer_buffer; | 354 | transfer_buffer = port->write_urb->transfer_buffer; |
| 355 | transfer_size = min(count, port->bulk_out_size - 1); | 355 | transfer_size = min(count, port->bulk_out_size - 1); |