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/generic.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/generic.c')
| -rw-r--r-- | drivers/usb/serial/generic.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/usb/serial/generic.c b/drivers/usb/serial/generic.c index 172713556393..21cbaa0fb96b 100644 --- a/drivers/usb/serial/generic.c +++ b/drivers/usb/serial/generic.c | |||
| @@ -175,14 +175,14 @@ int usb_serial_generic_write(struct usb_serial_port *port, const unsigned char * | |||
| 175 | 175 | ||
| 176 | /* only do something if we have a bulk out endpoint */ | 176 | /* only do something if we have a bulk out endpoint */ |
| 177 | if (serial->num_bulk_out) { | 177 | if (serial->num_bulk_out) { |
| 178 | spin_lock(&port->lock); | 178 | spin_lock_bh(&port->lock); |
| 179 | if (port->write_urb_busy) { | 179 | if (port->write_urb_busy) { |
| 180 | spin_unlock(&port->lock); | 180 | spin_unlock_bh(&port->lock); |
| 181 | dbg("%s - already writing", __FUNCTION__); | 181 | dbg("%s - already writing", __FUNCTION__); |
| 182 | return 0; | 182 | return 0; |
| 183 | } | 183 | } |
| 184 | port->write_urb_busy = 1; | 184 | port->write_urb_busy = 1; |
| 185 | spin_unlock(&port->lock); | 185 | spin_unlock_bh(&port->lock); |
| 186 | 186 | ||
| 187 | count = (count > port->bulk_out_size) ? port->bulk_out_size : count; | 187 | count = (count > port->bulk_out_size) ? port->bulk_out_size : count; |
| 188 | 188 | ||