On Wed, Aug 29, 2018 at 9:13 PM Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > > On Wed, 2018-08-29 at 21:12 +0200, Lorenzo Bianconi wrote: > > > That has been fixed by Sara's patch since now we add the pad directly > > on the head of the > > subsequent subframes (not to the tail of the first one). Am I missing something? > > As far as the pad is concerned, yes, but I think the header length also > changes? Ops, maybe I got your point: ieee80211_amsdu_realloc_pad() in ieee80211_amsdu_prepare_head() can expand the headroom on the first frame but if ieee80211_amsdu_realloc_pad() on the second one fails, we do not take into account the extra len added on the first subframe. Is that what you mean? > > I'm too tired now - will think about it again tomorrow. > ack :) Regards, Lorenzo > johannes
