Date: Sat, 10 Jul 2021 20:30:22 +0100 From: Jessica Clarke <jrtc27@freebsd.org> To: Hans Petter Selasky <hselasky@FreeBSD.org> Cc: "src-committers@freebsd.org" <src-committers@FreeBSD.org>, "dev-commits-src-all@freebsd.org" <dev-commits-src-all@FreeBSD.org>, "dev-commits-src-main@freebsd.org" <dev-commits-src-main@FreeBSD.org> Subject: Re: git: d038463bd269 - main - Make sure the XHCI driver obeys the isochronous scheduling threshold value as given by the XHCI hardware parameters to avoid scheduling isochronous transfers too early. Message-ID: <09444339-973B-4A2B-ACEA-B00B8E48F460@freebsd.org> In-Reply-To: <202107101904.16AJ4lj3035432@gitrepo.freebsd.org> References: <202107101904.16AJ4lj3035432@gitrepo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 10 Jul 2021, at 20:04, Hans Petter Selasky <hselasky@FreeBSD.org> = wrote: >=20 > The branch main has been updated by hselasky: >=20 > URL: = https://cgit.FreeBSD.org/src/commit/?id=3Dd038463bd269031fa8f13c0951b223bc= ae02bbca >=20 > commit d038463bd269031fa8f13c0951b223bcae02bbca > Author: Hans Petter Selasky <hselasky@FreeBSD.org> > AuthorDate: 2021-07-10 16:09:56 +0000 > Commit: Hans Petter Selasky <hselasky@FreeBSD.org> > CommitDate: 2021-07-10 17:57:52 +0000 >=20 > Make sure the XHCI driver obeys the isochronous scheduling = threshold value > as given by the XHCI hardware parameters to avoid scheduling = isochronous > transfers too early. >=20 > MFC after: 1 week > Sponsored by: NVIDIA Networking > --- > sys/dev/usb/controller/xhci.c | 11 ++++++++++- > sys/dev/usb/controller/xhci.h | 3 +++ > 2 files changed, 13 insertions(+), 1 deletion(-) >=20 > diff --git a/sys/dev/usb/controller/xhci.c = b/sys/dev/usb/controller/xhci.c > index 10e37c97c254..dd64c2d45141 100644 > --- a/sys/dev/usb/controller/xhci.c > +++ b/sys/dev/usb/controller/xhci.c > @@ -644,6 +644,9 @@ xhci_init(struct xhci_softc *sc, device_t self, = uint8_t dma32) >=20 > DPRINTF("HCS2=3D0x%08x\n", temp); >=20 > + /* get isochronous scheduling threshold */ > + sc->sc_ist =3D XHCI_HCS2_IST(temp); > + > /* get number of scratchpads */ > sc->sc_noscratch =3D XHCI_HCS2_SPB_MAX(temp); >=20 > @@ -2075,7 +2078,13 @@ xhci_setup_generic_chain(struct usb_xfer *xfer) >=20 > x =3D XREAD4(temp.sc, runt, XHCI_MFINDEX); >=20 > - DPRINTF("MFINDEX=3D0x%08x\n", x); > + DPRINTF("MFINDEX=3D0x%08x IST=3D0x%x\n", x, sc->sc_ist); This broke amd64 GENERIC with error: use of undeclared identifier =E2=80=98= sc' Should this be temp.sc? Jess
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?09444339-973B-4A2B-ACEA-B00B8E48F460>