On 08/19/2015 07:08 PM, Eric Dumazet wrote: > On Wed, 2015-08-19 at 15:07 +0000, Grumbach, Emmanuel wrote: > >> I'll look at it. >> I was almost starting to implement that but then I thought with another >> (good?) reason to use LSO. LSO gives me the guarantee that the packet is >> directed to one peer, which might not be the case with xmit_more since >> we have one Qdisc for several clients in case we are in AP mode. >> Building an A-MSDU for several clients is not possible, at least not for >> several client in the L2 (different MAC addresses). >> LSO avoids this problem completely. > > Then, simply calling skb_gso_segment() from the driver might be enough, > and less work for you. > > This would even support TSO on IPv6 > Well... I did take care of IPv6. > segs = skb_gso_segment(skb, tp->dev->features & > ~(NETIF_F_TSO | NETIF_F_TSO6)); > > Thing is that our HW layers are currently implemented to receive one skb per 802.11 packet. So that if I call skb_gso_segment, I'd have to re-assemble the segs into one A-MSDU which would translate one skb. I guess I could change the HW layer in the driver to be able to get a list of skbs and make a single packet out of it, but that'd be tricky or wasteful. skb_gso_segment will duplicate the wifi header while it is not needed. Only the TCP / IP / SNAP headers need to be duplicated. Moreover, each subframe in the A-MSDU needs it own subframe header (same format as ethhdr) and there is also some padding in there. So that would be even more complicated IMHO. My code doesn't copy any payload. Only the headers. This is why I thought it'd be better than segmenting and then re-assembling. I did call skb_gso_segment if I get lots of payload in the header (more than 2 * mss) in order to simplify the implementation. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
