Hi Deren, On Mon, May 15, 2023 at 8:20 AM Deren Wu <deren.wu@xxxxxxxxxxxx> wrote: > > Get missing tx retries/failed counts from txfree done events and report > them via mt7921_sta_statistics(). > > Signed-off-by: Deren Wu <deren.wu@xxxxxxxxxxxx> Reviewed-by: Sean Wang <sean.wang@xxxxxxxxxxxx> > --- > v1: based on "[v3,1/6] wifi: mt76: mt7915: report tx retries/failed counts for non-WED path" > https://patchwork.kernel.org/project/linux-wireless/patch/e3cddf1cff5f060478c2de9e4e4021541549e750.1683670255.git.ryder.lee@xxxxxxxxxxxx/ > --- > drivers/net/wireless/mediatek/mt76/mt76_connac2_mac.h | 2 +- > drivers/net/wireless/mediatek/mt76/mt7921/mac.c | 8 +++++++- > drivers/net/wireless/mediatek/mt76/mt7921/main.c | 6 ++++++ > 3 files changed, 14 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/mediatek/mt76/mt76_connac2_mac.h b/drivers/net/wireless/mediatek/mt76/mt76_connac2_mac.h > index a5ec0f631385..cb612d7c7616 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt76_connac2_mac.h > +++ b/drivers/net/wireless/mediatek/mt76/mt76_connac2_mac.h > @@ -34,7 +34,7 @@ enum { > > #define MT_TX_FREE_MSDU_CNT GENMASK(9, 0) > #define MT_TX_FREE_WLAN_ID GENMASK(23, 14) > -#define MT_TX_FREE_LATENCY GENMASK(12, 0) > +#define MT_TX_FREE_COUNT GENMASK(12, 0) > /* 0: success, others: dropped */ > #define MT_TX_FREE_STATUS GENMASK(14, 13) > #define MT_TX_FREE_MSDU_ID GENMASK(30, 16) > diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/mac.c b/drivers/net/wireless/mediatek/mt76/mt7921/mac.c > index 1675bf520481..a3d31b69217b 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7921/mac.c > +++ b/drivers/net/wireless/mediatek/mt76/mt7921/mac.c > @@ -614,6 +614,7 @@ static void mt7921_mac_tx_free(struct mt7921_dev *dev, void *data, int len) > struct mt76_dev *mdev = &dev->mt76; > struct mt76_txwi_cache *txwi; > struct ieee80211_sta *sta = NULL; > + struct mt76_wcid *wcid = NULL; > struct sk_buff *skb, *tmp; > void *end = data + len; > LIST_HEAD(free_list); > @@ -637,7 +638,6 @@ static void mt7921_mac_tx_free(struct mt7921_dev *dev, void *data, int len) > */ > if (info & MT_TX_FREE_PAIR) { > struct mt7921_sta *msta; > - struct mt76_wcid *wcid; > u16 idx; > > count++; > @@ -658,6 +658,12 @@ static void mt7921_mac_tx_free(struct mt7921_dev *dev, void *data, int len) > msdu = FIELD_GET(MT_TX_FREE_MSDU_ID, info); > stat = FIELD_GET(MT_TX_FREE_STATUS, info); > > + if (wcid) { > + wcid->stats.tx_retries += > + FIELD_GET(MT_TX_FREE_COUNT, info) - 1; > + wcid->stats.tx_failed += !!stat; > + } > + > txwi = mt76_token_release(mdev, msdu, &wake); > if (!txwi) > continue; > diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/main.c b/drivers/net/wireless/mediatek/mt76/mt7921/main.c > index 0c9a472bc81a..418a1fa67477 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7921/main.c > +++ b/drivers/net/wireless/mediatek/mt76/mt7921/main.c > @@ -1400,6 +1400,12 @@ static void mt7921_sta_statistics(struct ieee80211_hw *hw, > sinfo->txrate.he_dcm = txrate->he_dcm; > sinfo->txrate.he_ru_alloc = txrate->he_ru_alloc; > } > + sinfo->tx_failed = msta->wcid.stats.tx_failed; > + sinfo->filled |= BIT_ULL(NL80211_STA_INFO_TX_FAILED); > + > + sinfo->tx_retries = msta->wcid.stats.tx_retries; > + sinfo->filled |= BIT_ULL(NL80211_STA_INFO_TX_RETRIES); > + > sinfo->txrate.flags = txrate->flags; > sinfo->filled |= BIT_ULL(NL80211_STA_INFO_TX_BITRATE); > > -- > 2.18.0 >
