From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9E06AC433F5 for ; Thu, 21 Oct 2021 07:57:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 7749E60FD8 for ; Thu, 21 Oct 2021 07:57:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231287AbhJUH70 (ORCPT ); Thu, 21 Oct 2021 03:59:26 -0400 Received: from mail.kernel.org ([198.145.29.99]:46024 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231133AbhJUH7Z (ORCPT ); Thu, 21 Oct 2021 03:59:25 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 2FECE60F6E; Thu, 21 Oct 2021 07:57:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1634803029; bh=TwkTbAm5NstR7g5m79874wYff81iXSsYOZknHgEBhhA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=p+6x3pYT1VtHZTNQ5P1YfrWMQUPxncHxxFIJEjwjvX8Qn5tk0w6xptCVuhtE7u/bc ZdAnfweeFhrZngPBzKCmD0vVR30f2MH+IWM5kVQWq4G7vvo7TmUS4V6emu/qoAmaOA Fc17aqzXCIbzsdxxmhQOC9cdpAzgR+DXGnFdLZCIXRgXvHSeXruV6q2IGaDVMYlmmT v7udn9lAx01mfAbub/K5XF7tkt/z3m4SUN2GAvCRSYUCBfcPG2eSgUovbGfKGNFOiJ TzA12a1TOVw5A6QI/yQXXgqVviZBqPtbdIJNMLRirQD/kALtWqgFkxt/HoL+09J8GF OZC+LdZKGAWTg== Date: Thu, 21 Oct 2021 09:57:05 +0200 From: Lorenzo Bianconi To: Shayne Chen Cc: Felix Fietkau , linux-wireless , Lorenzo Bianconi , Ryder Lee , Evelyn Tsai , linux-mediatek Subject: Re: [PATCH v4 2/2] mt76: mt7915: add debugfs knobs for MCU utilization Message-ID: References: <20211021060726.9291-1-shayne.chen@mediatek.com> <20211021060726.9291-2-shayne.chen@mediatek.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="sn5BZ59whcT+g5rT" Content-Disposition: inline In-Reply-To: <20211021060726.9291-2-shayne.chen@mediatek.com> Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org --sn5BZ59whcT+g5rT Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable > Add debugfs knobs to read MCU utilization, which helps user know > firmware status more easily to narrow down CPU bottleneck issues. >=20 > Co-developed-by: Ryder Lee > Signed-off-by: Ryder Lee > Signed-off-by: Shayne Chen > --- > v2: abbreviate utilization to util > v3: rebase to staging tree > v4: rebase to staging tree > --- > .../wireless/mediatek/mt76/mt7915/debugfs.c | 49 ++++++++++++++++++- > .../net/wireless/mediatek/mt76/mt7915/mcu.c | 5 +- > .../net/wireless/mediatek/mt76/mt7915/mcu.h | 2 + > .../net/wireless/mediatek/mt76/mt7915/mmio.c | 3 ++ > .../wireless/mediatek/mt76/mt7915/mt7915.h | 1 + > .../net/wireless/mediatek/mt76/mt7915/regs.h | 17 +++++++ > 6 files changed, 73 insertions(+), 4 deletions(-) >=20 > diff --git a/drivers/net/wireless/mediatek/mt76/mt7915/debugfs.c b/driver= s/net/wireless/mediatek/mt76/mt7915/debugfs.c > index a829014..6823926 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7915/debugfs.c > +++ b/drivers/net/wireless/mediatek/mt76/mt7915/debugfs.c > @@ -100,6 +100,12 @@ mt7915_fw_debug_wm_set(void *data, u64 val) > for (debug =3D DEBUG_TXCMD; debug <=3D DEBUG_RPT_RX; debug++) > mt7915_mcu_fw_dbg_ctrl(dev, debug, !!dev->fw_debug_wm); > =20 > + /* WM CPU info record control */ > + mt76_clear(dev, MT_CPU_UTIL_CTRL, BIT(0)); > + mt76_wr(dev, MT_DIC_CMD_REG_CMD, BIT(2) | BIT(13) | !dev->fw_debug_wm); > + mt76_wr(dev, MT_MCU_WM_CIRQ_IRQ_MASK_CLR_ADDR, BIT(5)); > + mt76_wr(dev, MT_MCU_WM_CIRQ_IRQ_SOFT_ADDR, BIT(5)); > + > return 0; > } > =20 > @@ -123,7 +129,11 @@ mt7915_fw_debug_wa_set(void *data, u64 val) > =20 > dev->fw_debug_wa =3D val ? MCU_FW_LOG_TO_HOST : 0; > =20 > - return mt7915_mcu_fw_log_2_host(dev, MCU_FW_LOG_WA, dev->fw_debug_wa); > + mt7915_mcu_fw_log_2_host(dev, MCU_FW_LOG_WA, dev->fw_debug_wa); > + mt7915_mcu_wa_cmd(dev, MCU_WA_PARAM_CMD(SET), MCU_WA_PARAM_PDMA_RX, > + !!dev->fw_debug_wa, 0); I guess we should not drop cmd result here (for both mt7915_mcu_fw_log_2_ho= st and mt7915_mcu_wa_cmd) > + > + return 0; > } > =20 > static int > @@ -139,6 +149,39 @@ mt7915_fw_debug_wa_get(void *data, u64 *val) > DEFINE_DEBUGFS_ATTRIBUTE(fops_fw_debug_wa, mt7915_fw_debug_wa_get, > mt7915_fw_debug_wa_set, "%lld\n"); > =20 > +static int > +mt7915_fw_util_wm_show(struct seq_file *file, void *data) > +{ > + struct mt7915_dev *dev =3D file->private; > + > + if (dev->fw_debug_wm) { > + seq_printf(file, "Busy: %u%% Peak busy: %u%%\n", > + mt76_rr(dev, MT_CPU_UTIL_BUSY_PCT), > + mt76_rr(dev, MT_CPU_UTIL_PEAK_BUSY_PCT)); > + seq_printf(file, "Idle count: %u Peak idle count: %u\n", > + mt76_rr(dev, MT_CPU_UTIL_IDLE_CNT), > + mt76_rr(dev, MT_CPU_UTIL_PEAK_IDLE_CNT)); > + } > + > + return 0; > +} > + > +DEFINE_SHOW_ATTRIBUTE(mt7915_fw_util_wm); > + > +static int > +mt7915_fw_util_wa_show(struct seq_file *file, void *data) > +{ > + struct mt7915_dev *dev =3D file->private; > + > + if (dev->fw_debug_wa) > + mt7915_mcu_wa_cmd(dev, MCU_WA_PARAM_CMD(QUERY), > + MCU_WA_PARAM_CPU_UTIL, 0, 0); > + same here Regards, Lorenzo > + return 0; > +} > + > +DEFINE_SHOW_ATTRIBUTE(mt7915_fw_util_wa); > + > static void > mt7915_ampdu_stat_read_phy(struct mt7915_phy *phy, > struct seq_file *file) > @@ -485,6 +528,10 @@ int mt7915_init_debugfs(struct mt7915_phy *phy) > debugfs_create_file("tx_stats", 0400, dir, phy, &mt7915_tx_stats_fops); > debugfs_create_file("fw_debug_wm", 0600, dir, dev, &fops_fw_debug_wm); > debugfs_create_file("fw_debug_wa", 0600, dir, dev, &fops_fw_debug_wa); > + debugfs_create_file("fw_util_wm", 0400, dir, dev, > + &mt7915_fw_util_wm_fops); > + debugfs_create_file("fw_util_wa", 0400, dir, dev, > + &mt7915_fw_util_wa_fops); > debugfs_create_file("implicit_txbf", 0600, dir, dev, > &fops_implicit_txbf); > debugfs_create_file("txpower_sku", 0400, dir, phy, > diff --git a/drivers/net/wireless/mediatek/mt76/mt7915/mcu.c b/drivers/ne= t/wireless/mediatek/mt76/mt7915/mcu.c > index 70193bb..56ed550 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7915/mcu.c > +++ b/drivers/net/wireless/mediatek/mt76/mt7915/mcu.c > @@ -416,8 +416,7 @@ exit: > return mt76_tx_queue_skb_raw(dev, mdev->q_mcu[qid], skb, 0); > } > =20 > -static int > -mt7915_mcu_wa_cmd(struct mt7915_dev *dev, int cmd, u32 a1, u32 a2, u32 a= 3) > +int mt7915_mcu_wa_cmd(struct mt7915_dev *dev, int cmd, u32 a1, u32 a2, u= 32 a3) > { > struct { > __le32 args[3]; > @@ -429,7 +428,7 @@ mt7915_mcu_wa_cmd(struct mt7915_dev *dev, int cmd, u3= 2 a1, u32 a2, u32 a3) > }, > }; > =20 > - return mt76_mcu_send_msg(&dev->mt76, cmd, &req, sizeof(req), true); > + return mt76_mcu_send_msg(&dev->mt76, cmd, &req, sizeof(req), false); > } > =20 > static void > diff --git a/drivers/net/wireless/mediatek/mt76/mt7915/mcu.h b/drivers/ne= t/wireless/mediatek/mt76/mt7915/mcu.h > index 855fbee..f35fa97 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7915/mcu.h > +++ b/drivers/net/wireless/mediatek/mt76/mt7915/mcu.h > @@ -310,6 +310,8 @@ enum { > }; > =20 > enum { > + MCU_WA_PARAM_PDMA_RX =3D 0x04, > + MCU_WA_PARAM_CPU_UTIL =3D 0x0b, > MCU_WA_PARAM_RED =3D 0x0e, > }; > =20 > diff --git a/drivers/net/wireless/mediatek/mt76/mt7915/mmio.c b/drivers/n= et/wireless/mediatek/mt76/mt7915/mmio.c > index 7bbb38e..1f6ba30 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7915/mmio.c > +++ b/drivers/net/wireless/mediatek/mt76/mt7915/mmio.c > @@ -34,6 +34,9 @@ static u32 __mt7915_reg_addr(struct mt7915_dev *dev, u3= 2 addr) > u32 mapped; > u32 size; > } fixed_map[] =3D { > + { 0x00400000, 0x80000, 0x10000 }, /* WF_MCU_SYSRAM */ > + { 0x00410000, 0x90000, 0x10000 }, /* WF_MCU_SYSRAM (configure regs) */ > + { 0x40000000, 0x70000, 0x10000 }, /* WF_UMAC_SYSRAM */ > { 0x54000000, 0x02000, 0x1000 }, /* WFDMA PCIE0 MCU DMA0 */ > { 0x55000000, 0x03000, 0x1000 }, /* WFDMA PCIE0 MCU DMA1 */ > { 0x58000000, 0x06000, 0x1000 }, /* WFDMA PCIE1 MCU DMA0 (MEM_DMA) */ > diff --git a/drivers/net/wireless/mediatek/mt76/mt7915/mt7915.h b/drivers= /net/wireless/mediatek/mt76/mt7915/mt7915.h > index 070950b..b318b3d 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7915/mt7915.h > +++ b/drivers/net/wireless/mediatek/mt76/mt7915/mt7915.h > @@ -450,6 +450,7 @@ int mt7915_mcu_get_rx_rate(struct mt7915_phy *phy, st= ruct ieee80211_vif *vif, > struct ieee80211_sta *sta, struct rate_info *rate); > int mt7915_mcu_rdd_cmd(struct mt7915_dev *dev, enum mt7915_rdd_cmd cmd, > u8 index, u8 rx_sel, u8 val); > +int mt7915_mcu_wa_cmd(struct mt7915_dev *dev, int cmd, u32 a1, u32 a2, u= 32 a3); > int mt7915_mcu_fw_log_2_host(struct mt7915_dev *dev, u8 type, u8 ctrl); > int mt7915_mcu_fw_dbg_ctrl(struct mt7915_dev *dev, u32 module, u8 level); > void mt7915_mcu_rx_event(struct mt7915_dev *dev, struct sk_buff *skb); > diff --git a/drivers/net/wireless/mediatek/mt76/mt7915/regs.h b/drivers/n= et/wireless/mediatek/mt76/mt7915/regs.h > index 99574ed..ff71f56 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7915/regs.h > +++ b/drivers/net/wireless/mediatek/mt76/mt7915/regs.h > @@ -528,6 +528,18 @@ > #define MT_HIF_REMAP_L2_BASE GENMASK(31, 12) > #define MT_HIF_REMAP_BASE_L2 0x00000 > =20 > +#define MT_DIC_CMD_REG_BASE 0x41f000 > +#define MT_DIC_CMD_REG(ofs) (MT_DIC_CMD_REG_BASE + (ofs)) > +#define MT_DIC_CMD_REG_CMD MT_DIC_CMD_REG(0x10) > + > +#define MT_CPU_UTIL_BASE 0x41f030 > +#define MT_CPU_UTIL(ofs) (MT_CPU_UTIL_BASE + (ofs)) > +#define MT_CPU_UTIL_BUSY_PCT MT_CPU_UTIL(0x00) > +#define MT_CPU_UTIL_PEAK_BUSY_PCT MT_CPU_UTIL(0x04) > +#define MT_CPU_UTIL_IDLE_CNT MT_CPU_UTIL(0x08) > +#define MT_CPU_UTIL_PEAK_IDLE_CNT MT_CPU_UTIL(0x0c) > +#define MT_CPU_UTIL_CTRL MT_CPU_UTIL(0x1c) > + > #define MT_SWDEF_BASE 0x41f200 > #define MT_SWDEF(ofs) (MT_SWDEF_BASE + (ofs)) > #define MT_SWDEF_MODE MT_SWDEF(0x3c) > @@ -591,4 +603,9 @@ > #define MT_WF_PHY_RXTD12_IRPI_SW_CLR_ONLY BIT(18) > #define MT_WF_PHY_RXTD12_IRPI_SW_CLR BIT(29) > =20 > +#define MT_MCU_WM_CIRQ_BASE 0x89010000 > +#define MT_MCU_WM_CIRQ(ofs) (MT_MCU_WM_CIRQ_BASE + (ofs)) > +#define MT_MCU_WM_CIRQ_IRQ_MASK_CLR_ADDR MT_MCU_WM_CIRQ(0x80) > +#define MT_MCU_WM_CIRQ_IRQ_SOFT_ADDR MT_MCU_WM_CIRQ(0xc0) > + > #endif > --=20 > 2.25.1 >=20 --sn5BZ59whcT+g5rT Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQTquNwa3Txd3rGGn7Y6cBh0uS2trAUCYXEdUQAKCRA6cBh0uS2t rDQCAP9BH5QzOWZn0bOjhV7wSu/xsk940L7bu7OuTIl23E2KAgEA3GNW6Rl3NNQb 0HhPkMjevpx1WIysVANrNVe72TZxzwU= =Dqiy -----END PGP SIGNATURE----- --sn5BZ59whcT+g5rT--