On Sat, Aug 31, 2013 at 12:11:49AM +0800, Giovanni wrote: > Hi Rui, > > >Hi, > > > >Can you give us a detail kernel top commit and config which you built, > >in order to locate the break line (address offset) accurately like below: > >.... > >I built on top v3.10, but it looks like line 1399 isn't the breakpoit. > > > >Thanks, > >Rui > > Sorry I missed your message, and sorry for incorrect information. In > bugzilla I set the version 3.10.9 but in description I not written. > In bugzilla I attached the config file: > > https://bugzilla.kernel.org/attachment.cgi?id=107368 > I drafted a patch, please have a test and enable CONFIG_XHCI_HCD_DEBUGGING in kernel configuration at same time. 8<---------------------- cut here ---------------------------- diff --git a/drivers/usb/host/xhci-ring.c b/drivers/usb/host/xhci-ring.c index cc3bfc5..9a255e6 100644 --- a/drivers/usb/host/xhci-ring.c +++ b/drivers/usb/host/xhci-ring.c @@ -1153,6 +1153,12 @@ static void handle_reset_ep_completion(struct xhci_hcd *xhci, xhci_dbg(xhci, "Ignoring reset ep completion code of %u\n", GET_COMP_CODE(le32_to_cpu(event->status))); + if (!xhci->devs[slot_id]) { + xhci_warn(xhci, "WARN: No device with on slot %d\n", + slot_id); + return; + } + /* HW with the reset endpoint quirk needs to have a configure endpoint * command complete before the endpoint can be used. Queue that here * because the HW can't handle two commands being queued in a row. --- Thanks, Rui -- 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
