On Mon, 29 May 2006 13:25:53 -0300
"Luiz Fernando N. Capitulino" <[EMAIL PROTECTED]> wrote:
| On Mon, 29 May 2006 12:01:02 -0300
| "Luiz Fernando N. Capitulino" <[EMAIL PROTECTED]> wrote:
|
| |
| | Hi Pete,
| |
| | On Fri, 26 May 2006 13:34:10 -0700
| | Pete Zaitcev <[EMAIL PROTECTED]> wrote:
| |
| | | On Fri, 26 May 2006 20:22:17 +0200, Frank Gevaerts <[EMAIL PROTECTED]>
wrote:
| | |
| | | > usb 1-4.5.7: USB disconnect, address 79
| | | > ------------[ cut here ]------------
| | | > kernel BUG at kernel/workqueue.c:110!
| | |
| | | Please let me know if this helps:
| | |
| | | --- linux-2.6.17-rc2/drivers/usb/serial/usb-serial.c 2006-04-23
21:06:18.000000000 -0700
| | | +++ linux-2.6.17-rc2-lem/drivers/usb/serial/usb-serial.c 2006-05-22
21:23:29.000000000 -0700
| | | @@ -162,6 +162,8 @@ static void destroy_serial(struct kref *
| | | }
| | | }
| | |
| | | + flush_scheduled_work(); /* port->work */
| | | +
| | | usb_put_dev(serial->dev);
| | |
| | | /* free up any memory that we allocated */
| |
| | IIUC, the problem occurred before the call to destroy_serial(),
| | otherwise it should be in the backtrace.
| |
| | It seems that 'port->work' is becoming NULL when the device is
| | disconnected, but the ipaq_write_bulk_callback() is executing after
| | that.
|
| Err, I meant 'port->work->entry' is empty, of course.
Frank, could you try this one please?
I have no sure whether this makes sense, but every USB-Serial driver
I know exits in the write URB callback if the URB got an error.
-------------------
usbserial: ipaq: Don't handle URB on errors.
ipaq_write_bulk_callback() should exit if the URB got an error, otherwise
we can get weird problems.
Signed-off-by: Luiz Fernando N. Capitulino <[EMAIL PROTECTED]>
---
drivers/usb/serial/ipaq.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
6ad106187344769a4722f9e6d6f265529844d568
diff --git a/drivers/usb/serial/ipaq.c b/drivers/usb/serial/ipaq.c
index 9a5c979..d263a5e 100644
--- a/drivers/usb/serial/ipaq.c
+++ b/drivers/usb/serial/ipaq.c
@@ -855,6 +855,7 @@ static void ipaq_write_bulk_callback(str
if (urb->status) {
dbg("%s - nonzero write bulk status received: %d",
__FUNCTION__, urb->status);
+ return;
}
spin_lock_irqsave(&write_list_lock, flags);
--
1.3.2
--
Luiz Fernando N. Capitulino
-------------------------------------------------------
All the advantages of Linux Managed Hosting--Without the Cost and Risk!
Fully trained technicians. The highest number of Red Hat certifications in
the hosting industry. Fanatical Support. Click to learn more
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=107521&bid=248729&dat=121642
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
