Attachment 301746 Details for Bug 435787 – Fix-ups which worked for me

[patch] Fix-ups which worked for me

linux-2.6.9-68.32.EL-ehci1.diff (text/plain), 5.31 KB, created by Pete Zaitcev on 2008-04-09 03:33:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Pete Zaitcev

Created: 2008-04-09 03:33:15 UTC

Size: 5.31 KB

patch

obsolete

>diff -urp -X dontdiff linux-2.6.9-68.32.EL/drivers/usb/host/ehci-hcd.c linux-2.6.9-68.32.EL-u1/drivers/usb/host/ehci-hcd.c
>--- linux-2.6.9-68.32.EL/drivers/usb/host/ehci-hcd.c	2008-04-08 23:40:39.000000000 -0700
>+++ linux-2.6.9-68.32.EL-u1/drivers/usb/host/ehci-hcd.c	2008-04-09 02:56:39.000000000 -0700
>@@ -946,6 +946,30 @@ static int ehci_urb_enqueue (
> 	}
> }
> 
>+static void unlink_async (struct ehci_hcd *ehci, struct ehci_qh *qh)
>+{
>+	/* if we need to use IAA and it's busy, defer */
>+	if (qh->qh_state == QH_STATE_LINKED
>+			&& ehci->reclaim
>+			&& HCD_IS_RUNNING (ehci->hcd.state)) {
>+		struct ehci_qh		*last;
>+
>+		for (last = ehci->reclaim;
>+				last->reclaim;
>+				last = last->reclaim)
>+			continue;
>+		qh->qh_state = QH_STATE_UNLINK_WAIT;
>+		last->reclaim = qh;
>+
>+	/* bypass IAA if the hc can't care */
>+	} else if (!HCD_IS_RUNNING (ehci->hcd.state) && ehci->reclaim)
>+		end_unlink_async (ehci, NULL);
>+
>+	/* something else might have unlinked the qh by now */
>+	if (qh->qh_state == QH_STATE_LINKED)
>+		start_unlink_async (ehci, qh);
>+}
>+
> /* remove from hardware lists
>  * completions normally happen asynchronously
>  */
>@@ -965,19 +989,7 @@ static int ehci_urb_dequeue (struct usb_
> 		if (!qh)
> 			break;
> 
>-		switch (qh->qh_state) {
>-		case QH_STATE_LINKED:
>-		case QH_STATE_COMPLETING:
>-			//unlink_async(ehci, qh);
>-			break;
>-		case QH_STATE_UNLINK:
>-		case QH_STATE_UNLINK_WAIT:
>-			/* already started */
>-			break;
>-		case QH_STATE_IDLE:
>-			WARN_ON(1);
>-			break;
>-		}
>+		unlink_async(ehci, qh);
> 		break;
> 
> 	case PIPE_INTERRUPT:
>@@ -1030,7 +1042,7 @@ ehci_endpoint_disable (struct usb_hcd *h
> 	struct ehci_hcd		*ehci = hcd_to_ehci (hcd);
> 	int			epnum;
> 	unsigned long		flags;
>-	struct ehci_qh		*qh;
>+	struct ehci_qh		*qh, *tmp;
> 
> 	/* ASSERT:  any requests/urbs are being unlinked */
> 	/* ASSERT:  nobody can be submitting urbs for this any more */
>@@ -1056,6 +1068,16 @@ rescan:
> 	if (!HCD_IS_RUNNING (ehci->hcd.state))
> 		qh->qh_state = QH_STATE_IDLE;
> 	switch (qh->qh_state) {
>+	case QH_STATE_LINKED:
>+		for (tmp = ehci->async->qh_next.qh;
>+				tmp && tmp != qh;
>+				tmp = tmp->qh_next.qh)
>+			continue;
>+		/* periodic qh self-unlinks on empty */
>+		if (!tmp)
>+			goto nogood;
>+		unlink_async (ehci, qh);
>+		/* FALL THROUGH */
> 	case QH_STATE_UNLINK:		/* wait for hw to finish? */
> 	case QH_STATE_UNLINK_WAIT:
> idle_timeout:
>@@ -1070,6 +1092,7 @@ idle_timeout:
> 		}
> 		/* else FALL THROUGH */
> 	default:
>+nogood:
> 		/* caller was supposed to have unlinked any requests;
> 		 * that's not our job.  just leak this memory.
> 		 */
>diff -urp -X dontdiff linux-2.6.9-68.32.EL/drivers/usb/host/ehci-q.c linux-2.6.9-68.32.EL-u1/drivers/usb/host/ehci-q.c
>--- linux-2.6.9-68.32.EL/drivers/usb/host/ehci-q.c	2008-04-08 23:40:39.000000000 -0700
>+++ linux-2.6.9-68.32.EL-u1/drivers/usb/host/ehci-q.c	2008-04-09 02:49:15.000000000 -0700
>@@ -96,6 +96,29 @@ qh_update (struct ehci_hcd *ehci, struct
> 	qh->hw_token &= __constant_cpu_to_le32 (QTD_TOGGLE | QTD_STS_PING);
> }
> 
>+/* if it weren't for a common silicon quirk (writing the dummy into the qh
>+ * overlay, so qh->hw_token wrongly becomes inactive/halted), only fault
>+ * recovery (including urb dequeue) would need software changes to a QH...
>+ */
>+static void
>+qh_refresh (struct ehci_hcd *ehci, struct ehci_qh *qh)
>+{
>+	struct ehci_qtd *qtd;
>+
>+	if (list_empty (&qh->qtd_list))
>+		qtd = qh->dummy;
>+	else {
>+		qtd = list_entry (qh->qtd_list.next,
>+				struct ehci_qtd, qtd_list);
>+		/* first qtd may already be partially processed */
>+		if (cpu_to_le32 (qtd->qtd_dma) == qh->hw_current)
>+			qtd = NULL;
>+	}
>+
>+	if (qtd)
>+		qh_update (ehci, qh, qtd);
>+}
>+
> /*-------------------------------------------------------------------------*/
> 
> static void qtd_copy_status (
>@@ -224,6 +247,11 @@ ehci_urb_done (struct ehci_hcd *ehci, st
> 	spin_lock (&ehci->lock);
> }
> 
>+static void start_unlink_async (struct ehci_hcd *ehci, struct ehci_qh *qh);
>+static void unlink_async (struct ehci_hcd *ehci, struct ehci_qh *qh);
>+
>+static void intr_deschedule (struct ehci_hcd *ehci, struct ehci_qh *qh, int wait);
>+static int qh_schedule (struct ehci_hcd *ehci, struct ehci_qh *qh);
> 
> /*
>  * Process and free completed qtds for a qh, returning URBs to drivers.
>@@ -368,17 +396,22 @@ halt:
> 	if (unlikely (stopped != 0)
> 			/* some EHCI 0.95 impls will overlay dummy qtds */ 
> 			|| qh->hw_qtd_next == EHCI_LIST_END) {
>-		if (list_empty (&qh->qtd_list))
>-			end = qh->dummy;
>-		else {
>-			end = list_entry (qh->qtd_list.next,
>-					struct ehci_qtd, qtd_list);
>-			/* first qtd may already be partially processed */
>-			if (cpu_to_le32 (end->qtd_dma) == qh->hw_current)
>-				end = NULL;
>+		switch (state) {
>+		case QH_STATE_IDLE:
>+			qh_refresh(ehci, qh);
>+			break;
>+		case QH_STATE_LINKED:
>+			/* should be rare for periodic transfers,
>+			 * except maybe high bandwidth ...
>+			 */
>+			if ((cpu_to_le32(0x00ff) & qh->hw_info2) != 0) {
>+				intr_deschedule (ehci, qh, 1);
>+				(void) qh_schedule (ehci, qh);
>+			} else
>+				unlink_async (ehci, qh);
>+			break;
>+		/* otherwise, unlink already started */
> 		}
>-		if (end)
>-			qh_update (ehci, qh, end);
> 	}
> 
> 	return count;
>@@ -931,8 +964,6 @@ submit_async (
> 
> /* the async qh for the qtds being reclaimed are now unlinked from the HC */
> 
>-static void start_unlink_async (struct ehci_hcd *ehci, struct ehci_qh *qh);
>-
> static void end_unlink_async (struct ehci_hcd *ehci, struct pt_regs *regs)
> {
> 	struct ehci_qh		*qh = ehci->reclaim;

Actions: View | Diff

Attachments on bug 435787: 297411 | 297591 | 301746 | 301913 | 303125 | 304091