David Laight <David.Laight@xxxxxxxxxx> writes: > From: linux-usb-owner@xxxxxxxxxxxxxxx [mailto:linux-usb-owner@xxxxxxxxxxxxxxx] On Behalf Of Jim Baxter >> Sent: 16 May 2017 18:41 >> >> The CDC-NCM driver can require large amounts of memory to create >> skb's and this can be a problem when the memory becomes fragmented. >> >> This especially affects embedded systems that have constrained >> resources but wish to maximise the throughput of CDC-NCM with 16KiB >> NTB's. > > Why is this driver copying multiple tx messages into a single skb. Mostly becasue it already did that when I started messing with it, and I didn't know how to avoid that. > Surely there are better ways to do this?? But I have been there thinking this exact thought a couple of times. Suggestions are appreciated. > I think it is generating a 'multi-ethernet frame' URB with an > overall header for each URB and a header for each ethernet frame. With some negotiated alignment restrictions, and a linked list of frame pointer arrays. But yes, that is basically it. (it's not always ethernet - with MBIM it can be IP or arbitrary as well, but I don't think that makes any difference) > Given that the USB stack allows multiple concurrent transmits I'm > surprised that batching large ethernet frames makes much difference. Me too. Actually, I don't think it does. The protocol was developed with specific device restrictions in mind. These might be invalid today. There is no reason to believe that using simple CDC ECM framing (i.e. one ethernet frame per URB) is any problem. > Also the USB target can't actually tell when URB that contain > multiples of the USB packet size end. > So it is possible to send a single NTB as multiple URB. Nice idea! Never thought of that. Yes, the driver could use a number smaller buffers regardless of the NTB size, by abusing the fact that the device will see them as a contigious USB transfer as long as they fall on USB packet boundaries. Started thinking about how to do this in practice. It seemed obviously simply to jsut fire off the buffers as they fill up until the the max aggregation size or time has been exceeded. But the header makes this harder than necessary. It contains both a length and a pointer to the first frame pointer array (NDP). So we will have to decide the size of the NTB and where to put the first NDP before sending the first USB packet. This is possible if we always go for the pad-to-max strategy. We'll also have to make some assumptions about the size of the NDP(s) as we add them, but we already do that so I don't think it is a big deal. Might be the way to go. Unless someone has a nice way to just collect a list of skbs and have them converted to proper framing on the fly when transmitting, without having to care about USB packet boundaries. Bjørn -- 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
