From 585d4af4e6b4fa30fd95a1f2cd88d284de647bdc Mon Sep 17 00:00:00 2001 From: Con Kolivas Date: Mon, 7 Oct 2013 23:32:01 +1100 Subject: [PATCH] Store statistics on how often we have to cancel async bulk transfers and add a debug message whenever we do. --- miner.h | 1 + usbutils.c | 8 ++++++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/miner.h b/miner.h index fdfc7797..697b8ee1 100644 --- a/miner.h +++ b/miner.h @@ -481,6 +481,7 @@ struct cgpu_info { #endif #ifdef USE_USBUTILS struct cg_usb_info usbinfo; + int usb_cancels; #endif #ifdef USE_MODMINER char fpgaid; diff --git a/usbutils.c b/usbutils.c index 4f7f6f9b..2532b33a 100644 --- a/usbutils.c +++ b/usbutils.c @@ -2226,7 +2226,8 @@ static void LIBUSB_CALL bulk_callback(struct libusb_transfer *transfer) /* Wait for callback function to tell us it has finished the USB transfer, but * use our own timer to cancel the request if we go beyond the timeout. */ -static int callback_wait(struct usb_transfer *ut, int *transferred, unsigned int timeout) +static int callback_wait(struct cgpu_info *cgpu, struct usb_transfer *ut, int *transferred, + unsigned int timeout) { struct libusb_transfer *transfer= ut->transfer; struct timespec ts_now, ts_end; @@ -2246,6 +2247,9 @@ static int callback_wait(struct usb_transfer *ut, int *transferred, unsigned int * it the same as a timeout. */ libusb_clear_halt(transfer->dev_handle, transfer->endpoint); libusb_cancel_transfer(transfer); + applog(LOG_DEBUG, "%s%i: libusb cancelling async bulk transfer", + cgpu->drv->name, cgpu->device_id); + cgpu->usb_cancels++; /* Now wait for the callback function to be invoked. */ pthread_cond_wait(&ut->cond, &ut->mutex); @@ -2304,7 +2308,7 @@ usb_bulk_transfer(struct libusb_device_handle *dev_handle, int intinfo, cg_runlock(&cgusb_fd_lock); errn = errno; if (!err) - err = callback_wait(&ut, transferred, timeout); + err = callback_wait(cgpu, &ut, transferred, timeout); STATS_TIMEVAL(&tv_finish); USB_STATS(cgpu, &tv_start, &tv_finish, err, mode, cmd, seq, timeout);