On Wed, Nov 20, 2013 at 09:54:02AM -0000, David Laight wrote: > > From: Of David Miller > > > From: "David Laight" <David.Laight@xxxxxxxxxx> > > > Date: Tue, 19 Nov 2013 10:04:11 -0000 > > > > > There is a patch to xhci-ring.c that should fix the SG problem. > > > http://www.spinics.net/lists/linux-usb/msg97176.html > > > > > > I think it should apply to the 3.12 sources. > > > > David, please get this into Linus's tree and queued up for -stable > > as soon as possible, thank you. > > What do I have to do to expedite it? > I thought that Sarah would have to push the patch through. Yes, I think the patch should come through my tree. > It might be necessary to limit the number of SG fragments that > the mass storage (etc) can generate to 63 (from unlimited). > That should be a 1 line change somewhere. Greg's USB tree is frozen until after 3.13-rc1. The patch will have to wait until then, but it will be marked for stable. I have no objection to the methodology of the patch, but we do need to figure out how to limit the number of scatter-gather list entries in the mass storage driver. We're still figuring out the exact limitation we need to use, and then the patch can be queued in my for-usb-linus branch for 3.13-rc1. Note that the patch is a bandaid fix, and will only help bulk endpoints that are submitting scatter-gather transfers. Interrupt endpoints using scatter-gather will still run into these issues, but with David's patch, the URB submission will now fail, rather than allowing the transfer to be silently corrupted. The real fix for this will probably be too large for stable, and is likely to take at least a kernel revision to fix. Sarah Sharp -- 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
