Wen Gong <wgong@xxxxxxxxxxxxxx> wrote: > When tx complete is disabled, all tx status will be set with status > HTT_TX_COMPL_STATE_ACK and indicate to mac80211 by ieee80211_tx_status, > then it does not have the statistics for retries and failed packets. > count of tx retries and tx failed of command "iw wlan0 station dump" > are both 0. If tx complete is not disabled, then firmware report the > tx status and ath10k indicate the status to mac80211, then mac80211 > save the statistics and command "iw wlan0 station dump" show them. > > for example: > localhost ~ # iw dev wlan0 station dump > Station 3c:28:6d:96:fd:69 (on wlan0) > inactive time: 5 ms > rx bytes: 1325012 > rx packets: 6477 > tx bytes: 85264 > tx packets: 518 > tx retries: 0 > tx failed: 0 > > This patch only effect chips with tx complete disabled, e.g. SDIO. > > with this patch, output of command "iw dev wlan0 station dump": > Station c4:04:15:5d:97:22 (on wlan0) > inactive time: 608 ms > rx bytes: 180366 > rx packets: 991 > tx bytes: 98765577 > tx packets: 64624 > tx retries: 14682 > tx failed: 47086 > > Tested with QCA6174 SDIO with firmware WLAN.RMH.4.4.1-00042. > > Signed-off-by: Wen Gong <wgong@xxxxxxxxxxxxxx> > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> Patch applied to ath-next branch of ath.git, thanks. 59a022cc14cf ath10k: add statistics of tx retries and tx failed when tx complete disable -- https://patchwork.kernel.org/patch/11504903/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
