Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/125657/?format=api
{ "id": 125657, "url": "http://patches.dpdk.org/api/patches/125657/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20230331040306.3143693-4-zhirun.yan@intel.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": "<20230331040306.3143693-4-zhirun.yan@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20230331040306.3143693-4-zhirun.yan@intel.com", "date": "2023-03-31T04:02:54", "name": "[v5,03/15] graph: move node process into inline function", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "a6521990b79d45de161acf764f3471bc639b4666", "submitter": { "id": 1154, "url": "http://patches.dpdk.org/api/people/1154/?format=api", "name": "Yan, Zhirun", "email": "zhirun.yan@intel.com" }, "delegate": { "id": 1, "url": "http://patches.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20230331040306.3143693-4-zhirun.yan@intel.com/mbox/", "series": [ { "id": 27590, "url": "http://patches.dpdk.org/api/series/27590/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=27590", "date": "2023-03-31T04:02:51", "name": "graph enhancement for multi-core dispatch", "version": 5, "mbox": "http://patches.dpdk.org/series/27590/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/125657/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/125657/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 BB7C94287E;\n\tFri, 31 Mar 2023 06:03:28 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id A417142BFE;\n\tFri, 31 Mar 2023 06:03:23 +0200 (CEST)", "from mga11.intel.com (mga11.intel.com [192.55.52.93])\n by mails.dpdk.org (Postfix) with ESMTP id 4349242BB1\n for <dev@dpdk.org>; Fri, 31 Mar 2023 06:03:21 +0200 (CEST)", "from fmsmga007.fm.intel.com ([10.253.24.52])\n by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 30 Mar 2023 21:03:19 -0700", "from dpdk-zhirun-lmm.sh.intel.com ([10.67.119.68])\n by fmsmga007.fm.intel.com with ESMTP; 30 Mar 2023 21:03:17 -0700" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1680235401; x=1711771401;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=EoeDHzSoMCh9gW/nQpMjzg2WI1Xo6w5PxEC7qbrHzkk=;\n b=a/6DBvneQTi1X5MNh90YUjCplC3E2EzXTNyNveqFffIx91MgV4BfEAfa\n yXzfcCYlugN83vvecp8f2kh+ZiKficVbnbL0d3t6HqJBYV4yOc0a/JfN2\n BHubwMrc99URfX+lfPQwyUlOhp8ZeG3L9JphSIVWCJZ1W9r+TEwK0si+G\n +j+XAaVa+LZnpIOXo5xFvyF62OBROERl9/h54OvdXVCP1LP5DQcTyqD5b\n 6TUJS7tnlOfVjaQaCNzT4EJtOJ+bOoI7XEGxZBGH4x/96eJwFgktp0ccr\n FJFZJ8nRbs7EfiC5onGOk3QRkUia59g2BchsBXmGp+duzb7MSA/K8QJna w==;", "X-IronPort-AV": [ "E=McAfee;i=\"6600,9927,10665\"; a=\"338849065\"", "E=Sophos;i=\"5.98,306,1673942400\"; d=\"scan'208\";a=\"338849065\"", "E=McAfee;i=\"6600,9927,10665\"; a=\"687502227\"", "E=Sophos;i=\"5.98,306,1673942400\"; d=\"scan'208\";a=\"687502227\"" ], "X-ExtLoop1": "1", "From": "Zhirun Yan <zhirun.yan@intel.com>", "To": "dev@dpdk.org, jerinj@marvell.com, kirankumark@marvell.com,\n ndabilpuram@marvell.com, stephen@networkplumber.org", "Cc": "cunming.liang@intel.com, haiyue.wang@intel.com,\n Zhirun Yan <zhirun.yan@intel.com>", "Subject": "[PATCH v5 03/15] graph: move node process into inline function", "Date": "Fri, 31 Mar 2023 13:02:54 +0900", "Message-Id": "<20230331040306.3143693-4-zhirun.yan@intel.com>", "X-Mailer": "git-send-email 2.37.2", "In-Reply-To": "<20230331040306.3143693-1-zhirun.yan@intel.com>", "References": "<20230330061834.3118201-1-zhirun.yan@intel.com>\n <20230331040306.3143693-1-zhirun.yan@intel.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "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": "Node process is a single and reusable block, move the code into an inline\nfunction.\n\nSigned-off-by: Haiyue Wang <haiyue.wang@intel.com>\nSigned-off-by: Cunming Liang <cunming.liang@intel.com>\nSigned-off-by: Zhirun Yan <zhirun.yan@intel.com>\n---\n lib/graph/rte_graph_model_rtc.h | 20 ++---------------\n lib/graph/rte_graph_worker_common.h | 33 +++++++++++++++++++++++++++++\n 2 files changed, 35 insertions(+), 18 deletions(-)", "diff": "diff --git a/lib/graph/rte_graph_model_rtc.h b/lib/graph/rte_graph_model_rtc.h\nindex 665560f831..0dcb7151e9 100644\n--- a/lib/graph/rte_graph_model_rtc.h\n+++ b/lib/graph/rte_graph_model_rtc.h\n@@ -20,9 +20,6 @@ rte_graph_walk_rtc(struct rte_graph *graph)\n \tconst rte_node_t mask = graph->cir_mask;\n \tuint32_t head = graph->head;\n \tstruct rte_node *node;\n-\tuint64_t start;\n-\tuint16_t rc;\n-\tvoid **objs;\n \n \t/*\n \t * Walk on the source node(s) ((cir_start - head) -> cir_start) and then\n@@ -41,21 +38,8 @@ rte_graph_walk_rtc(struct rte_graph *graph)\n \t */\n \twhile (likely(head != graph->tail)) {\n \t\tnode = (struct rte_node *)RTE_PTR_ADD(graph, cir_start[(int32_t)head++]);\n-\t\tRTE_ASSERT(node->fence == RTE_GRAPH_FENCE);\n-\t\tobjs = node->objs;\n-\t\trte_prefetch0(objs);\n-\n-\t\tif (rte_graph_has_stats_feature()) {\n-\t\t\tstart = rte_rdtsc();\n-\t\t\trc = node->process(graph, node, objs, node->idx);\n-\t\t\tnode->total_cycles += rte_rdtsc() - start;\n-\t\t\tnode->total_calls++;\n-\t\t\tnode->total_objs += rc;\n-\t\t} else {\n-\t\t\tnode->process(graph, node, objs, node->idx);\n-\t\t}\n-\t\t\tnode->idx = 0;\n-\t\t\thead = likely((int32_t)head > 0) ? head & mask : head;\n+\t\t__rte_node_process(graph, node);\n+\t\thead = likely((int32_t)head > 0) ? head & mask : head;\n \t}\n \tgraph->tail = 0;\n }\ndiff --git a/lib/graph/rte_graph_worker_common.h b/lib/graph/rte_graph_worker_common.h\nindex b58f8f6947..41428974db 100644\n--- a/lib/graph/rte_graph_worker_common.h\n+++ b/lib/graph/rte_graph_worker_common.h\n@@ -130,6 +130,39 @@ void __rte_node_stream_alloc_size(struct rte_graph *graph,\n \n /* Fast path helper functions */\n \n+/**\n+ * @internal\n+ *\n+ * Enqueue a given node to the tail of the graph reel.\n+ *\n+ * @param graph\n+ * Pointer Graph object.\n+ * @param node\n+ * Pointer to node object to be enqueued.\n+ */\n+static __rte_always_inline void\n+__rte_node_process(struct rte_graph *graph, struct rte_node *node)\n+{\n+\tuint64_t start;\n+\tuint16_t rc;\n+\tvoid **objs;\n+\n+\tRTE_ASSERT(node->fence == RTE_GRAPH_FENCE);\n+\tobjs = node->objs;\n+\trte_prefetch0(objs);\n+\n+\tif (rte_graph_has_stats_feature()) {\n+\t\tstart = rte_rdtsc();\n+\t\trc = node->process(graph, node, objs, node->idx);\n+\t\tnode->total_cycles += rte_rdtsc() - start;\n+\t\tnode->total_calls++;\n+\t\tnode->total_objs += rc;\n+\t} else {\n+\t\tnode->process(graph, node, objs, node->idx);\n+\t}\n+\tnode->idx = 0;\n+}\n+\n /**\n * @internal\n *\n", "prefixes": [ "v5", "03/15" ] }