Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 25 Oct 2013 06:47:01 +0000 (UTC)
From:      Hans Petter Selasky <hselasky@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-8@freebsd.org
Subject:   svn commit: r257106 - stable/8/sys/dev/usb/controller
Message-ID:  <201310250647.r9P6l1fW035839@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: hselasky
Date: Fri Oct 25 06:47:01 2013
New Revision: 257106
URL: http://svnweb.freebsd.org/changeset/base/257106

Log:
  MFC r256750:
  Improve XHCI stability. When a command timeout happens, the command
  should be aborted else the command queue can stop. Refer to section
  "4.6.1.2" of the XHCI specification.

Modified:
  stable/8/sys/dev/usb/controller/xhci.c
Directory Properties:
  stable/8/sys/   (props changed)
  stable/8/sys/dev/   (props changed)
  stable/8/sys/dev/usb/   (props changed)

Modified: stable/8/sys/dev/usb/controller/xhci.c
==============================================================================
--- stable/8/sys/dev/usb/controller/xhci.c	Fri Oct 25 06:37:43 2013	(r257105)
+++ stable/8/sys/dev/usb/controller/xhci.c	Fri Oct 25 06:47:01 2013	(r257106)
@@ -1110,6 +1110,25 @@ xhci_do_command(struct xhci_softc *sc, s
 	}
 	if (err != 0) {
 		DPRINTFN(0, "Command timeout!\n");
+
+		/*
+		 * Try to abort the last command as per section
+		 * 4.6.1.2 "Aborting a Command" of the XHCI
+		 * specification:
+		 */
+		temp = XREAD4(sc, oper, XHCI_CRCR_LO);
+		XWRITE4(sc, oper, XHCI_CRCR_LO, temp | XHCI_CRCR_LO_CA);
+
+		/* wait for abort event, if any */
+		err = cv_timedwait(&sc->sc_cmd_cv, &sc->sc_bus.bus_mtx, hz / 16);
+
+		if (err != 0 && xhci_interrupt_poll(sc) != 0) {
+			DPRINTF("Command was completed when polling\n");
+			err = 0;
+		}
+		if (err != 0) {
+			DPRINTF("Command abort timeout!\n");
+		}
 		err = USB_ERR_TIMEOUT;
 		trb->dwTrb2 = 0;
 		trb->dwTrb3 = 0;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201310250647.r9P6l1fW035839>