Hi Andiry,
As I mentioned, I had been running into some issues with a prototype
host controller and several isochronous devices. I've tracked down the
bugs I was seeing to some issues in the URB enqueue and dequeue error
paths.
I couldn't quite reproduce the failure I had last week, but I injected
some errors into the xHCI driver to make the stop endpoint command
watchdog timer fire, and to create a failure while enqueueing the second
TD in a multi-TD isochronous URB.
Since you wrote most of the isochronous code, could you review these
patches and make sure they look sane? If they do, I'll send them off to
Greg, along with your other two bug fixes, tomorrow.
Thanks,
Sarah Sharp
The following changes since commit 977f74b023b1e26123e6a4bbd9084040b9d384f2:
xHCI: report USB2 port in resuming as suspend (2011-08-03 09:51:01 -0700)
are available in the git repository at:
git+ssh://master.kernel.org/pub/scm/linux/kernel/git/sarah/xhci.git for-usb-linus
Sarah Sharp (3):
xhci: Fix memory leak during failed enqueue.
xhci: Fix failed enqueue in the middle of isoch TD.
xhci: Remove TDs from TD lists when URBs are canceled.
drivers/usb/host/xhci-ring.c | 68 +++++++++++++++++++++++++++++++----------
drivers/usb/host/xhci.c | 28 +++++++++++++++--
2 files changed, 75 insertions(+), 21 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
