laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/simtrace2/+/27389 )

Change subject: simtrace2-cardem-pcsc: continue in case of LIBUSB_TRANSFER_ERROR
......................................................................

simtrace2-cardem-pcsc: continue in case of LIBUSB_TRANSFER_ERROR

Sometimes I get LIBUSB_TRANSFER_ERROR particularly when the USB bus
is very busy.  We shouldn't terminate the program, but simply resubmit
it.  That's what we have multiple transfers for...

Change-Id: I77d7bc636c21171fcff7e70e87c0109cbaee9b51
---
M host/src/simtrace2-cardem-pcsc.c
1 file changed, 6 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/host/src/simtrace2-cardem-pcsc.c b/host/src/simtrace2-cardem-pcsc.c
index 031e1f0..b7d41bd 100644
--- a/host/src/simtrace2-cardem-pcsc.c
+++ b/host/src/simtrace2-cardem-pcsc.c
@@ -278,6 +278,9 @@
                /* hand the message up the stack */
                process_usb_msg(ci, xfer->buffer, xfer->actual_length);
                break;
+       case LIBUSB_TRANSFER_ERROR:
+               LOGCI(ci, LOGL_FATAL, "USB IN transfer error, trying 
resubmit\n");
+               break;
        case LIBUSB_TRANSFER_NO_DEVICE:
                LOGCI(ci, LOGL_FATAL, "USB device disappeared\n");
                exit(1);
@@ -329,6 +332,9 @@
        case LIBUSB_TRANSFER_COMPLETED:
                process_usb_msg_irq(ci, xfer->buffer, xfer->actual_length);
                break;
+       case LIBUSB_TRANSFER_ERROR:
+               LOGCI(ci, LOGL_FATAL, "USB INT transfer error, trying 
resubmit\n");
+               break;
        case LIBUSB_TRANSFER_NO_DEVICE:
                LOGCI(ci, LOGL_FATAL, "USB device disappeared\n");
                exit(1);

--
To view, visit https://gerrit.osmocom.org/c/simtrace2/+/27389
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: simtrace2
Gerrit-Branch: master
Gerrit-Change-Id: I77d7bc636c21171fcff7e70e87c0109cbaee9b51
Gerrit-Change-Number: 27389
Gerrit-PatchSet: 4
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-MessageType: merged

Reply via email to