ChangeSet 1.1043.1.12, 2003/02/17 10:09:22-08:00, [EMAIL PROTECTED]
[PATCH] USB speedtouch: Fix speedtouch maxi race
Instead of trying to close all ATM connections in the USB disconnect routine,
just notify the ATM layer that the ATM device should go down when the last
connection is closed. This should be the last big speedtouch stability fix.
diff -Nru a/drivers/usb/misc/speedtouch.c b/drivers/usb/misc/speedtouch.c
--- a/drivers/usb/misc/speedtouch.c Tue Feb 18 16:40:35 2003
+++ b/drivers/usb/misc/speedtouch.c Tue Feb 18 16:40:35 2003
@@ -202,6 +202,7 @@
* atm driver prototypes and stuctures
*/
+static void udsl_atm_dev_close (struct atm_dev *dev);
static int udsl_atm_open (struct atm_vcc *vcc, short vpi, int vci);
static void udsl_atm_close (struct atm_vcc *vcc);
static int udsl_atm_ioctl (struct atm_dev *dev, unsigned int cmd, void *arg);
@@ -209,6 +210,7 @@
static int udsl_atm_proc_read (struct atm_dev *atm_dev, loff_t *pos, char *page);
static struct atmdev_ops udsl_atm_devops = {
+ .dev_close = udsl_atm_dev_close,
.open = udsl_atm_open,
.close = udsl_atm_close,
.ioctl = udsl_atm_ioctl,
@@ -691,24 +693,21 @@
*
****************************************************************************/
-static void udsl_atm_stopdevice (struct udsl_instance_data *instance)
+static void udsl_atm_dev_close (struct atm_dev *dev)
{
- struct atm_vcc *walk;
- struct atm_dev *atm_dev;
+ struct udsl_instance_data *instance = dev->dev_data;
- if (!instance->atm_dev)
+ if (!instance) {
+ PDEBUG ("udsl_atm_dev_close: NULL instance!\n");
return;
+ }
- atm_dev = instance->atm_dev;
-
- atm_dev->signal = ATM_PHY_SIG_LOST;
- walk = atm_dev->vccs;
- shutdown_atm_dev (atm_dev);
-
- for (; walk; walk = walk->next)
- wake_up (&walk->sleep);
+ PDEBUG ("udsl_atm_dev_close: queue has %u elements\n",
+instance->sndqueue.qlen);
- instance->atm_dev = NULL;
+ PDEBUG ("udsl_atm_dev_close: killing tasklet\n");
+ tasklet_kill (&instance->receive_tasklet);
+ PDEBUG ("udsl_atm_dev_close: freeing instance\n");
+ kfree (instance);
}
@@ -1038,7 +1037,7 @@
tasklet_disable (&instance->receive_tasklet);
- /* flush spare receivers */
+ /* receive finalize */
down (&instance->serialize); /* vs udsl_fire_receivers */
/* no need to take the spinlock */
list_for_each (pos, &instance->spare_receivers)
@@ -1088,8 +1087,7 @@
kfree_skb (rcv->skb);
}
- udsl_atm_stopdevice (instance);
-
+ /* send finalize */
tasklet_disable (&instance->send_tasklet);
for (i = 0; i < UDSL_NUMBER_SND_URBS; i++)
@@ -1120,7 +1118,6 @@
instance->current_buffer = NULL;
tasklet_enable (&instance->receive_tasklet);
- tasklet_kill (&instance->receive_tasklet);
PDEBUG ("udsl_usb_disconnect: freeing senders\n");
for (i = 0; i < UDSL_NUMBER_SND_URBS; i++)
@@ -1130,8 +1127,8 @@
for (i = 0; i < UDSL_NUMBER_SND_BUFS; i++)
kfree (instance->all_buffers[i].base);
- PDEBUG ("udsl_usb_disconnect: freeing instance\n");
- kfree (instance);
+ /* atm finalize */
+ shutdown_atm_dev (instance->atm_dev);
}
-------------------------------------------------------
This SF.net email is sponsored by: SlickEdit Inc. Develop an edge.
The most comprehensive and flexible code editor you can use.
Code faster. C/C++, C#, Java, HTML, XML, many more. FREE 30-Day Trial.
www.slickedit.com/sourceforge
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
