> > On Wed, 2018-08-29 at 21:03 +0200, Lorenzo Bianconi wrote: > > Do not try to aggregate packets in a A-MSDU frame and add A-MSDU header > > on the first packet if max_tx_fragments or max_amsdu_subframes are > > set to one > > > > Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@xxxxxxxxxx> > > --- > > Changes since v1: > > - rebased on top of mac80211 master branch > > - removed ieee80211_amsdu_realloc_pad chunk > > > Don't you still have to account for the changed header length if the > second one fails or something? > 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? Regards, Lorenzo > You said you wanted to account for the pad bytes - which now aren't > there - but didn't that also take the changed header of the first > subframe into account? > > johannes