diff options
| author | Felix Fietkau <nbd@nbd.name> | 2019-10-07 13:32:14 +0300 |
|---|---|---|
| committer | Felix Fietkau <nbd@nbd.name> | 2019-11-20 15:23:50 +0300 |
| commit | 1a817fa73c3b27a593aadf0029de24db1bbc1a3e (patch) | |
| tree | 2f5f7e93dc9a596d0eca175c97855e277177b3bc /drivers/net/wireless/mediatek/mt76/mt7603/main.c | |
| parent | 36f7e2b2bb1de86f0072cd49ca93d82b9e8fd894 (diff) | |
| download | linux-1a817fa73c3b27a593aadf0029de24db1bbc1a3e.tar.xz | |
mt76: add missing locking around ampdu action
This is needed primarily to avoid races in dealing with rx aggregation
related data structures
Signed-off-by: Felix Fietkau <nbd@nbd.name>
Diffstat (limited to 'drivers/net/wireless/mediatek/mt76/mt7603/main.c')
| -rw-r--r-- | drivers/net/wireless/mediatek/mt76/mt7603/main.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/wireless/mediatek/mt76/mt7603/main.c b/drivers/net/wireless/mediatek/mt76/mt7603/main.c index 31cce1c005c6..a0632ca198f1 100644 --- a/drivers/net/wireless/mediatek/mt76/mt7603/main.c +++ b/drivers/net/wireless/mediatek/mt76/mt7603/main.c @@ -575,6 +575,7 @@ mt7603_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif, mtxq = (struct mt76_txq *)txq->drv_priv; + mutex_lock(&dev->mt76.mutex); switch (action) { case IEEE80211_AMPDU_RX_START: mt76_rx_aggr_start(&dev->mt76, &msta->wcid, tid, ssn, @@ -603,6 +604,7 @@ mt7603_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif, ieee80211_stop_tx_ba_cb_irqsafe(vif, sta->addr, tid); break; } + mutex_unlock(&dev->mt76.mutex); return 0; } |