On Thu, Nov 10, 2005 at 10:04:09AM +0100, Oliver Neukum wrote:
> 
> OK, I see.
> urb->status != kaweth->status
> There is no race of this kind, but you are right, due to usb_kill_urb()
> working also while the callback is running kaweth->status is no longer
> needed, but it doesn't do harm either.

You're absolutely right.  We can remove kaweth->status as well.  However,
before we do that we should add in some code so that the driver doesn't
log scary (but harmless) error messages on shutdown.

In fact even now it is possible to get those printks from the resubmit
function if usb_kill_urb() fails to kill it using unlink.

I'll send you a patch for that tomorrow.

Cheers,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[EMAIL PROTECTED]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt


-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server. Download
it for free - -and be entered to win a 42" plasma tv or your very own
Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to