On 9/13/2024 1:29 AM, Roopni Devanathan wrote: > From: Dinesh Karthikeyan <quic_dinek@xxxxxxxxxxx> > > 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@xxxxxxxxxxx> > Signed-off-by: Roopni Devanathan <quic_rdevanat@xxxxxxxxxxx> Acked-by: Jeff Johnson <quic_jjohnson@xxxxxxxxxxx> > --- > 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