Re: Bug 60810 - Kernel oops with controller XHCI while wait usb packet

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux