Re: usb-serial ipaq kernel problem
From: Luiz Fernando N. Capitulino
Date: Mon May 29 2006 - 13:17:26 EST
On Mon, 29 May 2006 13:25:53 -0300
"Luiz Fernando N. Capitulino" <lcapitulino@xxxxxxxxxxxxxxx> wrote:
| On Mon, 29 May 2006 12:01:02 -0300
| "Luiz Fernando N. Capitulino" <lcapitulino@xxxxxxxxxxxxxxx> wrote:
|
| |
| | Hi Pete,
| |
| | On Fri, 26 May 2006 13:34:10 -0700
| | Pete Zaitcev <zaitcev@xxxxxxxxxx> wrote:
| |
| | | On Fri, 26 May 2006 20:22:17 +0200, Frank Gevaerts <frank.gevaerts@xxxxxx> 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 <lcapitulino@xxxxxxxxxxxxxxx>
---
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
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/