Commit e3ca8fa0 authored by Felix Fietkau's avatar Felix Fietkau
Browse files

mt76: do not send BAR frame on tx aggregation flush stop 



There is no need to send a BAR frame after stopping aggregation, and doing
so could lead to sending class 3 frames after deauthentication from an AP

Signed-off-by: default avatarBalakrishna Bandi <b.balakrishna@globaledgesoft.com>
Signed-off-by: default avatarFelix Fietkau <nbd@nbd.name>
parent 58bab0d4

drivers/net/wireless/mediatek/mt76/mt7603/main.c

+0 −1
Original line number Diff line number Diff line
@@ -578,7 +578,6 @@ mt7603_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif, 
	case IEEE80211_AMPDU_TX_STOP_FLUSH:

	case IEEE80211_AMPDU_TX_STOP_FLUSH_CONT:

		mtxq->aggr = false;

		ieee80211_send_bar(vif, sta->addr, tid, mtxq->agg_ssn);

		mt7603_mac_tx_ba_reset(dev, msta->wcid.idx, tid, -1);

		break;

	case IEEE80211_AMPDU_TX_START:

drivers/net/wireless/mediatek/mt76/mt7615/main.c

+0 −1
Original line number Diff line number Diff line
@@ -473,7 +473,6 @@ mt7615_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif, 
	case IEEE80211_AMPDU_TX_STOP_FLUSH:

	case IEEE80211_AMPDU_TX_STOP_FLUSH_CONT:

		mtxq->aggr = false;

		ieee80211_send_bar(vif, sta->addr, tid, mtxq->agg_ssn);

		mt7615_mcu_set_tx_ba(dev, params, 0);

		break;

	case IEEE80211_AMPDU_TX_START:

drivers/net/wireless/mediatek/mt76/mt76x02_util.c

+0 −1
Original line number Diff line number Diff line
@@ -390,7 +390,6 @@ int mt76x02_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif, 
	case IEEE80211_AMPDU_TX_STOP_FLUSH:

	case IEEE80211_AMPDU_TX_STOP_FLUSH_CONT:

		mtxq->aggr = false;

		ieee80211_send_bar(vif, sta->addr, tid, mtxq->agg_ssn);

		break;

	case IEEE80211_AMPDU_TX_START:

		mtxq->agg_ssn = IEEE80211_SN_TO_SEQ(ssn);

CRA Git | Maintained and supported by SUSTech CRA and CCSE