diff options
author | Henry Gebhardt <gebhardt@astro.uni-tuebingen.de> | 2009-11-04 11:19:28 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-11-17 16:46:33 -0800 |
commit | 18a77b5d237a67d2c621a46f5271a3b51da1b380 (patch) | |
tree | c3b359ae9c655a65ab96e68205fd8d5bf4c6bddb | |
parent | a1f17a872bc7b1cb7efdd5486a2963e88a536e61 (diff) | |
download | op-kernel-dev-18a77b5d237a67d2c621a46f5271a3b51da1b380.zip op-kernel-dev-18a77b5d237a67d2c621a46f5271a3b51da1b380.tar.gz |
USB: cdc_acm: Fix race condition when opening tty
If acm_rx_tasklet() gets called before tty_port_block_til_ready()
returns, then bulk IN urbs may not be sent. This fixes it.
Signed-off-by: Henry Gebhardt <gebhardt@astro.uni-tuebingen.de>
Acked-by: Oliver Neukum <oliver@neukum.org>
Cc: stable <stable@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/usb/class/cdc-acm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c index e3861b2..b72fa49 100644 --- a/drivers/usb/class/cdc-acm.c +++ b/drivers/usb/class/cdc-acm.c @@ -609,9 +609,9 @@ static int acm_tty_open(struct tty_struct *tty, struct file *filp) acm->throttle = 0; - tasklet_schedule(&acm->urb_task); set_bit(ASYNCB_INITIALIZED, &acm->port.flags); rv = tty_port_block_til_ready(&acm->port, tty, filp); + tasklet_schedule(&acm->urb_task); done: mutex_unlock(&acm->mutex); err_out: |