Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/128484/?format=api
{ "id": 128484, "url": "http://patches.dpdk.org/api/patches/128484/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20230609191245.252521-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": "<20230609191245.252521-4-zhirun.yan@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20230609191245.252521-4-zhirun.yan@intel.com", "date": "2023-06-09T19:12:32", "name": "[v12,03/16] 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/20230609191245.252521-4-zhirun.yan@intel.com/mbox/", "series": [ { "id": 28444, "url": "http://patches.dpdk.org/api/series/28444/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=28444", "date": "2023-06-09T19:12:29", "name": "graph enhancement for multi-core dispatch", "version": 12, "mbox": "http://patches.dpdk.org/series/28444/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/128484/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/128484/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 66FE942C71;\n\tFri, 9 Jun 2023 21:20:15 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id E2ECF427E9;\n\tFri, 9 Jun 2023 21:19:59 +0200 (CEST)", "from mga09.intel.com (mga09.intel.com [134.134.136.24])\n by mails.dpdk.org (Postfix) with ESMTP id 365BB41611\n for <dev@dpdk.org>; Fri, 9 Jun 2023 21:19:58 +0200 (CEST)", "from fmsmga006.fm.intel.com ([10.253.24.20])\n by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 09 Jun 2023 12:19:57 -0700", "from dpdk-zhirun-lmm.sh.intel.com ([10.67.119.94])\n by fmsmga006.fm.intel.com with ESMTP; 09 Jun 2023 12:19:54 -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=1686338398; x=1717874398;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=wX5PugPPNBMIb4g2SurCWdXg02rVgSLxjrxsl8wDmko=;\n b=YNTGyd7B+pKzIE3WPTpp+12c+Vgpu78J/e3w1unjd7okT2F3KY60tyjM\n 8UulsaJT6plBBoqF169b+5JERX1DswNaZf2O7z18CtOwHGwDuGy2H5pfm\n H7zvbF4qe7O9Khhfp208tj1nLR4K1S/ihRSwxug8S/RN2cregayxvfbi9\n u8piZIhra/4+alXTkUCtt/eatJ00bSmXutoXPuve9GhXAFd0RT2SLt+nr\n F/zcPKsOb4H7uOFmlWIhaK0w6wAIj5LIV6HEceyv/j2J1xzd3sGSgFVk9\n 2AjgcmCJoAqoMSN61g71HKfMHUkKFD/6XPjF3Fslm5eZhRR5OnI3aqFRp g==;", "X-IronPort-AV": [ "E=McAfee;i=\"6600,9927,10736\"; a=\"360154944\"", "E=Sophos;i=\"6.00,230,1681196400\"; d=\"scan'208\";a=\"360154944\"", "E=McAfee;i=\"6600,9927,10736\"; a=\"957254628\"", "E=Sophos;i=\"6.00,230,1681196400\"; d=\"scan'208\";a=\"957254628\"" ], "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,\n pbhagavatula@marvell.com, jerinjacobk@gmail.com, david.marchand@redhat.com", "Cc": "cunming.liang@intel.com, haiyue.wang@intel.com,\n mattias.ronnblom@ericsson.com, Zhirun Yan <zhirun.yan@intel.com>", "Subject": "[PATCH v12 03/16] graph: move node process into inline function", "Date": "Sat, 10 Jun 2023 03:12:32 +0800", "Message-Id": "<20230609191245.252521-4-zhirun.yan@intel.com>", "X-Mailer": "git-send-email 2.37.2", "In-Reply-To": "<20230609191245.252521-1-zhirun.yan@intel.com>", "References": "<20230608151844.1823783-1-zhirun.yan@intel.com>\n <20230609191245.252521-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>\nAcked-by: Jerin Jacob <jerinj@marvell.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 10b359772f..4b6236e301 100644\n--- a/lib/graph/rte_graph_model_rtc.h\n+++ b/lib/graph/rte_graph_model_rtc.h\n@@ -21,9 +21,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@@ -42,21 +39,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 475ccdc0ee..a90addb172 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": [ "v12", "03/16" ] }