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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
