ExternalVendorCode/openwifi
1
0
Fork 0
mirror of https://github.com/open-sdr/openwifi.git synced 2025-01-31 00:24:09 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix the AMPDU reporting bug in tx interrupt:

Browse Source 
When pkt_cnt is 1, it is also possible the aggregation case (single packet aggregation). So the reporting type (non aggregation or aggregation) should be decided automatically based on a dedicated use_ht_aggr flag instead of pkt_cnt, otherwise the Linux minstrel_ht link adaptation can not get correct TX status report in the aggregation case and will keep using the lowest MCS0
This commit is contained in:
Xianjun Jiao 2022-03-29 14:20:38 +02:00
parent 106a5e441f
commit 2c78ef135f
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
driver/sdr.c
View File

@ -561,6 +561,7 @@ static irqreturn_t openwifi_tx_interrupt(int irq, void *dev_id)
u64 blk_ack_bitmap; u64 blk_ack_bitmap;
// u16 prio_rd_idx_store[64]={0}; // u16 prio_rd_idx_store[64]={0};
bool tx_fail=false; bool tx_fail=false;
bool use_ht_aggr;
spin_lock(&priv->lock); spin_lock(&priv->lock);
@ -611,10 +612,11 @@ static irqreturn_t openwifi_tx_interrupt(int irq, void *dev_id)
skb->len, DMA_MEM_TO_DEV); skb->len, DMA_MEM_TO_DEV);
info = IEEE80211_SKB_CB(skb); info = IEEE80211_SKB_CB(skb);
use_ht_aggr = ((info->flags&IEEE80211_TX_CTL_AMPDU)!=0);
ieee80211_tx_info_clear_status(info); ieee80211_tx_info_clear_status(info);
// Aggregation packet // Aggregation packet
if(pkt_cnt > 1) if (use_ht_aggr)
{ {
start_idx = (seq_no>=blk_ack_ssn) ? (seq_no-blk_ack_ssn) : (seq_no+((~blk_ack_ssn+1)&0x0FFF)); start_idx = (seq_no>=blk_ack_ssn) ? (seq_no-blk_ack_ssn) : (seq_no+((~blk_ack_ssn+1)&0x0FFF));
tx_fail = (((blk_ack_bitmap>>start_idx)&0x1)==0); tx_fail = (((blk_ack_bitmap>>start_idx)&0x1)==0);