[PATCH 3/4] wifi: ath12k: Support pdev Transmit Multi-user stats

[Jeff Johnson]

On 9/13/2024 1:29 AM, Roopni Devanathan wrote:
> From: Dinesh Karthikeyan <quic_dinek at quicinc.com>
> 
> Add support to request multi-user stats from firmware for transmitter
> through HTT stats type 17. These stats give information about multi-
> user MIMO, OFDMA and MPDU for different Wi-Fi standards.
> 
> Sample output:
> -------------
> echo 17 > /sys/kernel/debug/ath12k/pci-0000\:06\:00.0/mac0/htt_stats_type
> cat /sys/kernel/debug/ath12k/pci-0000\:06\:00.0/mac0/htt_stats
> HTT_TX_PDEV_MU_MIMO_SCH_STATS_TLV:
> mu_mimo_sch_posted = 0
> mu_mimo_sch_failed = 0
> mu_mimo_ppdu_posted = 0
> 
> ac_mu_mimo_sch_posted_per_group_index 0 (SU) = 0
> ac_mu_mimo_sch_posted_per_group_index 1 (TOTAL STREAMS = 2) = 0
> .....
> 
> ax_mu_mimo_sch_posted_per_group_index 0 (SU) = 0
> ax_mu_mimo_sch_posted_per_group_index 1 (TOTAL STREAMS = 2) = 0
> .....
> 
> be_mu_mimo_sch_posted_per_group_index 0 (SU) = 0
> be_mu_mimo_sch_posted_per_group_index 1 (TOTAL STREAMS = 2) = 0
> .....
> 
> 11ac MU_MIMO SCH STATS:
> ac_mu_mimo_sch_nusers_0 = 0
> ac_mu_mimo_sch_nusers_1 = 0
> ac_mu_mimo_sch_nusers_2 = 0
> ac_mu_mimo_sch_nusers_3 = 0
> 
> 11ax MU_MIMO SCH STATS:
> ax_mu_mimo_sch_nusers_0 = 0
> ax_mu_mimo_sch_nusers_1 = 0
> ax_mu_mimo_sch_nusers_2 = 0
> .....
> 
> 11be MU_MIMO SCH STATS:
> be_mu_mimo_sch_nusers_0 = 0
> be_mu_mimo_sch_nusers_1 = 0
> be_mu_mimo_sch_nusers_2 = 0
> .....
> 
> 11ax OFDMA SCH STATS:
> ax_ofdma_sch_nusers_0 = 0
> ax_ul_ofdma_basic_sch_nusers_0 = 0
> ax_ul_ofdma_bsr_sch_nusers_0 = 0
> ax_ul_ofdma_bar_sch_nusers_0 = 0
> ax_ul_ofdma_brp_sch_nusers_0 = 0
> .....
> 
> 11ax UL MUMIMO SCH STATS:
> ax_ul_mumimo_basic_sch_nusers_0 = 0
> ax_ul_mumimo_brp_sch_nusers_0 = 0
> 
> ax_ul_mumimo_basic_sch_nusers_1 = 0
> ax_ul_mumimo_brp_sch_nusers_1 = 0
> .....
> 
> HTT_TX_PDEV_MUMIMO_GRP_STATS:
> dl_mumimo_grp_tputs_observed (per bin = 300 mbps) =  0:0, 1:0, 2:0, 3:0,
> 4:0, 5:0, 6:0, 7:0, 8:0, 9:0
> dl_mumimo_grp eligible =  0:0, 1:0, 2:0, 3:0, 4:0, 5:0, 6:0, 7:0
> dl_mumimo_grp_ineligible =  0:0, 1:0, 2:0, 3:0, 4:0, 5:0, 6:0, 7:0
> .....
> 
> HTT_TX_PDEV_MU_MIMO_AC_MPDU_STATS:
> ac_mu_mimo_mpdus_queued_usr_0 = 0
> ac_mu_mimo_mpdus_tried_usr_0 = 0
> ac_mu_mimo_mpdus_failed_usr_0 = 0
> .....
> 
> HTT_TX_PDEV_MU_MIMO_AX_MPDU_STATS:
> ax_mu_mimo_mpdus_queued_usr_0 = 0
> ax_mu_mimo_mpdus_tried_usr_0 = 0
> ax_mu_mimo_mpdus_failed_usr_0 = 0
> .....
> 
> HTT_TX_PDEV_AX_MU_OFDMA_MPDU_STATS:
> ax_mu_ofdma_mpdus_queued_usr_0 = 0
> ax_mu_ofdma_mpdus_tried_usr_0 = 0
> ax_mu_ofdma_mpdus_failed_usr_0 = 0
> .....
> 
> 11ac MU_MIMO SCH STATS:
> ac_mu_mimo_sch_nusers_0 = 0
> ac_mu_mimo_sch_nusers_1 = 0
> ac_mu_mimo_sch_nusers_2 = 0
> ac_mu_mimo_sch_nusers_3 = 0
> .....
> 
> Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.1.1-00214-QCAHKSWPL_SILICONZ-1
> 
> Signed-off-by: Dinesh Karthikeyan <quic_dinek at quicinc.com>
> Signed-off-by: Roopni Devanathan <quic_rdevanat at quicinc.com>
Acked-by: Jeff Johnson <quic_jjohnson at quicinc.com>

> ---
> Note: MCC firmware version -
> WLAN.HMT.1.0-03427-QCAHMTSWPL_V1.0_V2.0_SILICONZ-1.15378.4 does not
> print stats because MCC firmware reports an event, but there are
> no tags or data. The length of the received TLV is 0.

IMO this note should be before the Tested-on tag so that it is part of the
commit text in git

no need to re-submit, Kalle can fixup in the pending branch