From 08406274ea31e44a87316d45af5a50c8b1989c4b Mon Sep 17 00:00:00 2001 From: Con Kolivas Date: Mon, 7 Oct 2013 20:34:13 +1100 Subject: [PATCH] Revert "Check for the async variants of the libusb errors and retry up to retry max when trying to clear a pipe error/stall." This reverts commit c7d5256b3657bd6d4f2f04f7288cd757bcbf77a5. --- usbutils.c | 15 ++++----------- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/usbutils.c b/usbutils.c index ddfc079d..76249890 100644 --- a/usbutils.c +++ b/usbutils.c @@ -20,10 +20,7 @@ #define NODEV(err) ((err) == LIBUSB_ERROR_NO_DEVICE || \ (err) == LIBUSB_ERROR_PIPE || \ - (err) == LIBUSB_ERROR_OTHER || \ - (err) == LIBUSB_TRANSFER_NO_DEVICE || \ - (err) == LIBUSB_TRANSFER_STALL || \ - (err) == LIBUSB_TRANSFER_ERROR) + (err) == LIBUSB_ERROR_OTHER) #define NOCONTROLDEV(err) ((err) == LIBUSB_ERROR_NO_DEVICE || \ (err) == LIBUSB_ERROR_OTHER) @@ -2268,10 +2265,10 @@ usb_bulk_transfer(struct libusb_device_handle *dev_handle, int intinfo, enum usb_cmds cmd, __maybe_unused int seq) { struct usb_epinfo *usb_epinfo; - int err, errn, retries = 0; struct usb_transfer ut; unsigned char endpoint; uint16_t MaxPacketSize; + int err, errn; #if DO_USB_STATS struct timeval tv_start, tv_finish; #endif @@ -2298,7 +2295,6 @@ usb_bulk_transfer(struct libusb_device_handle *dev_handle, int intinfo, libusb_fill_bulk_transfer(ut.transfer, dev_handle, endpoint, buf, length, bulk_callback, &ut, timeout); -retry: STATS_TIMEVAL(&tv_start); cg_rlock(&cgusb_fd_lock); err = libusb_submit_transfer(ut.transfer); @@ -2310,13 +2306,12 @@ retry: STATS_TIMEVAL(&tv_finish); USB_STATS(cgpu, &tv_start, &tv_finish, err, mode, cmd, seq, timeout); - if (err < 0) { + if (err < 0) applog(LOG_DEBUG, "%s%i: %s (amt=%d err=%d ern=%d)", cgpu->drv->name, cgpu->device_id, usb_cmdname(cmd), *transferred, err, errn); - } - if ((err == LIBUSB_ERROR_PIPE || err == LIBUSB_TRANSFER_STALL) && ++retries < USB_RETRY_MAX) { + if (err == LIBUSB_ERROR_PIPE || err == LIBUSB_TRANSFER_STALL) { cgpu->usbinfo.last_pipe = time(NULL); cgpu->usbinfo.pipe_count++; applog(LOG_INFO, "%s%i: libusb pipe error, trying to clear", @@ -2327,8 +2322,6 @@ retry: if (err) cgpu->usbinfo.clear_fail_count++; - else - goto retry; } if ((endpoint & LIBUSB_ENDPOINT_DIR_MASK) == LIBUSB_ENDPOINT_IN) memcpy(data, buf, length);