usb: wusbcore: skip done segs before completing aborted transfer
authorThomas Pugliese <thomas.pugliese@gmail.com>
Tue, 16 Sep 2014 21:36:02 +0000 (16:36 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 24 Sep 2014 05:06:59 +0000 (22:06 -0700)
When completing an aborted transfer, skip done segs before calling
wa_complete_remaining_xfer_segs to avoid a runtime warning.  The warning
is harmless in this case but avoiding it prevents false error reports.

Signed-off-by: Thomas Pugliese <thomas.pugliese@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/wusbcore/wa-xfer.c

index e279015be466166742367f7bcb958639f856b447..69af4fd9e07220b29edeeec2b46fbde469d7d2ac 100644 (file)
@@ -459,14 +459,25 @@ static void __wa_xfer_abort_cb(struct urb *urb)
                        __func__, urb->status);
                if (xfer) {
                        unsigned long flags;
-                       int done;
+                       int done, seg_index = 0;
                        struct wa_rpipe *rpipe = xfer->ep->hcpriv;
 
                        dev_err(dev, "%s: cleaning up xfer %p ID 0x%08X.\n",
                                __func__, xfer, wa_xfer_id(xfer));
                        spin_lock_irqsave(&xfer->lock, flags);
-                       /* mark all segs as aborted. */
-                       wa_complete_remaining_xfer_segs(xfer, 0,
+                       /* skip done segs. */
+                       while (seg_index < xfer->segs) {
+                               struct wa_seg *seg = xfer->seg[seg_index];
+
+                               if ((seg->status == WA_SEG_DONE) ||
+                                       (seg->status == WA_SEG_ERROR)) {
+                                       ++seg_index;
+                               } else {
+                                       break;
+                               }
+                       }
+                       /* mark remaining segs as aborted. */
+                       wa_complete_remaining_xfer_segs(xfer, seg_index,
                                WA_SEG_ABORTED);
                        done = __wa_xfer_is_done(xfer);
                        spin_unlock_irqrestore(&xfer->lock, flags);