diff options
Diffstat (limited to 'target/linux/bcm27xx/patches-5.10/950-0733-usb-xhci-workaround-for-bogus-SET_DEQ_PENDING-endpoi.patch')
-rw-r--r-- | target/linux/bcm27xx/patches-5.10/950-0733-usb-xhci-workaround-for-bogus-SET_DEQ_PENDING-endpoi.patch | 41 |
1 files changed, 41 insertions, 0 deletions
diff --git a/target/linux/bcm27xx/patches-5.10/950-0733-usb-xhci-workaround-for-bogus-SET_DEQ_PENDING-endpoi.patch b/target/linux/bcm27xx/patches-5.10/950-0733-usb-xhci-workaround-for-bogus-SET_DEQ_PENDING-endpoi.patch new file mode 100644 index 0000000000..6e43bebf5e --- /dev/null +++ b/target/linux/bcm27xx/patches-5.10/950-0733-usb-xhci-workaround-for-bogus-SET_DEQ_PENDING-endpoi.patch @@ -0,0 +1,41 @@ +From dedd960b894217ae0960a7ac493def9edfa96905 Mon Sep 17 00:00:00 2001 +From: Jonathan Bell <jonathan@raspberrypi.com> +Date: Wed, 11 Aug 2021 15:33:57 +0100 +Subject: [PATCH] usb: xhci: workaround for bogus SET_DEQ_PENDING + endpoint state + +See https://github.com/raspberrypi/linux/issues/3981 + +An unknown unsafe memory access can result in the ep_state variable +in xhci_virt_ep being trampled with a stuck SET_DEQ_PENDING state +despite successful completion of a Set TR Deq Pointer command. + +All URB enqueue/dequeue calls for the endpoint will fail in this state +so no transfers are possible until the device is reconnected. + +As a workaround, clear the flag if we see it set and issue a new Set +TR Deq command anyway - this should be harmless, as a prior Set TR Deq +command will only have been issued in the Stopped state, and if the +endpoint is Running then the controller is required to ignore it and +respond with a Context State Error event TRB. + +Signed-off-by: Jonathan Bell <jonathan@raspberrypi.com> +--- + drivers/usb/host/xhci-ring.c | 6 +++--- + 1 file changed, 3 insertions(+), 3 deletions(-) + +--- a/drivers/usb/host/xhci-ring.c ++++ b/drivers/usb/host/xhci-ring.c +@@ -4255,9 +4255,9 @@ void xhci_queue_new_dequeue_state(struct + } + ep = &xhci->devs[slot_id]->eps[ep_index]; + if ((ep->ep_state & SET_DEQ_PENDING)) { +- xhci_warn(xhci, "WARN Cannot submit Set TR Deq Ptr\n"); +- xhci_warn(xhci, "A Set TR Deq Ptr command is pending.\n"); +- return; ++ xhci_warn(xhci, "WARN A Set TR Deq Ptr command is pending for slot %u ep %u\n", ++ slot_id, ep_index); ++ ep->ep_state &= ~SET_DEQ_PENDING; + } + + /* This function gets called from contexts where it cannot sleep */ |