> You're right. I do wish the spec had been written more clearly. I've read a lot of hardware specs in my time ... > > 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. If all the fragments are larger than the MBP (assume 16k) then that would be relatively easy. However that is very dependant on the source of the data. It might be true for disk data, but is unlikely to be true for ethernet data. For bulk data the link TRB can be forced at a packet boundary by splitting the TD up - the receiving end won't know the difference. > 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. 250 interrupts/sec is noise. Send/receive 13000 ethernet packets/sec and then look at the interrupt rate! There is no necessity for taking an interrupt from every link segment. OTOH an interrupt is requested for every bulk TD. I'm sending ethernet data (with TSO) and each TD is just under 64k mostly made up of 3 or 4 fragments. The receive side is interrupting for every receive packet. > Using larger ring segments would help. The current ring segments contain 64 entries, a strange choice since they are created with 2 segments. (The ring expansion code soon doubles that for my ethernet traffic.) I would change the code to use a single segment (for coding simplicity) and queue bulk URB when there isn't enough ring space. URB with too many fragments could either be rejected, sent in sections, or partially linearised (and probably still sent in sections). David -- 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
