From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 43CBF43A76; Mon, 5 Feb 2024 11:57:10 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D63D040E8A; Mon, 5 Feb 2024 11:56:32 +0100 (CET) Received: from szxga07-in.huawei.com (szxga07-in.huawei.com [45.249.212.35]) by mails.dpdk.org (Postfix) with ESMTP id EDA8840A8A for ; Mon, 5 Feb 2024 11:56:24 +0100 (CET) Received: from mail.maildlp.com (unknown [172.19.163.17]) by szxga07-in.huawei.com (SkyGuard) with ESMTP id 4TT3D450fGz1Q8hm; Mon, 5 Feb 2024 18:54:28 +0800 (CST) Received: from kwepemd100004.china.huawei.com (unknown [7.221.188.31]) by mail.maildlp.com (Postfix) with ESMTPS id D84791A0172; Mon, 5 Feb 2024 18:56:23 +0800 (CST) Received: from localhost.localdomain (10.67.165.2) by kwepemd100004.china.huawei.com (7.221.188.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.2.1258.28; Mon, 5 Feb 2024 18:56:23 +0800 From: Jie Hai To: CC: , , , , Subject: [PATCH v2 7/7] net/hns3: support filter dump of registers Date: Mon, 5 Feb 2024 18:51:51 +0800 Message-ID: <20240205105151.275591-8-haijie1@huawei.com> X-Mailer: git-send-email 2.30.0 In-Reply-To: <20240205105151.275591-1-haijie1@huawei.com> References: <20231214015650.3738578-1-haijie1@huawei.com> <20240205105151.275591-1-haijie1@huawei.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [10.67.165.2] X-ClientProxiedBy: dggems705-chm.china.huawei.com (10.3.19.182) To kwepemd100004.china.huawei.com (7.221.188.31) X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org This patch supports reporting names of the dfx registers which filtering them by names. Signed-off-by: Jie Hai --- drivers/net/hns3/hns3_regs.c | 271 +++++++++++++++++++++++++++++------ 1 file changed, 224 insertions(+), 47 deletions(-) diff --git a/drivers/net/hns3/hns3_regs.c b/drivers/net/hns3/hns3_regs.c index ac07563ebaec..53b5a2dbe15f 100644 --- a/drivers/net/hns3/hns3_regs.c +++ b/drivers/net/hns3/hns3_regs.c @@ -12,7 +12,8 @@ #define HNS3_64_BIT_REG_SIZE (sizeof(uint64_t) / sizeof(uint32_t)) -static int hns3_get_dfx_reg_cnt(struct hns3_hw *hw, uint32_t *count); +static int hns3_get_dfx_reg_cnt(struct hns3_hw *hw, + uint32_t *count, char *filter); struct direct_reg_list { const char *name; @@ -853,12 +854,41 @@ hns3_get_direct_regs_cnt(const struct direct_reg_list *list, return count; } +static uint32_t +hns3_get_32_64_regs_cnt(struct hns3_hw *hw, char *filter) +{ + uint32_t regs_num_32_bit, regs_num_64_bit; + int ret; + uint32_t i; + uint32_t count = 0; + + ret = hns3_get_regs_num(hw, ®s_num_32_bit, ®s_num_64_bit); + if (ret) { + hns3_err(hw, "fail to get the number of registers, " + "ret = %d.", ret); + return ret; + } + + for (i = 0 ; i < regs_num_32_bit; i++) { + if (filter != NULL && + !strstr(regs_32_bit_list[i].new_name, filter)) + continue; + count++; + } + for (i = 0 ; i < regs_num_64_bit * HNS3_64_BIT_REG_SIZE; i++) { + if (filter != NULL && + !strstr(regs_64_bit_list[i].new_name, filter)) + continue; + count++; + } + return count; +} + static int hns3_get_regs_length(struct hns3_hw *hw, uint32_t *length, char *filter) { struct hns3_adapter *hns = HNS3_DEV_HW_TO_ADAPTER(hw); uint32_t cmdq_cnt, common_cnt, ring_cnt, tqp_intr_cnt; - uint32_t regs_num_32_bit, regs_num_64_bit; uint32_t dfx_reg_cnt; uint32_t len; int ret; @@ -880,16 +910,9 @@ hns3_get_regs_length(struct hns3_hw *hw, uint32_t *length, char *filter) tqp_intr_cnt * hw->intr_tqps_num; if (!hns->is_vf) { - ret = hns3_get_regs_num(hw, ®s_num_32_bit, ®s_num_64_bit); - if (ret) { - hns3_err(hw, "fail to get the number of registers, " - "ret = %d.", ret); - return ret; - } - dfx_reg_cnt = regs_num_32_bit + - regs_num_64_bit * HNS3_64_BIT_REG_SIZE; + dfx_reg_cnt = hns3_get_32_64_regs_cnt(hw, filter); - ret = hns3_get_dfx_reg_cnt(hw, &dfx_reg_cnt); + ret = hns3_get_dfx_reg_cnt(hw, &dfx_reg_cnt, filter); if (ret) { hns3_err(hw, "fail to get the number of dfx registers, " "ret = %d.", ret); @@ -903,19 +926,19 @@ hns3_get_regs_length(struct hns3_hw *hw, uint32_t *length, char *filter) } static int -hns3_get_32_bit_regs(struct hns3_hw *hw, uint32_t regs_num, void *data) +hns3_get_32_bit_regs(struct hns3_hw *hw, void *data) { #define HNS3_32_BIT_REG_RTN_DATANUM 8 #define HNS3_32_BIT_DESC_NODATA_LEN 2 struct hns3_cmd_desc *desc; uint32_t *reg_val = data; uint32_t *desc_data; + uint32_t regs_num; int cmd_num; int i, k, n; int ret; - if (regs_num == 0) - return 0; + regs_num = RTE_DIM(regs_32_bit_list); cmd_num = DIV_ROUND_UP(regs_num + HNS3_32_BIT_DESC_NODATA_LEN, HNS3_32_BIT_REG_RTN_DATANUM); @@ -959,20 +982,66 @@ hns3_get_32_bit_regs(struct hns3_hw *hw, uint32_t regs_num, void *data) return 0; } +static void +hns3_filter_32_bit_regs(struct rte_dev_reg_info *regs, + uint32_t *count, uint32_t *data) +{ + uint32_t *regs_data; + regs_data = regs->data; + regs_data += *count; + uint32_t i; + + for (i = 0; i < RTE_DIM(regs_32_bit_list); i++) { + if (regs->filter != NULL && + !strstr(regs_32_bit_list[i].new_name, regs->filter)) { + data++; + continue; + } + *regs_data++ = *data++; + snprintf(regs->names[(*count)++].name, RTE_ETH_REG_NAME_SIZE, + "%s", regs_32_bit_list[i].new_name); + } +} + static int -hns3_get_64_bit_regs(struct hns3_hw *hw, uint32_t regs_num, void *data) +hns3_get_32_bit_regs_filtered(struct hns3_hw *hw, + struct rte_dev_reg_info *regs, uint32_t *count) +{ + uint32_t *data; + int ret; + + if (count == NULL) + return -EINVAL; + + data = rte_zmalloc(NULL, sizeof(uint32_t) * RTE_DIM(regs_32_bit_list), 0); + if (data == NULL) + return -ENOMEM; + + ret = hns3_get_32_bit_regs(hw, data); + if (ret) { + hns3_err(hw, "Get 32 bit register failed, ret = %d", ret); + rte_free(data); + return ret; + } + + hns3_filter_32_bit_regs(regs, count, data); + return 0; +} + +static int +hns3_get_64_bit_regs(struct hns3_hw *hw, void *data) { #define HNS3_64_BIT_REG_RTN_DATANUM 4 #define HNS3_64_BIT_DESC_NODATA_LEN 1 struct hns3_cmd_desc *desc; uint64_t *reg_val = data; uint64_t *desc_data; + uint32_t regs_num; int cmd_num; int i, k, n; int ret; - if (regs_num == 0) - return 0; + regs_num = RTE_DIM(regs_64_bit_list); cmd_num = DIV_ROUND_UP(regs_num + HNS3_64_BIT_DESC_NODATA_LEN, HNS3_64_BIT_REG_RTN_DATANUM); @@ -1016,6 +1085,52 @@ hns3_get_64_bit_regs(struct hns3_hw *hw, uint32_t regs_num, void *data) return 0; } +static void +hns3_filter_64_bit_regs(struct rte_dev_reg_info *regs, + uint32_t *count, uint32_t *data) +{ + uint32_t *regs_data; + regs_data = regs->data; + regs_data += *count; + uint32_t i; + + for (i = 0; i < RTE_DIM(regs_64_bit_list); i++) { + if (regs->filter != NULL && + !strstr(regs_64_bit_list[i].new_name, regs->filter)) { + data++; + continue; + } + *regs_data++ = *data++; + snprintf(regs->names[(*count)++].name, RTE_ETH_REG_NAME_SIZE, + "%s", regs_64_bit_list[i].new_name); + } +} + +static int +hns3_get_64_bit_regs_filtered(struct hns3_hw *hw, + struct rte_dev_reg_info *regs, uint32_t *count) +{ + uint32_t *data; + int ret = 0; + + if (count == NULL) + return -EINVAL; + + data = rte_zmalloc(NULL, sizeof(uint32_t) * RTE_DIM(regs_64_bit_list), 0); + if (data == NULL) + return -ENOMEM; + + ret = hns3_get_64_bit_regs(hw, data); + if (ret) { + hns3_err(hw, "Get 64 bit register failed, ret = %d", ret); + goto out; + } + + hns3_filter_64_bit_regs(regs, count, data); +out: + rte_free(data); + return 0; +} static uint32_t hns3_direct_access_cmdq_reg(struct hns3_hw *hw, @@ -1107,7 +1222,7 @@ hns3_direct_access_ring_reg(struct hns3_hw *hw, *data++ = hns3_read_dev(hw, ring_reg_list[i].addr + reg_offset); snprintf(regs->names[count++].name, RTE_ETH_REG_NAME_SIZE, - "queue_%u_%s", j, ring_reg_list[i].name); + "queue_%u_%s", j, ring_reg_list[i].name); } } @@ -1137,7 +1252,7 @@ hns3_direct_access_tqp_intr_reg(struct hns3_hw *hw, *data++ = hns3_read_dev(hw, tqp_intr_reg_list[i].addr + reg_offset); snprintf(regs->names[count++].name, RTE_ETH_REG_NAME_SIZE, - "queue_%u_%s", j, tqp_intr_reg_list[i].name); + "queue_%u_%s", j, tqp_intr_reg_list[i].name); } } @@ -1239,31 +1354,48 @@ hns3_dfx_reg_fetch_data(struct hns3_cmd_desc *desc, int bd_num, uint32_t *reg) } static int -hns3_get_dfx_reg_cnt(struct hns3_hw *hw, uint32_t *count) +hns3_get_dfx_reg_cnt(struct hns3_hw *hw, uint32_t *count, char *filter) { int opcode_num = RTE_DIM(hns3_dfx_reg_opcode_list); + uint32_t bd_num, data_len, reg_num = 0; + const struct hns3_reg_entry *regs; uint32_t bd_num_list[opcode_num]; + uint32_t i, j; int ret; - int i; ret = hns3_get_dfx_reg_bd_num(hw, bd_num_list, opcode_num); if (ret) return ret; - for (i = 0; i < opcode_num; i++) - *count += bd_num_list[i] * HNS3_CMD_DESC_DATA_NUM; + for (i = 0; i < (uint32_t)opcode_num; i++) { + bd_num = bd_num_list[i]; + data_len = bd_num * HNS3_CMD_DESC_DATA_NUM; + if (data_len != hns3_dfx_reg_list[i].entry_num) { + hns3_err(hw, "The number of registers(%u) diff from registers list(%u)!\n", + data_len, hns3_dfx_reg_list[i].entry_num); + return -EINVAL; + } + regs = hns3_dfx_reg_list[i].regs; + for (j = 0; j < data_len; j++) { + if (filter != NULL && + !strstr(regs[j].new_name, filter)) + continue; + reg_num++; + } + } + + *count += reg_num; return 0; } static int -hns3_get_dfx_regs(struct hns3_hw *hw, void **data) +hns3_get_dfx_regs(struct hns3_hw *hw, uint32_t *data) { int opcode_num = RTE_DIM(hns3_dfx_reg_opcode_list); uint32_t max_bd_num, bd_num, opcode; uint32_t bd_num_list[opcode_num]; struct hns3_cmd_desc *cmd_descs; - uint32_t *reg_val = (uint32_t *)*data; int ret; int i; @@ -1287,32 +1419,85 @@ hns3_get_dfx_regs(struct hns3_hw *hw, void **data) ret = hns3_dfx_reg_cmd_send(hw, cmd_descs, bd_num, opcode); if (ret) break; - reg_val += hns3_dfx_reg_fetch_data(cmd_descs, bd_num, reg_val); + data += hns3_dfx_reg_fetch_data(cmd_descs, bd_num, data); } rte_free(cmd_descs); - *data = (void *)reg_val; return ret; } +static void +hns3_filter_dfx_regs(struct hns3_hw *hw, struct rte_dev_reg_info *regs, + uint32_t *count, uint32_t *data) +{ + uint32_t *regs_data = regs->data; + const char *name = NULL; + uint32_t i, j, cnt; + + cnt = *count; + regs_data += cnt; + for (i = 0; i < RTE_DIM(hns3_dfx_reg_list); i++) { + for (j = 0; j < hns3_dfx_reg_list[i].entry_num; j++) { + if (hw->revision < PCI_REVISION_ID_HIP09_A && + hns3_dfx_reg_list[i].regs[j].old_name != NULL) + name = hns3_dfx_reg_list[i].regs[j].old_name; + else + name = hns3_dfx_reg_list[i].regs[j].new_name; + + if (regs->filter != NULL && !strstr(name, regs->filter)) { + data++; + continue; + } + + snprintf(regs->names[cnt++].name, + RTE_ETH_REG_NAME_SIZE, "%s", name); + *regs_data++ = *data++; + } + } + *count = cnt; +} + +static int +hns3_get_dfx_regs_filtered(struct hns3_hw *hw, struct rte_dev_reg_info *regs, + uint32_t *count) +{ + uint32_t reg_num = 0; + uint32_t *data; + uint32_t i; + int ret; + + for (i = 0; i < RTE_DIM(hns3_dfx_reg_list); i++) + reg_num += hns3_dfx_reg_list[i].entry_num; + + data = rte_zmalloc(NULL, sizeof(uint32_t) * reg_num, 0); + if (data == NULL) { + hns3_err(hw, "No memory for dfx regs!\n"); + return -ENOMEM; + } + ret = hns3_get_dfx_regs(hw, data); + if (ret != 0) + goto out; + + hns3_filter_dfx_regs(hw, regs, count, data); +out: + rte_free(data); + return ret; +} + int hns3_get_regs(struct rte_eth_dev *eth_dev, struct rte_dev_reg_info *regs) { struct hns3_adapter *hns = eth_dev->data->dev_private; struct hns3_hw *hw = &hns->hw; - uint32_t regs_num_32_bit; - uint32_t regs_num_64_bit; uint32_t count = 0; uint32_t length; - uint32_t *data; int ret; ret = hns3_get_regs_length(hw, &length, regs->filter); if (ret) return ret; - data = regs->data; - if (data == NULL) { + if (regs->data == NULL) { regs->length = length; regs->width = sizeof(uint32_t); return 0; @@ -1326,31 +1511,23 @@ hns3_get_regs(struct rte_eth_dev *eth_dev, struct rte_dev_reg_info *regs) /* fetching per-PF registers values from PF PCIe register space */ count = hns3_direct_access_regs(hw, regs, count); - data += count; if (hns->is_vf) return 0; - ret = hns3_get_regs_num(hw, ®s_num_32_bit, ®s_num_64_bit); - if (ret) { - hns3_err(hw, "Get register number failed, ret = %d", ret); - return ret; - } - - /* fetching PF common registers values from firmware */ - ret = hns3_get_32_bit_regs(hw, regs_num_32_bit, data); - if (ret) { + ret = hns3_get_32_bit_regs_filtered(hw, regs, &count); + if (ret != 0) { hns3_err(hw, "Get 32 bit register failed, ret = %d", ret); return ret; } - data += regs_num_32_bit; - ret = hns3_get_64_bit_regs(hw, regs_num_64_bit, data); - if (ret) { + ret = hns3_get_64_bit_regs_filtered(hw, regs, &count); + if (ret != 0) { hns3_err(hw, "Get 64 bit register failed, ret = %d", ret); return ret; } - data += regs_num_64_bit * HNS3_64_BIT_REG_SIZE; - return hns3_get_dfx_regs(hw, (void **)&data); + ret = hns3_get_dfx_regs_filtered(hw, regs, &count); + regs->length = count; + return 0; } -- 2.30.0