Loic Poulain <loic.poulain@xxxxxxxxxx> wrote: > Since firmware uses its own sequence number counters, we need to > use firmware number as well when mac80211 generates the ADD_BA > request packet. Indeed the firmware sequence counters tend to > slightly drift from the mac80211 ones because of firmware offload > features like ARP responses. This causes the starting sequence > number field of the ADD_BA request to be unaligned, and can possibly > cause issues with strict/picky APs. > > To fix this, we retrieve the current firmware sequence number for > a given TID through the smd_trigger_ba API, and use that number as > replacement of the mac80211 starting sequence number. > > This change also ensures that any issue in the smd *ba procedures > will cause the ba action to properly fail, and remove useless call > to smd_trigger_ba() from IEEE80211_AMPDU_RX_START. > > Signed-off-by: Loic Poulain <loic.poulain@xxxxxxxxxx> > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> Patch applied to ath-next branch of ath.git, thanks. 23cddeb5a770 wcn36xx: Use correct SSN for ADD BA request -- https://patchwork.kernel.org/project/linux-wireless/patch/1637604251-11763-1-git-send-email-loic.poulain@xxxxxxxxxx/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches