Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/136899/?format=api
{ "id": 136899, "url": "https://patches.dpdk.org/api/patches/136899/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20240220105823.570841-3-haijie1@huawei.com/", "project": { "id": 1, "url": "https://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20240220105823.570841-3-haijie1@huawei.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20240220105823.570841-3-haijie1@huawei.com", "date": "2024-02-20T10:58:18", "name": "[v3,2/7] ethdev: add telemetry cmd for registers", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "ee767ccfa588ea2faac2fbc212dc513ab21609fb", "submitter": { "id": 2935, "url": "https://patches.dpdk.org/api/people/2935/?format=api", "name": "Jie Hai", "email": "haijie1@huawei.com" }, "delegate": { "id": 319, "url": "https://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20240220105823.570841-3-haijie1@huawei.com/mbox/", "series": [ { "id": 31148, "url": "https://patches.dpdk.org/api/series/31148/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=31148", "date": "2024-02-20T10:58:16", "name": "support dump reigser names and filter them", "version": 3, "mbox": "https://patches.dpdk.org/series/31148/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/136899/comments/", "check": "success", "checks": "https://patches.dpdk.org/api/patches/136899/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 9986B43B53;\n\tTue, 20 Feb 2024 12:02:52 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id E534840EF1;\n\tTue, 20 Feb 2024 12:02:40 +0100 (CET)", "from szxga06-in.huawei.com (szxga06-in.huawei.com [45.249.212.32])\n by mails.dpdk.org (Postfix) with ESMTP id 86E4040ED1\n for <dev@dpdk.org>; Tue, 20 Feb 2024 12:02:38 +0100 (CET)", "from mail.maildlp.com (unknown [172.19.88.234])\n by szxga06-in.huawei.com (SkyGuard) with ESMTP id 4TfGgt5ycnzqjkF;\n Tue, 20 Feb 2024 19:02:02 +0800 (CST)", "from kwepemd100004.china.huawei.com (unknown [7.221.188.31])\n by mail.maildlp.com (Postfix) with ESMTPS id 8C5B0140133;\n Tue, 20 Feb 2024 19:02:36 +0800 (CST)", "from localhost.localdomain (10.67.165.2) by\n kwepemd100004.china.huawei.com (7.221.188.31) with Microsoft SMTP Server\n (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id\n 15.2.1258.28; Tue, 20 Feb 2024 19:02:35 +0800" ], "From": "Jie Hai <haijie1@huawei.com>", "To": "<dev@dpdk.org>", "CC": "<lihuisong@huawei.com>, <fengchengwen@huawei.com>,\n <liuyonglong@huawei.com>, <huangdengdui@huawei.com>, <ferruh.yigit@amd.com>", "Subject": "[PATCH v3 2/7] ethdev: add telemetry cmd for registers", "Date": "Tue, 20 Feb 2024 18:58:18 +0800", "Message-ID": "<20240220105823.570841-3-haijie1@huawei.com>", "X-Mailer": "git-send-email 2.30.0", "In-Reply-To": "<20240220105823.570841-1-haijie1@huawei.com>", "References": "<20231214015650.3738578-1-haijie1@huawei.com>\n <20240220105823.570841-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": "dggems706-chm.china.huawei.com (10.3.19.183) To\n 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 <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "This patch adds a telemetry command for registers dump,\nand supports get registers with specified names.\nThe length of the string exported by telemetry is limited\nby MAX_OUTPUT_LEN. Therefore, the filter should be more\nprecise.\n\nAn example usage is shown below:\n--> /ethdev/regs,0,INTR\n{\n \"/ethdev/regs\": {\n \"registers_length\": 318,\n \"registers_width\": 4,\n \"register_offset\": \"0x0\",\n \"version\": \"0x1140011\",\n \"group_0\": {\n \"HNS3_CMDQ_INTR_STS_REG\": \"0x0\",\n \"HNS3_CMDQ_INTR_EN_REG\": \"0x2\",\n \"HNS3_CMDQ_INTR_GEN_REG\": \"0x0\",\n \"queue_0_HNS3_TQP_INTR_CTRL_REG\": \"0x0\",\n \"queue_0_HNS3_TQP_INTR_GL0_REG\": \"0xa\",\n \"queue_0_HNS3_TQP_INTR_GL1_REG\": \"0xa\",\n \"queue_0_HNS3_TQP_INTR_GL2_REG\": \"0x0\",\n ...\n },\n \"group_1\": {\n ...\n },\n ...\n}\n\nor as below if the number of registers not exceed the\nRTE_TEL_MAX_DICT_ENTRIES:\n--> /ethdev/regs,0,ppp\n{\n \"/ethdev/regs\": {\n \"registers_length\": 156,\n \"registers_width\": 4,\n \"register_offset\": \"0x0\",\n \"version\": \"0x1140011\",\n \"ppp_key_drop_num\": \"0x0\",\n \"ppp_rlt_drop_num\": \"0x0\",\n \"ssu_ppp_mac_key_num_l\": \"0x1\",\n \"ssu_ppp_mac_key_num_h\": \"0x0\",\n \"ssu_ppp_host_key_num_l\": \"0x1\",\n \"ssu_ppp_host_key_num_h\": \"0x0\",\n \"ppp_ssu_mac_rlt_num_l\": \"0x1\",\n ...\n }\n}\n\nSigned-off-by: Jie Hai <haijie1@huawei.com>\n---\n lib/ethdev/rte_ethdev_telemetry.c | 126 ++++++++++++++++++++++++++++++\n 1 file changed, 126 insertions(+)", "diff": "diff --git a/lib/ethdev/rte_ethdev_telemetry.c b/lib/ethdev/rte_ethdev_telemetry.c\nindex 6b873e7abe68..f1ebb2fae632 100644\n--- a/lib/ethdev/rte_ethdev_telemetry.c\n+++ b/lib/ethdev/rte_ethdev_telemetry.c\n@@ -5,6 +5,7 @@\n #include <ctype.h>\n #include <stdlib.h>\n \n+#include <rte_malloc.h>\n #include <rte_kvargs.h>\n #include <rte_telemetry.h>\n \n@@ -1395,6 +1396,129 @@ eth_dev_handle_port_tm_node_caps(const char *cmd __rte_unused,\n \treturn ret;\n }\n \n+static int\n+eth_dev_store_regs(struct rte_tel_data *d, struct rte_dev_reg_info *reg_info)\n+{\n+\tstruct rte_tel_data *groups[RTE_TEL_MAX_DICT_ENTRIES] = {NULL};\n+\tchar group_name[RTE_TEL_MAX_STRING_LEN] = {0};\n+\tstruct rte_tel_data *group = NULL;\n+\tuint32_t grp_num = 0;\n+\tuint32_t *data;\n+\tint ret = 0;\n+\tuint32_t i;\n+\n+\trte_tel_data_start_dict(d);\n+\trte_tel_data_add_dict_uint(d, \"register_length\", reg_info->length);\n+\trte_tel_data_add_dict_uint(d, \"register_width\", reg_info->width);\n+\trte_tel_data_add_dict_uint_hex(d, \"register_offset\", reg_info->offset, 0);\n+\trte_tel_data_add_dict_uint_hex(d, \"version\", reg_info->version, 0);\n+\n+\tdata = reg_info->data;\n+\tif (reg_info->length <= RTE_TEL_MAX_DICT_ENTRIES) {\n+\t\tfor (i = 0; i < reg_info->length; i++, data++)\n+\t\t\trte_tel_data_add_dict_uint_hex(d,\n+\t\t\t\treg_info->names[i].name, *data, 0);\n+\t\treturn 0;\n+\t}\n+\n+\tfor (i = 0; i < reg_info->length; i++, data++) {\n+\t\tif (i % RTE_TEL_MAX_DICT_ENTRIES == 0) {\n+\t\t\tif (i != 0)\n+\t\t\t\trte_tel_data_add_dict_container(d, group_name,\n+\t\t\t\t\t\t\t\tgroup, 0);\n+\n+\t\t\tgroup = rte_tel_data_alloc();\n+\t\t\tif (group == NULL) {\n+\t\t\t\tret = -ENOMEM;\n+\t\t\t\tgoto out;\n+\t\t\t}\n+\t\t\trte_tel_data_start_dict(group);\n+\t\t\tsnprintf(group_name, RTE_TEL_MAX_STRING_LEN,\n+\t\t\t\t\t\"group_%u\", grp_num);\n+\t\t\tif (grp_num >= RTE_TEL_MAX_DICT_ENTRIES) {\n+\t\t\t\tRTE_ETHDEV_LOG_LINE(NOTICE,\n+\t\t\t\t\t\"Too many regs, please filter\");\n+\t\t\t\treturn 0;\n+\t\t\t}\n+\t\t\tgroups[grp_num++] = group;\n+\t\t}\n+\t\trte_tel_data_add_dict_uint_hex(group, reg_info->names[i].name,\n+\t\t\t\t\t\t*data, 0);\n+\t}\n+\tif (i % RTE_TEL_MAX_DICT_ENTRIES != 0)\n+\t\trte_tel_data_add_dict_container(d, group_name, group, 0);\n+\n+\treturn 0;\n+out:\n+\tfor (i = 0; i < grp_num; i++)\n+\t\trte_tel_data_free(groups[i]);\n+\n+\treturn ret;\n+}\n+\n+static int\n+eth_dev_get_port_regs(int port_id, struct rte_tel_data *d, char *filter)\n+{\n+\tstruct rte_dev_reg_info reg_info;\n+\tint ret;\n+\n+\tmemset(®_info, 0, sizeof(reg_info));\n+\treg_info.filter = filter;\n+\n+\tret = rte_eth_dev_get_reg_info_ext(port_id, ®_info);\n+\tif (ret != 0) {\n+\t\tRTE_ETHDEV_LOG_LINE(ERR,\n+\t\t\t\"Error getting device reg info: %d\", ret);\n+\t\treturn ret;\n+\t}\n+\n+\treg_info.data = calloc(reg_info.length, reg_info.width);\n+\tif (!reg_info.data)\n+\t\treturn -ENOMEM;\n+\n+\treg_info.names = calloc(reg_info.length, sizeof(struct rte_eth_reg_name));\n+\tif (!reg_info.names) {\n+\t\tfree(reg_info.data);\n+\t\treturn -ENOMEM;\n+\t}\n+\n+\tret = rte_eth_dev_get_reg_info_ext(port_id, ®_info);\n+\tif (ret != 0) {\n+\t\tRTE_ETHDEV_LOG_LINE(ERR,\n+\t\t\t\"Error getting regs from device: %d\", ret);\n+\t\tret = -EINVAL;\n+\t\tgoto out;\n+\t}\n+\n+\tret = eth_dev_store_regs(d, ®_info);\n+out:\n+\tfree(reg_info.data);\n+\tfree(reg_info.names);\n+\n+\treturn ret;\n+}\n+\n+static int\n+eth_dev_handle_port_regs(const char *cmd __rte_unused,\n+\t\tconst char *params,\n+\t\tstruct rte_tel_data *d)\n+{\n+\tchar *filter = NULL;\n+\tuint16_t port_id;\n+\tchar *end_param;\n+\tint ret;\n+\n+\tret = eth_dev_parse_port_params(params, &port_id, &end_param, true);\n+\tif (ret != 0)\n+\t\treturn ret;\n+\n+\tfilter = strtok(end_param, \",\");\n+\tif (filter != NULL && strlen(filter) == 0)\n+\t\tfilter = NULL;\n+\n+\treturn eth_dev_get_port_regs(port_id, d, filter);\n+}\n+\n RTE_INIT(ethdev_init_telemetry)\n {\n \trte_telemetry_register_cmd(\"/ethdev/list\", eth_dev_handle_port_list,\n@@ -1436,4 +1560,6 @@ RTE_INIT(ethdev_init_telemetry)\n \t\t\t\"Returns TM Level Capabilities info for a port. Parameters: int port_id, int level_id (see tm_capability for the max)\");\n \trte_telemetry_register_cmd(\"/ethdev/tm_node_capability\", eth_dev_handle_port_tm_node_caps,\n \t\t\t\"Returns TM Node Capabilities info for a port. Parameters: int port_id, int node_id (see tm_capability for the max)\");\n+\trte_telemetry_register_cmd(\"/ethdev/regs\", eth_dev_handle_port_regs,\n+\t\t\t\"Returns regs for a port. Parameters: int port_id, string filter\");\n }\n", "prefixes": [ "v3", "2/7" ] }