On Wed, 28 Sep 2011, Pitt, Jason N wrote:
> For some reason that patch didn't work, but I manually made your
> changes to the code and it seems to be working fine. Here's the
> dmesg output after the 12 camera test
Hmmm. Clearly we are seeing the point where the bug first hits, but
there isn't enough information to tell exactly what's going wrong.
Try this patch instead of the previous one; it will print out more of
the important values.
You don't need to send all of the debugging output from the dmesg log.
Just the first 20 lines or so will be plenty.
Alan Stern
Index: usb-3.1/drivers/usb/host/ehci-sched.c
===================================================================
--- usb-3.1.orig/drivers/usb/host/ehci-sched.c
+++ usb-3.1/drivers/usb/host/ehci-sched.c
@@ -2265,6 +2265,8 @@ scan_periodic (struct ehci_hcd *ehci)
{
unsigned now_uframe, frame, clock, clock_frame, mod;
unsigned modified;
+unsigned clock0, uframe0;
+int oclock;
mod = ehci->periodic_size << 3;
@@ -2288,6 +2290,9 @@ scan_periodic (struct ehci_hcd *ehci)
clock &= mod - 1;
clock_frame = clock >> 3;
++ehci->periodic_stamp;
+clock0 = clock;
+uframe0 = now_uframe;
+oclock = -1;
for (;;) {
union ehci_shadow q, *q_p;
@@ -2373,6 +2378,14 @@ restart:
*hw_p = ehci->dummy->qh_dma;
type = Q_NEXT_TYPE(ehci, q.itd->hw_next);
wmb();
+
+if (q.itd->hw_transaction[7] & ITD_ACTIVE(ehci)) {
+ ehci_info(ehci, "Bad iTD frame %u index0 %u startf %d\n",
+ frame, q.itd->index[0], q.itd->urb->start_frame);
+ ehci_info(ehci, " clockuf %u nowuf %u oclock %d clock0 %u uframe0 %u\n",
+ clock, now_uframe, oclock, clock0, uframe0);
+}
+
modified = itd_complete (ehci, q.itd);
q = *q_p;
break;
@@ -2460,6 +2473,7 @@ restart:
break;
/* rescan the rest of this frame, then ... */
+oclock = clock;
clock = now;
clock_frame = clock >> 3;
if (ehci->clock_frame != clock_frame) {
--
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
