diff options
| author | Andiry Xu <andiry.xu@amd.com> | 2010-08-09 22:56:15 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-08-24 05:50:16 +0200 |
| commit | 14184f9b8047026f1812f49df074e89dad3a09bc (patch) | |
| tree | 5492adad4d8ce5d263abbdad713c13a35b08202c /fs | |
| parent | USB: xhci: Remove buggy assignment in next_trb() (diff) | |
| download | linux-14184f9b8047026f1812f49df074e89dad3a09bc.tar.xz linux-14184f9b8047026f1812f49df074e89dad3a09bc.zip | |
USB: xHCI: update ring dequeue pointer when process missed tds
This patch fixes a isoc transfer bug reported by Sander Eikelenboom.
When ep->skip is set, endpoint ring dequeue pointer should be updated
when processed every missed td. Although ring dequeue pointer will also
be updated when ep->skip is clear, leave it intact during missed tds
processing may cause two issues:
1). If the very next valid transfer following missed tds is a short
transfer, its actual_length will be miscalculated;
2). If there are too many missed tds during transfer, new inserted tds
may found the transfer ring full and urb enqueue fails.
Reported-by: Sander Eikelenboom <linux@eikelenboom.it>
Tested-by: Sander Eikelenboom <linux@eikelenboom.it>
Signed-off-by: Andiry Xu <andiry.xu@amd.com>
Signed-off-by: Sarah Sharp <sarah.a.sharp@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'fs')
0 files changed, 0 insertions, 0 deletions