From: Ramya Gnanasekar <rgnanase@qti.qualcomm.com>
To: "Jeff Johnson (QUIC)" <quic_jjohnson@quicinc.com>,
"Ramya Gnanasekar (Temp) (QUIC)" <quic_rgnanase@quicinc.com>,
"ath12k@lists.infradead.org" <ath12k@lists.infradead.org>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: RE: [PATCH 0/4] wifi: ath12k: Add support to enable debugfs_htt_stats
Date: Tue, 30 Apr 2024 17:25:52 +0000 [thread overview]
Message-ID: <IA1PR02MB95749A2932C41D5F5046BED9F11A2@IA1PR02MB9574.namprd02.prod.outlook.com> (raw)
In-Reply-To: <adb4e20d-a6e5-4508-a3d6-9c46a1aeebaa@quicinc.com>
> -----Original Message-----
> From: Jeff Johnson (QUIC) <quic_jjohnson@quicinc.com>
> Sent: Tuesday, April 30, 2024 3:05 AM
> To: Ramya Gnanasekar (Temp) (QUIC) <quic_rgnanase@quicinc.com>;
> ath12k@lists.infradead.org
> Cc: linux-wireless@vger.kernel.org
> Subject: Re: [PATCH 0/4] wifi: ath12k: Add support to enable debugfs_htt_stats
>
> On 4/29/2024 10:38 AM, Ramya Gnanasekar wrote:
> > Bring in the basic infrastructure necessary for enabling htt_stats via debugfs.
> > Patch series bring support to request stats type to firmware, dump the
> > stats and request to reset the stats from firmware.
> >
> > Schema with one ath12k device:
> >
> > ath12k
> > `-- pci-0000:06:00.0
> > |-- mac0
> > `-- htt_stats
> > |-- htt_stats_type
> > |-- htt_stats_reset
> >
> > Dinesh Karthikeyan (3):
> > wifi: ath12k: Add support to enable debugfs_htt_stats
> > wifi: ath12k: Add htt_stats_dump file ops support
> > wifi: ath12k: Add support to parse requested stats_type
> >
> > Ramya Gnanasekar (1):
> > wifi: ath12k: Dump additional Tx PDEV HTT stats
> >
> > drivers/net/wireless/ath/ath12k/Makefile | 2 +-
> > drivers/net/wireless/ath/ath12k/core.h | 11 +
> > drivers/net/wireless/ath/ath12k/debugfs.c | 3 +
> > .../wireless/ath/ath12k/debugfs_htt_stats.c | 713 ++++++++++++++++++
> > .../wireless/ath/ath12k/debugfs_htt_stats.h | 300 ++++++++
> > drivers/net/wireless/ath/ath12k/dp_rx.c | 10 +-
> > drivers/net/wireless/ath/ath12k/dp_rx.h | 4 +
> > 7 files changed, 1038 insertions(+), 5 deletions(-) create mode
> > 100644 drivers/net/wireless/ath/ath12k/debugfs_htt_stats.c
> > create mode 100644
> > drivers/net/wireless/ath/ath12k/debugfs_htt_stats.h
> >
> >
> > base-commit: 326f8f68f28b0b831233acfabffb486a5b0f4717
>
> testing on my laptop
>
> good news is the files are being populated:
> # ls /sys/kernel/debug/ath12k/pci-0000\:03\:00.0/mac0/
> dfs_simulate_radar htt_stats htt_stats_reset htt_stats_type
>
> bad news is:
> # echo 1 > /sys/kernel/debug/ath12k/pci-0000\:03\:00.0/mac0/htt_stats_type
> # cat /sys/kernel/debug/ath12k/pci-0000\:03\:00.0/mac0/htt_stats
> cat: '/sys/kernel/debug/ath12k/pci-0000:03:00.0/mac0/htt_stats': Connection
> timed out
>
> journalctl shows:
> Apr 29 14:17:16 qca-HP-ZBook-14-G2 kernel: ath12k_pci 0000:03:00.0: htc rx
> completion ep 2 skb ffff8c4e50e5ed00 Apr 29 14:17:19 qca-HP-ZBook-14-G2
> kernel: ath12k_pci 0000:03:00.0: stats request timed out
>
> so something isn't right, at least for WCN7850
>
> has this been tested with upstream kernel?
Tested on QCN9274 upstream kernel. HTT stats event is not received by host upon stats request.
Will check further on this.
prev parent reply other threads:[~2024-04-30 17:26 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-29 17:38 [PATCH 0/4] wifi: ath12k: Add support to enable debugfs_htt_stats Ramya Gnanasekar
2024-04-29 17:38 ` [PATCH 1/4] " Ramya Gnanasekar
2024-04-29 17:38 ` [PATCH 2/4] wifi: ath12k: Add htt_stats_dump file ops support Ramya Gnanasekar
2024-04-29 17:38 ` [PATCH 3/4] wifi: ath12k: Add support to parse requested stats_type Ramya Gnanasekar
2024-04-29 17:38 ` [PATCH 4/4] wifi: ath12k: Dump additional Tx PDEV HTT stats Ramya Gnanasekar
2024-04-29 21:34 ` [PATCH 0/4] wifi: ath12k: Add support to enable debugfs_htt_stats Jeff Johnson
2024-04-30 17:25 ` Ramya Gnanasekar [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=IA1PR02MB95749A2932C41D5F5046BED9F11A2@IA1PR02MB9574.namprd02.prod.outlook.com \
--to=rgnanase@qti.qualcomm.com \
--cc=ath12k@lists.infradead.org \
--cc=linux-wireless@vger.kernel.org \
--cc=quic_jjohnson@quicinc.com \
--cc=quic_rgnanase@quicinc.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).