rt2x00: do not set IEEE80211_TX_STAT_AMPDU_NO_BACK on tx status
authorStanislaw Gruszka <sgruszka@redhat.com>
Fri, 23 Aug 2019 07:09:56 +0000 (09:09 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 3 Sep 2019 13:41:15 +0000 (16:41 +0300)
commit567a9b766b47caffe4b1bf74823e7bc18532d875
treedae2c7bcc257d7a349617f40609e60f6077b4334
parent14aba89386a4c921844886d97930611fc252cce2
rt2x00: do not set IEEE80211_TX_STAT_AMPDU_NO_BACK on tx status

According to documentation IEEE80211_TX_STAT_AMPDU_NO_BACK is suppose
to be used when we do not recive BA (BlockAck). However on rt2x00 we
use it when remote station fail to decode one or more subframes within
AMPDU (some bits are not set in BlockAck bitmap). Setting the flag result
in sent of BAR (BlockAck Request) frame and this might result of abuse
of BA session, since remote station can sent BA with incorrect
sequence numbers after receiving BAR. This problem is visible especially
when connecting two rt2800 devices.

Previously I observed some performance benefits when using the flag
when connecting with iwlwifi devices. But currently possibly due
to reacent changes in rt2x00 removing the flag has no effect on
those test cases.

So remove the IEEE80211_TX_STAT_AMPDU_NO_BACK.

Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/ralink/rt2x00/rt2x00dev.c