Update dev_err() messages to report status (including success) for each
step of USB RX HALT and TX HALT error recovery. If error recovery fails,
show the message:
stuck RX HALT state requires USB Reset Device to clear
or
stuck TX HALT state requires USB Reset Device to clear
This and related message revisions pertain to functions:
mceusb_defer_kevent() error handler dispatcher
mceusb_deferred_kevent() error handler itself
This patch just affects RX and TX HALT error handling and recovery
reporting.
A possible future patch may enable the mceusb driver to attempt
itself usb_reset_device() when necessary.
---
As seen on very rare occasions with mceusb device 2304:0225
[59388.696941] mceusb 1-1.1.2:1.0: Error: urb status = -32 (RX HALT)
[59388.698838] mceusb 1-1.1.2:1.0: rx clear halt error -32
the device can get into RX or TX HALT state where usb_clear_halt()
also fails and also returns -EPIPE (HALT/STALL). After which,
all further mceusb device control and data I/O fails with HALT/STALL.
Cause and problem replication conditions are still unknown.
As seen in https://ubuntuforums.org/showthread.php?t=2406882
...
mceusb 2-1.4:1.0: Error: urb status = -32 (RX HALT)
...
for an unknown mceusb device, usb_clear_halt() apparently can return
false success. After which, RX HALT immediately reoccurs resulting in
a RX halt and clear halt infinite loop and message flooding. Again,
cause and problem replication conditions remain unknown.
After observing a rx clear halt fault with mceusb 2304:0225, further
troubleshooting reveals usb_reset_device() is able to restore device
functionality.
$ lsusb
Bus 001 Device 010: ID 2304:0225 Pinnacle Systems, Inc. Remote Kit
Infrared Transceiver
...
$ sudo ./usbreset /dev/bus/usb/001/010 # ioctl(fd, USBDEVFS_RESET, 0);
Resetting USB device /dev/bus/usb/001/010
Reset successful
$ dmesg
...
[272392.540679] usb 1-1.2.7: reset full-speed USB device number 10
using dwc_otg
[272392.676616] Registered IR keymap rc-rc6-mce
...
[272392.678313] rc rc0: lirc_dev: driver ir-lirc-codec (mceusb)
registered at minor = 0
[272392.678616] mceusb 1-1.2.7:1.0: long-range (0x1) receiver active
[272392.940671] mceusb 1-1.2.7:1.0: Registered Pinnacle Systems PCTV
Remote USB with mce emulator interface version 1
[272392.940687] mceusb 1-1.2.7:1.0: 2 tx ports (0x3 cabled) and 2 rx
sensors (0x1 active)
...
Additional findings are "modprobe -r mceusb" and "modprobe mceusb"
are unable to clear stuck RX or TX HALT state.
Attempts to call usb_reset_endpoint() and usb_reset_configuration()
from within the mceusb driver also did not clear stuck HALTs.
A possible future patch could have the mceusb call usb_reset_device()
on itself, when necessary. Limiting the number of HALT error recovery
attempts may also be necessary to prevent halt/clear-halt loops.
Unresolved for now, deadlock complications occur if mceusb's worker
thread, mceusb_deferred_kevent(), calls usb_reset_device(),
which calls mceusb_dev_disconnect(), which calls cancel_work_sync(),
which waits on the still active worker thread.
Signed-off-by: A Sun <[email protected]>
---
drivers/media/rc/mceusb.c | 35 +++++++++++++++++++++++++----------
1 file changed, 25 insertions(+), 10 deletions(-)
diff --git a/drivers/media/rc/mceusb.c b/drivers/media/rc/mceusb.c
index efffb1795..5d81ccafc 100644
--- a/drivers/media/rc/mceusb.c
+++ b/drivers/media/rc/mceusb.c
@@ -765,7 +765,7 @@ static void mceusb_defer_kevent(struct mceusb_dev *ir, int
kevent)
{
set_bit(kevent, &ir->kevent_flags);
if (!schedule_work(&ir->kevent))
- dev_err(ir->dev, "kevent %d may have been dropped", kevent);
+ dev_dbg(ir->dev, "kevent %d already scheduled", kevent);
else
dev_dbg(ir->dev, "kevent %d scheduled", kevent);
}
@@ -1404,19 +1404,26 @@ static void mceusb_deferred_kevent(struct work_struct
*work)
container_of(work, struct mceusb_dev, kevent);
int status;
+ dev_info(ir->dev, "kevent handler called (flags 0x%lx)",
+ ir->kevent_flags);
+
if (test_bit(EVENT_RX_HALT, &ir->kevent_flags)) {
usb_unlink_urb(ir->urb_in);
status = usb_clear_halt(ir->usbdev, ir->pipe_in);
+ dev_err(ir->dev, "rx clear halt status = %d", status);
if (status < 0) {
- dev_err(ir->dev, "rx clear halt error %d",
- status);
- }
- clear_bit(EVENT_RX_HALT, &ir->kevent_flags);
- if (status == 0) {
+ /*
+ * Unable to clear RX stall/halt.
+ * Will need to call usb_reset_device().
+ */
+ dev_err(ir->dev,
+ "stuck RX HALT state requires USB Reset Device
to clear");
+ } else {
+ clear_bit(EVENT_RX_HALT, &ir->kevent_flags);
status = usb_submit_urb(ir->urb_in, GFP_KERNEL);
if (status < 0) {
dev_err(ir->dev,
- "rx unhalt submit urb error %d",
+ "rx unhalt submit urb error = %d",
status);
}
}
@@ -1424,9 +1431,17 @@ static void mceusb_deferred_kevent(struct work_struct
*work)
if (test_bit(EVENT_TX_HALT, &ir->kevent_flags)) {
status = usb_clear_halt(ir->usbdev, ir->pipe_out);
- if (status < 0)
- dev_err(ir->dev, "tx clear halt error %d", status);
- clear_bit(EVENT_TX_HALT, &ir->kevent_flags);
+ dev_err(ir->dev, "tx clear halt status = %d", status);
+ if (status < 0) {
+ /*
+ * Unable to clear TX stall/halt.
+ * Will need to call usb_reset_device().
+ */
+ dev_err(ir->dev,
+ "stuck TX HALT state requires USB Reset Device
to clear");
+ } else {
+ clear_bit(EVENT_TX_HALT, &ir->kevent_flags);
+ }
}
}
--
2.11.0
