On Mon, 5 Nov 2012, Christof Meerwald wrote: > BTW, I have been able to reproduce the problem on a completely > different machine (also running Ubuntu 12.10, but different hardware). > The important thing appears to be that the USB audio device is > connected via a USB 2.0 hub (and then using the test code posted in > http://pastebin.com/aHGe1S1X specifying the audio device as > "plughw:Set" (or whatever it's called) seems to trigger the freeze). Christof: Thank you for that reference, it was a big help. After crashing my system many times I have tracked the problem, at least in part. The patch below should prevent your system from freezing. Takashi: It turns out the the problem is triggered when the audio subsystem calls snd_usb_endpoint_stop() with wait == 0 and then calls snd_usb_endpoint_start(). Since the driver doesn't wait for the outstanding URBs to finish, it tries to submit them again while they are still active. Normally the USB core would realize this and fail the submission, but a bug in ehci-hcd prevented this from happening. (That bug is what the patch below fixes.) The URB gets added to the active list twice, resulting in list corruption and an oops in interrupt context, which freezes the system. The user program that triggers the problem basically looks like this: snd_pcm_prepare(rec_pcm); snd_pcm_start(rec_pcm); snd_pcm_drop(rec_pcm); snd_pcm_prepare(rec_pcm); snd_pcm_start(rec_pcm); The snd_pcm_drop call unlinks the URBs but does not wait for them to finish. Then the second snd_pcm_start call submits the URBs before they have finished. What is the right solution for this problem? Alan Stern Index: usb-3.7/drivers/usb/host/ehci-sched.c =================================================================== --- usb-3.7.orig/drivers/usb/host/ehci-sched.c +++ usb-3.7/drivers/usb/host/ehci-sched.c @@ -1632,7 +1632,7 @@ static void itd_link_urb( /* don't need that schedule data any more */ iso_sched_free (stream, iso_sched); - urb->hcpriv = NULL; + urb->hcpriv = stream; ++ehci->isoc_count; enable_periodic(ehci); @@ -2031,7 +2031,7 @@ static void sitd_link_urb( /* don't need that schedule data any more */ iso_sched_free (stream, sched); - urb->hcpriv = NULL; + urb->hcpriv = stream; ++ehci->isoc_count; enable_periodic(ehci); -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
