Re: [linux-usb-devel] urb_dequeue failing

2003-01-10 Thread David Brownell
Oliver Neukum wrote: retval = hcd->driver->urb_dequeue (hcd, urb); /* hcds shouldn't really fail these calls, but... */ if (retval) { ... } is deeply racy. It can change the context under a running completion handler. I'd suggest that you declare urb_dequeue void and remove it. May

[linux-usb-devel] urb_dequeue failing

2003-01-10 Thread Oliver Neukum
Hi, this code from hcd.c: retval = hcd->driver->urb_dequeue (hcd, urb); /* hcds shouldn't really fail these calls, but... */ if (retval) { dev_dbg (sys, "dequeue %p --> %d\n", urb, retval); if (!(urb-