Hi Andrew,
If someone tries to _urb_unlink while _urb_queue_head is running,
he'll see
_urb->queue == NULL and fail to do any locking. Prevent that from
happening
by strategically placed barriers.
let me repeat this, the hci_usb driver is not worth fixing. Doing our
own URB handling is a bad idea. The btusb driver should fix all of
this. Only exception is that it is missing all the quirks, but that
was me being lazy.
ok... But as long as the old code is buildable and installable, we
should
fix bugs in it?
I am okay with it and happy to accept any fixes, but to be quite
honest, that this driver still works is in some cases pure luck. Doing
the URB handling by ourself is really simply plain work. No excuses
here and parts of it is my fault. I know that. Hence I started a new
implementation from scratch.
Regards
Marcel
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
