On Mon, Nov 11, 2013 at 03:34:53PM -0500, Alan Stern wrote: > On Mon, 11 Nov 2013, David Laight wrote: > > > > Suppose, for example, the MBP is 1024. If you have a TD with length > > > 1500, and if it had only one fragment, the last (and only) fragment's > > > length would not less than the MBP and it would not be an exact > > > multiple of the MBP. > > > > That doesn't matter - eg example 2 in figure 25 > > You're right. I do wish the spec had been written more clearly. > > > Reading it all again makes me think that a LINK trb is only > > allowed on the burst boundary (which might be 16k bytes). > > The only real way to implement that is to ensure that TD never > > contain LINK TRB. > > That's one way to do it. Or you could allow a Link TRB at an > intermediate MBP boundary. I like this idea instead. The xHCI driver should be modified to be able to handle link TRBs in the middle of the segments (the cancellation code would have to be touched as well). We would keep a running count of the number of bytes left in a TD fragment, as we fill in the TRBs. If we find the TD fragment would span a link TRB, we backtrack to the end of the last TD fragment, put in a link TRB, and then continue on the next segment. > It comes down to a question of how often you want the controller to > issue an interrupt. If a ring segment is 4 KB (one page), then it can > hold 256 TRBs. With scatter-gather transfers, each SG element > typically refers to something like a 2-page buffer (depends on how > fragmented the memory is). Therefore a ring segment will describe > somewhere around 512 pages of data, i.e., something like 2 MB. Since > SuperSpeed is 500 MB/s, you'd end up getting in the vicinity of 250 > interrupts every second just because of ring segment crossings. The driver is currently defined to have 64 TRBs per ring segment. But that doesn't matter; we don't get an interrupt when a ring segment is crossed. Instead we set the interrupt-on-completion flag on the last TRB of the TD, not on any earlier fragment or link TRB. > Using larger ring segments would help. Ring segments have to be physically contiguous, so I'm not sure if we want to ask for segments that are bigger than a page. I've already got a report from someone else about the ring expansion getting out of control, so I would like to figure that out before we talk about using even bigger segments. Finally, it's interesting to note that the USB mass storage driver is using scatter gather lists just fine without the driver following the TD fragment rules. Or at least no one has reported any issues. I wonder why it works? 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
