Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/123464/?format=api
{ "id": 123464, "url": "http://patches.dpdk.org/api/patches/123464/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20230208132824.775985-6-adwivedi@marvell.com/", "project": { "id": 1, "url": "http://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": "<20230208132824.775985-6-adwivedi@marvell.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20230208132824.775985-6-adwivedi@marvell.com", "date": "2023-02-08T13:28:23", "name": "[v10,5/6] ethdev: add trace points for mtr", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "76f9469659f5fadc74acb020268cdcf2fae04976", "submitter": { "id": 1561, "url": "http://patches.dpdk.org/api/people/1561/?format=api", "name": "Ankur Dwivedi", "email": "adwivedi@marvell.com" }, "delegate": { "id": 319, "url": "http://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20230208132824.775985-6-adwivedi@marvell.com/mbox/", "series": [ { "id": 26891, "url": "http://patches.dpdk.org/api/series/26891/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=26891", "date": "2023-02-08T13:28:18", "name": "add trace points in ethdev library", "version": 10, "mbox": "http://patches.dpdk.org/series/26891/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/123464/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/123464/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 2C1EB41C40;\n\tWed, 8 Feb 2023 14:33:37 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 14CEB40EE1;\n\tWed, 8 Feb 2023 14:33:37 +0100 (CET)", "from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com\n [67.231.156.173])\n by mails.dpdk.org (Postfix) with ESMTP id 3898040141\n for <dev@dpdk.org>; Wed, 8 Feb 2023 14:33:35 +0100 (CET)", "from pps.filterd (m0045851.ppops.net [127.0.0.1])\n by mx0b-0016f401.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id\n 31899nhS002582; Wed, 8 Feb 2023 05:31:05 -0800", "from dc5-exch02.marvell.com ([199.233.59.182])\n by mx0b-0016f401.pphosted.com (PPS) with ESMTPS id 3nm65ktksh-1\n (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT);\n Wed, 08 Feb 2023 05:31:05 -0800", "from DC5-EXCH01.marvell.com (10.69.176.38) by DC5-EXCH02.marvell.com\n (10.69.176.39) with Microsoft SMTP Server (TLS) id 15.0.1497.42;\n Wed, 8 Feb 2023 05:31:03 -0800", "from maili.marvell.com (10.69.176.80) by DC5-EXCH01.marvell.com\n (10.69.176.38) with Microsoft SMTP Server id 15.0.1497.42 via Frontend\n Transport; Wed, 8 Feb 2023 05:31:02 -0800", "from localhost.localdomain (unknown [10.28.36.185])\n by maili.marvell.com (Postfix) with ESMTP id 0C9683F709D;\n Wed, 8 Feb 2023 05:30:40 -0800 (PST)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com;\n h=from : to : cc :\n subject : date : message-id : in-reply-to : references : mime-version :\n content-transfer-encoding : content-type; s=pfpt0220;\n bh=PXq63eF/UlKjraAN/tKqIUMIH1Uyti44QbcGMx5at50=;\n b=XDnR8rdJDqyMLBoYGXuksRBwwhDbptjwVq25rIyhsOhf4nv8ewyc1sNUEu6O5ZlyB2fz\n YX2SZ+5tcZ3l18CcfxzixNlpz5+Fn3DQLYK7WBeCTFpAlCTlD15Vp6zS5og/G31OHJew\n ha+cJJ3ocyRCVo3uhYrkkIh6bE6fNav0ukZtjuhcEopm9P2CDLz73Q2pgUjjfyA4NO6W\n unC0F+zOSsuS/Dwpxx2KdTsLS1wHXm1Du/pAjWTho9ia5bCMH/GMsT0Xii2OtJ1+QdgY\n AosJq9rMdEDjUYnpFaPc4aOIKi3ibDJ36Jo1QtM0GlFirXJPIJvJhAkH/edB5986MLt6 xw==", "From": "Ankur Dwivedi <adwivedi@marvell.com>", "To": "<dev@dpdk.org>", "CC": "<thomas@monjalon.net>, <david.marchand@redhat.com>, <mdr@ashroe.eu>,\n <orika@nvidia.com>, <ferruh.yigit@amd.com>, <chas3@att.com>,\n <humin29@huawei.com>, <linville@tuxdriver.com>,\n <ciara.loftus@intel.com>, <qi.z.zhang@intel.com>, <mw@semihalf.com>,\n <mk@semihalf.com>, <shaibran@amazon.com>, <evgenys@amazon.com>,\n <igorch@amazon.com>, <chandu@amd.com>, <irusskikh@marvell.com>,\n <shepard.siegel@atomicrules.com>, <ed.czeck@atomicrules.com>,\n <john.miller@atomicrules.com>, <ajit.khaparde@broadcom.com>,\n <somnath.kotur@broadcom.com>, <jerinj@marvell.com>,\n <mczekaj@marvell.com>, <sthotton@marvell.com>,\n <srinivasan@marvell.com>, <hkalra@marvell.com>,\n <rahul.lakkireddy@chelsio.com>, <johndale@cisco.com>,\n <hyonkim@cisco.com>, <liudongdong3@huawei.com>,\n <yisen.zhuang@huawei.com>, <xuanziyang2@huawei.com>,\n <cloud.wangxiaoyun@huawei.com>, <zhouguoyang@huawei.com>,\n <simei.su@intel.com>, <wenjun1.wu@intel.com>, <qiming.yang@intel.com>,\n <Yuying.Zhang@intel.com>, <beilei.xing@intel.com>,\n <xiao.w.wang@intel.com>, <jingjing.wu@intel.com>,\n <junfeng.guo@intel.com>, <rosen.xu@intel.com>,\n <ndabilpuram@marvell.com>, <kirankumark@marvell.com>,\n <skori@marvell.com>, <skoteshwar@marvell.com>, <lironh@marvell.com>,\n <zr@semihalf.com>, <radhac@marvell.com>, <vburru@marvell.com>,\n <sedara@marvell.com>, <matan@nvidia.com>, <viacheslavo@nvidia.com>,\n <longli@microsoft.com>, <spinler@cesnet.cz>,\n <chaoyong.he@corigine.com>, <niklas.soderlund@corigine.com>,\n <hemant.agrawal@nxp.com>, <sachin.saxena@oss.nxp.com>,\n <g.singh@nxp.com>, <apeksha.gupta@nxp.com>, <sachin.saxena@nxp.com>,\n <aboyer@pensando.io>, <rmody@marvell.com>, <shshaikh@marvell.com>,\n <dsinghrawat@marvell.com>, <andrew.rybchenko@oktetlabs.ru>,\n <jiawenwu@trustnetic.com>, <jianwang@trustnetic.com>,\n <jbehrens@vmware.com>, <maxime.coquelin@redhat.com>,\n <chenbo.xia@intel.com>, <steven.webster@windriver.com>,\n <matt.peters@windriver.com>, <bruce.richardson@intel.com>,\n <mtetsuyah@gmail.com>, <grive@u256.net>, <jasvinder.singh@intel.com>,\n <cristian.dumitrescu@intel.com>, <jgrajcia@cisco.com>,\n <mb@smartsharesystems.com>, Ankur Dwivedi <adwivedi@marvell.com>", "Subject": "[PATCH v10 5/6] ethdev: add trace points for mtr", "Date": "Wed, 8 Feb 2023 18:58:23 +0530", "Message-ID": "<20230208132824.775985-6-adwivedi@marvell.com>", "X-Mailer": "git-send-email 2.25.1", "In-Reply-To": "<20230208132824.775985-1-adwivedi@marvell.com>", "References": "<20230207063254.401538-1-adwivedi@marvell.com>\n <20230208132824.775985-1-adwivedi@marvell.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Content-Type": "text/plain", "X-Proofpoint-GUID": "AVmt0YB8xAF4E1uWnFoGVumPvzpxfz6y", "X-Proofpoint-ORIG-GUID": "AVmt0YB8xAF4E1uWnFoGVumPvzpxfz6y", "X-Proofpoint-Virus-Version": "vendor=baseguard\n engine=ICAP:2.0.219,Aquarius:18.0.930,Hydra:6.0.562,FMLib:17.11.122.1\n definitions=2023-02-08_05,2023-02-08_02,2022-06-22_01", "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": "Adds trace points for rte_mtr specific functions in ethdev lib.\n\nSigned-off-by: Ankur Dwivedi <adwivedi@marvell.com>\nAcked-by: Sunil Kumar Kori <skori@marvell.com>\n---\n lib/ethdev/ethdev_trace.h | 206 +++++++++++++++++++++++++++++++\n lib/ethdev/ethdev_trace_points.c | 63 ++++++++++\n lib/ethdev/rte_mtr.c | 150 ++++++++++++++++++----\n 3 files changed, 398 insertions(+), 21 deletions(-)", "diff": "diff --git a/lib/ethdev/ethdev_trace.h b/lib/ethdev/ethdev_trace.h\nindex be231b408b..973e2ac6d4 100644\n--- a/lib/ethdev/ethdev_trace.h\n+++ b/lib/ethdev/ethdev_trace.h\n@@ -19,6 +19,7 @@ extern \"C\" {\n #include <rte_trace_point.h>\n \n #include \"rte_ethdev.h\"\n+#include \"rte_mtr.h\"\n \n RTE_TRACE_POINT(\n \trte_ethdev_trace_configure,\n@@ -1655,6 +1656,181 @@ RTE_TRACE_POINT(\n \trte_trace_point_emit_int(ret);\n )\n \n+RTE_TRACE_POINT(\n+\trte_mtr_trace_capabilities_get,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id,\n+\t\tconst struct rte_mtr_capabilities *cap, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_ptr(cap);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_profile_add,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id,\n+\t\tuint32_t meter_profile_id,\n+\t\tconst struct rte_mtr_meter_profile *profile, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(meter_profile_id);\n+\trte_trace_point_emit_ptr(profile);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_profile_delete,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id,\n+\t\tuint32_t meter_profile_id, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(meter_profile_id);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_profile_get,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id,\n+\t\tuint32_t meter_profile_id, const void *ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(meter_profile_id);\n+\trte_trace_point_emit_ptr(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_policy_validate,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id,\n+\t\tconst struct rte_mtr_meter_policy_params *policy, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_ptr(policy);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_policy_add,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t policy_id,\n+\t\tconst struct rte_mtr_meter_policy_params *policy, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(policy_id);\n+\trte_trace_point_emit_ptr(policy);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_policy_get,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t policy_id,\n+\t\tconst void *ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(policy_id);\n+\trte_trace_point_emit_ptr(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_enable,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_disable,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_profile_update,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id,\n+\t\tuint32_t meter_profile_id, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_u32(meter_profile_id);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_policy_update,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id,\n+\t\tuint32_t meter_policy_id, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_u32(meter_policy_id);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_dscp_table_update,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id,\n+\t\tenum rte_color *dscp_table, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_ptr(dscp_table);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_meter_vlan_table_update,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id,\n+\t\tconst enum rte_color *vlan_table, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_ptr(vlan_table);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_color_in_protocol_set,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id,\n+\t\tenum rte_mtr_color_in_protocol proto, uint32_t priority,\n+\t\tint ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_int(proto);\n+\trte_trace_point_emit_u32(priority);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_color_in_protocol_get,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_color_in_protocol_priority_get,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id,\n+\t\tenum rte_mtr_color_in_protocol proto, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_int(proto);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_stats_update,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id,\n+\t\tuint64_t stats_mask, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_u64(stats_mask);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+RTE_TRACE_POINT(\n+\trte_mtr_trace_stats_read,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id,\n+\t\tconst struct rte_mtr_stats *stats, uint64_t stats_mask,\n+\t\tint clear, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_ptr(stats);\n+\trte_trace_point_emit_u64(stats_mask);\n+\trte_trace_point_emit_int(clear);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n /* Fast path trace points */\n \n /* Called in loop in examples/qos_sched and examples/distributor */\n@@ -2006,6 +2182,36 @@ RTE_TRACE_POINT_FP(\n \trte_trace_point_emit_int(ret);\n )\n \n+/* Called in loop in app/test-flow-perf */\n+RTE_TRACE_POINT_FP(\n+\trte_mtr_trace_create,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id,\n+\t\tconst struct rte_mtr_params *params, int shared, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_ptr(params);\n+\trte_trace_point_emit_int(shared);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+/* Called in loop in app/test-flow-perf */\n+RTE_TRACE_POINT_FP(\n+\trte_mtr_trace_destroy,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t mtr_id, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(mtr_id);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n+/* Called in loop in app/test-flow-perf */\n+RTE_TRACE_POINT_FP(\n+\trte_mtr_trace_meter_policy_delete,\n+\tRTE_TRACE_POINT_ARGS(uint16_t port_id, uint32_t policy_id, int ret),\n+\trte_trace_point_emit_u16(port_id);\n+\trte_trace_point_emit_u32(policy_id);\n+\trte_trace_point_emit_int(ret);\n+)\n+\n #ifdef __cplusplus\n }\n #endif\ndiff --git a/lib/ethdev/ethdev_trace_points.c b/lib/ethdev/ethdev_trace_points.c\nindex b8cbb285de..c34c06941c 100644\n--- a/lib/ethdev/ethdev_trace_points.c\n+++ b/lib/ethdev/ethdev_trace_points.c\n@@ -591,3 +591,66 @@ RTE_TRACE_POINT_REGISTER(rte_flow_trace_async_action_handle_update,\n \n RTE_TRACE_POINT_REGISTER(rte_flow_trace_async_action_handle_query,\n \tlib.ethdev.flow.async.action.handle.query)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_capabilities_get,\n+\tlib.ethdev.mtr.capabilities_get)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_create,\n+\tlib.ethdev.mtr.create)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_destroy,\n+\tlib.ethdev.mtr.destroy)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_disable,\n+\tlib.ethdev.mtr.meter_disable)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_dscp_table_update,\n+\tlib.ethdev.mtr.meter_dscp_table_update)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_enable,\n+\tlib.ethdev.mtr.meter_enable)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_profile_add,\n+\tlib.ethdev.mtr.meter_profile_add)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_profile_delete,\n+\tlib.ethdev.mtr.meter_profile_delete)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_profile_get,\n+\tlib.ethdev.mtr.meter_profile_get)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_profile_update,\n+\tlib.ethdev.mtr.meter_profile_update)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_stats_read,\n+\tlib.ethdev.mtr.stats_read)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_stats_update,\n+\tlib.ethdev.mtr.stats_update)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_policy_add,\n+\tlib.ethdev.mtr.meter_policy_add)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_policy_delete,\n+\tlib.ethdev.mtr.meter_policy_delete)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_policy_get,\n+\tlib.ethdev.mtr.meter_policy_get)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_policy_update,\n+\tlib.ethdev.mtr.meter_policy_update)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_policy_validate,\n+\tlib.ethdev.mtr.meter_policy_validate)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_meter_vlan_table_update,\n+\tlib.ethdev.mtr.meter_vlan_table_update)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_color_in_protocol_get,\n+\tlib.ethdev.mtr.color_in_protocol_get)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_color_in_protocol_priority_get,\n+\tlib.ethdev.mtr.color_in_protocol_priority_get)\n+\n+RTE_TRACE_POINT_REGISTER(rte_mtr_trace_color_in_protocol_set,\n+\tlib.ethdev.mtr.color_in_protocol_set)\ndiff --git a/lib/ethdev/rte_mtr.c b/lib/ethdev/rte_mtr.c\nindex 3954839531..4e94af9d9f 100644\n--- a/lib/ethdev/rte_mtr.c\n+++ b/lib/ethdev/rte_mtr.c\n@@ -5,6 +5,7 @@\n #include <stdint.h>\n \n #include <rte_errno.h>\n+#include \"ethdev_trace.h\"\n #include \"rte_ethdev.h\"\n #include \"rte_mtr_driver.h\"\n #include \"rte_mtr.h\"\n@@ -82,8 +83,13 @@ rte_mtr_capabilities_get(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, capabilities_get)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, capabilities_get)(dev,\n \t\tcap, error);\n+\n+\trte_mtr_trace_capabilities_get(port_id, cap, ret);\n+\n+\treturn ret;\n }\n \n /* MTR meter profile add */\n@@ -94,8 +100,14 @@ rte_mtr_meter_profile_add(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_profile_add)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_profile_add)(dev,\n \t\tmeter_profile_id, profile, error);\n+\n+\trte_mtr_trace_meter_profile_add(port_id, meter_profile_id, profile,\n+\t\t\t\t\tret);\n+\n+\treturn ret;\n }\n \n /** MTR meter profile delete */\n@@ -105,8 +117,13 @@ rte_mtr_meter_profile_delete(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_profile_delete)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_profile_delete)(dev,\n \t\tmeter_profile_id, error);\n+\n+\trte_mtr_trace_meter_profile_delete(port_id, meter_profile_id, ret);\n+\n+\treturn ret;\n }\n \n /** MTR meter profile get */\n@@ -116,8 +133,13 @@ rte_mtr_meter_profile_get(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_HNDL_FUNC(port_id, meter_profile_get)(dev,\n+\tstruct rte_flow_meter_profile *ret;\n+\tret = RTE_MTR_HNDL_FUNC(port_id, meter_profile_get)(dev,\n \t\tmeter_profile_id, error);\n+\n+\trte_mtr_trace_meter_profile_get(port_id, meter_profile_id, ret);\n+\n+\treturn ret;\n }\n \n /* MTR meter policy validate */\n@@ -127,8 +149,13 @@ rte_mtr_meter_policy_validate(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_policy_validate)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_policy_validate)(dev,\n \t\tpolicy, error);\n+\n+\trte_mtr_trace_meter_policy_validate(port_id, policy, ret);\n+\n+\treturn ret;\n }\n \n /* MTR meter policy add */\n@@ -139,8 +166,13 @@ rte_mtr_meter_policy_add(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_policy_add)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_policy_add)(dev,\n \t\tpolicy_id, policy, error);\n+\n+\trte_mtr_trace_meter_policy_add(port_id, policy_id, policy, ret);\n+\n+\treturn ret;\n }\n \n /** MTR meter policy delete */\n@@ -150,8 +182,13 @@ rte_mtr_meter_policy_delete(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_policy_delete)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_policy_delete)(dev,\n \t\tpolicy_id, error);\n+\n+\trte_mtr_trace_meter_policy_delete(port_id, policy_id, ret);\n+\n+\treturn ret;\n }\n \n /** MTR meter policy get */\n@@ -161,8 +198,13 @@ rte_mtr_meter_policy_get(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_HNDL_FUNC(port_id, meter_policy_get)(dev,\n+\tstruct rte_flow_meter_policy *ret;\n+\tret = RTE_MTR_HNDL_FUNC(port_id, meter_policy_get)(dev,\n \t\tpolicy_id, error);\n+\n+\trte_mtr_trace_meter_policy_get(port_id, policy_id, ret);\n+\n+\treturn ret;\n }\n \n /** MTR object create */\n@@ -174,8 +216,13 @@ rte_mtr_create(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, create)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, create)(dev,\n \t\tmtr_id, params, shared, error);\n+\n+\trte_mtr_trace_create(port_id, mtr_id, params, shared, ret);\n+\n+\treturn ret;\n }\n \n /** MTR object destroy */\n@@ -185,8 +232,13 @@ rte_mtr_destroy(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, destroy)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, destroy)(dev,\n \t\tmtr_id, error);\n+\n+\trte_mtr_trace_destroy(port_id, mtr_id, ret);\n+\n+\treturn ret;\n }\n \n /** MTR object meter enable */\n@@ -196,8 +248,13 @@ rte_mtr_meter_enable(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_enable)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_enable)(dev,\n \t\tmtr_id, error);\n+\n+\trte_mtr_trace_meter_enable(port_id, mtr_id, ret);\n+\n+\treturn ret;\n }\n \n /** MTR object meter disable */\n@@ -207,8 +264,13 @@ rte_mtr_meter_disable(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_disable)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_disable)(dev,\n \t\tmtr_id, error);\n+\n+\trte_mtr_trace_meter_disable(port_id, mtr_id, ret);\n+\n+\treturn ret;\n }\n \n /** MTR object meter profile update */\n@@ -219,8 +281,13 @@ rte_mtr_meter_profile_update(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_profile_update)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_profile_update)(dev,\n \t\tmtr_id, meter_profile_id, error);\n+\n+\trte_mtr_trace_meter_profile_update(port_id, mtr_id, meter_profile_id, ret);\n+\n+\treturn ret;\n }\n \n /** MTR object meter policy update */\n@@ -231,8 +298,13 @@ rte_mtr_meter_policy_update(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_policy_update)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_policy_update)(dev,\n \t\tmtr_id, meter_policy_id, error);\n+\n+\trte_mtr_trace_meter_policy_update(port_id, mtr_id, meter_policy_id, ret);\n+\n+\treturn ret;\n }\n \n /** MTR object meter DSCP table update */\n@@ -243,8 +315,13 @@ rte_mtr_meter_dscp_table_update(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_dscp_table_update)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_dscp_table_update)(dev,\n \t\tmtr_id, proto, dscp_table, error);\n+\n+\trte_mtr_trace_meter_dscp_table_update(port_id, mtr_id, dscp_table, ret);\n+\n+\treturn ret;\n }\n \n /** MTR object meter VLAN table update */\n@@ -255,8 +332,13 @@ rte_mtr_meter_vlan_table_update(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, meter_vlan_table_update)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, meter_vlan_table_update)(dev,\n \t\tmtr_id, proto, vlan_table, error);\n+\n+\trte_mtr_trace_meter_vlan_table_update(port_id, mtr_id, vlan_table, ret);\n+\n+\treturn ret;\n }\n \n /** Set the input color protocol on MTR object */\n@@ -268,8 +350,13 @@ rte_mtr_color_in_protocol_set(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, in_proto_set)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, in_proto_set)(dev,\n \t\tmtr_id, proto, priority, error);\n+\n+\trte_mtr_trace_color_in_protocol_set(port_id, mtr_id, proto, priority, ret);\n+\n+\treturn ret;\n }\n \n /** Get input color protocols of MTR object */\n@@ -280,8 +367,13 @@ rte_mtr_color_in_protocol_get(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, in_proto_get)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, in_proto_get)(dev,\n \t\tmtr_id, proto_mask, error);\n+\n+\trte_mtr_trace_color_in_protocol_get(port_id, mtr_id, ret);\n+\n+\treturn ret;\n }\n \n /** Get input color protocol priority of MTR object */\n@@ -293,8 +385,13 @@ rte_mtr_color_in_protocol_priority_get(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, in_proto_prio_get)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, in_proto_prio_get)(dev,\n \t\tmtr_id, proto, priority, error);\n+\n+\trte_mtr_trace_color_in_protocol_priority_get(port_id, mtr_id, proto, ret);\n+\n+\treturn ret;\n }\n \n /** MTR object enabled stats update */\n@@ -305,8 +402,13 @@ rte_mtr_stats_update(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, stats_update)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, stats_update)(dev,\n \t\tmtr_id, stats_mask, error);\n+\n+\trte_mtr_trace_stats_update(port_id, mtr_id, stats_mask, ret);\n+\n+\treturn ret;\n }\n \n /** MTR object stats read */\n@@ -319,6 +421,12 @@ rte_mtr_stats_read(uint16_t port_id,\n \tstruct rte_mtr_error *error)\n {\n \tstruct rte_eth_dev *dev = &rte_eth_devices[port_id];\n-\treturn RTE_MTR_FUNC(port_id, stats_read)(dev,\n+\tint ret;\n+\tret = RTE_MTR_FUNC(port_id, stats_read)(dev,\n \t\tmtr_id, stats, stats_mask, clear, error);\n+\n+\trte_mtr_trace_stats_read(port_id, mtr_id, stats, *stats_mask, clear,\n+\t\t\t\t ret);\n+\n+\treturn ret;\n }\n", "prefixes": [ "v10", "5/6" ] }