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 7CB5146875; Wed, 4 Jun 2025 12:13:13 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 57680427C3; Wed, 4 Jun 2025 12:13:13 +0200 (CEST) Received: from mx0a-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by mails.dpdk.org (Postfix) with ESMTP id A17704042E for ; Wed, 4 Jun 2025 12:13:11 +0200 (CEST) Received: from pps.filterd (m0431384.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 553NVhfA018889; Wed, 4 Jun 2025 03:13:05 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=pfpt0220; bh=s W4hRVrIE7o823b3GXJHauAR+9YPXandA6zaSDCUnBE=; b=Fsr2DtDI16hMcFW+t kl97FQvWP8MJ8F6Ur/Qj9MqtaF16hrJku3WOXWVXLUPUtTdeiN+18aeNsN384E5d EaKvAEAFVIxtRvpco+nqCguPVctzpbSV/NIpDZRebJe77eIJ0B0u8lEqKHtw7OgR R5hRpMsMbRiE5nhCDMDYLriP/fFo884lo8FhkeLuFuvY4NNJFTCFiTIgnBurLtVc DdKapQh85h6xtbw2utj3oA9t7I12N32/JzRj9Ft3le7mY3gEXn6HffA47dtpgkY6 tEIvX6E/i+IDhLuOHXT/qIcWO4kocCLqj3M/v/IGuMWlnM1Y3pZnxmPeg+A7rDp1 9oaZA== Received: from dc6wp-exch02.marvell.com ([4.21.29.225]) by mx0a-0016f401.pphosted.com (PPS) with ESMTPS id 472at514f3-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 04 Jun 2025 03:13:04 -0700 (PDT) Received: from DC6WP-EXCH02.marvell.com (10.76.176.209) by DC6WP-EXCH02.marvell.com (10.76.176.209) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.4; Wed, 4 Jun 2025 03:13:03 -0700 Received: from maili.marvell.com (10.69.176.80) by DC6WP-EXCH02.marvell.com (10.76.176.209) with Microsoft SMTP Server id 15.2.1544.4 via Frontend Transport; Wed, 4 Jun 2025 03:13:03 -0700 Received: from cavium-PowerEdge-R640.. (unknown [10.28.36.207]) by maili.marvell.com (Postfix) with ESMTP id 8109E3F7081; Wed, 4 Jun 2025 03:13:00 -0700 (PDT) From: Nitin Saxena To: Jerin Jacob , Kiran Kumar K , Nithin Dabilpuram , Zhirun Yan , Robin Jarry , Christophe Fontaine CC: , Nitin Saxena Subject: [PATCH v10 0/7] add feature arc in rte_graph Date: Wed, 4 Jun 2025 15:42:27 +0530 Message-ID: <20250604101259.4181992-1-nsaxena@marvell.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250103060612.2671836-1-nsaxena@marvell.com> References: <20250103060612.2671836-1-nsaxena@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Proofpoint-ORIG-GUID: 1TLqEu9Zezom4HM5e6K37bmHTnNrlXMi X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwNjA0MDA3NiBTYWx0ZWRfXyh4W4LZMc64M KyEYIm+3BsYSEC6q5m46YL1b6SMxbrd6fUVDqCiU0tiXCwWW+x59W132JW+r88g6SCfBhNdLoF2 u74qRmLG4AucjTP3M41KUGI9UR02f7ro8n4OnojG34IrYS46qf/PPN70BrVC/m96UZ+a5hbTjPt 10e54llXhA94SkioSr3gmjD9Np9mHqoyCw5AKHvHkqdtINWGF77QLcRMFUi+HhVJavqYoJzCWmO lkZRywrgtlItJRu5sO+3x+wLXYGm6y1cyGHqK1QiOlH4JB/V0vTg3Ln5R60txkAx3YEu867/qsH H3V54m9PXxEzLDisodcvUhmtiKe3P6WIfvx34CroS2dHMkImYZ4ovdUm6+MGMVkvLb85lueeRcp g+EWyrbGnPe2WnlK7cuyEuT/B2RhTtnzjOZ9IDsjURDw9tUTYyWL62WxVtyagr/KPX+pjG8O X-Authority-Analysis: v=2.4 cv=F9dXdrhN c=1 sm=1 tr=0 ts=68401c30 cx=c_pps a=gIfcoYsirJbf48DBMSPrZA==:117 a=gIfcoYsirJbf48DBMSPrZA==:17 a=6IFa9wvqVegA:10 a=NEAV23lmAAAA:8 a=0A_AvAqZAcpRaedzLnUA:9 X-Proofpoint-GUID: 1TLqEu9Zezom4HM5e6K37bmHTnNrlXMi X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1099,Hydra:6.0.736,FMLib:17.12.80.40 definitions=2025-06-04_02,2025-06-03_02,2025-03-28_01 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 Feature arc represents an ordered list of features/protocols at a given networking layer. It is a high level abstraction to express relationship between rte_graph nodes, as feature nodes, and allow packets steering across these nodes in a simplified manner. Features (or feature nodes) are nodes which handles partial or complete handling of a protocol in fast path. Like ipv4-rewrite node, which adds rewrite data to an outgoing IPv4 packet. However in above example, outgoing interface(say "eth0") may have outbound IPsec policy enabled, hence packets must be steered from ipv4-rewrite node to outbound-IPsec-policy node. On the other hand, packets routed to another interface (say eth1) must ot be sent to IPsec node, as IPsec feature is disabled on eth1. Feature-arc allows rte_graph applications to manage such constraints easily Feature arc abstraction allows rte_graph based application to 1. Runtime enable/disable features on an index (like interface), so that if a feature is enabled at runtime, only then packets associated with that interface will steer to corresponding feature node. Control plane enables/disables features. Features enabled on one interface may not be enabled on another interface with in a same feature arc. 2. Single feature arc represents a hook point at a network layer in an given direction. Multiple feature arcs can be added in desginated nodes to add various hook points in stack. Likt IPv4-output, IPv4-input, IPv6-output, IPv6-input, ethernet-input, mpls-input, mpls-output etc. 3. Hook custom/out-of-tree nodes to DPDK in-built nodes and allow packet steering from in-built node to custom node without requiring changes to in-built fast path function 4. Express features in a particular sequential order so that packets are steered in an ordered way across nodes in fast path. For eg: if IPsec and IPv4 features are enabled on an ingress interface, packets must be sent to IPsec inbound policy node first and then to ipv4 lookup node. This patch series adds feature arc library in rte_graph and also adds "ipv4-output" feature arc handling in "ipv4-rewrite" node. Changes in v10 - Incorporated all comments from Jerin - Splitted feature arc spec patch into 3 patches - added programming guide in each patch - Fixed documentation comments Changes in v9: - Fix build failures Changes in v8: - Fix CI iol testing Changes in v7: - Rebased to latest main for DPDK-25.07 - Added override_index_cb() in RTE_GRAPH_FEATURE_REGISTER() to allow features to take control of "max_index" (passed to rte_graph_feature_arc_create()) - Streamlined fast path APIs to make them more simpler - Added quad_loop in "if_tx_feature" node - Added functional test cases and fixed bugs - Based on feedback from Redhat on v6 patches, it was tried to move feature arc APIs inside rte_graph library (to reduce node burden) but it was adding 4% performance regression for L3fwd usecase. For interested users: https://github.com/nsaxena16/dpdk/pull/2 Changes in v6: - Rebased to latest main for DPDK-25.03 - Added constructor based feature arc/feature registration - Changed design to handle fast path synchronization via RCU mechanism when any feature is enabled or disabled - Added feature arc specific mbuf dynamic field to carry feature data across nodes - Added feature arc example in app/graph - Programming guide and functional test cases in future versions Nitin Saxena (7): graph: add API to override node process function graph: add feature arc registrations graph: add feature arc init APIs graph: add feature enable/disable APIs ip4: add ip4 output feature arc app/graph: add custom feature nodes for ip4 output arc test/graph_feature_arc: add functional tests app/graph/commands.list | 6 + app/graph/feature.c | 141 ++ app/graph/feature.h | 13 + app/graph/graph.c | 4 + app/graph/ip4_output_hook.c | 180 ++ app/graph/main.c | 15 +- app/graph/meson.build | 2 + app/graph/module_api.h | 2 + app/test/meson.build | 1 + app/test/test_graph_feature_arc.c | 1374 +++++++++++++ doc/api/doxy-api-index.md | 2 + doc/guides/prog_guide/graph_lib.rst | 491 +++++ doc/guides/prog_guide/img/feature_arc-1.svg | 269 +++ doc/guides/prog_guide/img/feature_arc-2.svg | 511 +++++ doc/guides/rel_notes/release_25_07.rst | 2 + doc/guides/tools/graph.rst | 15 + lib/graph/graph_feature_arc.c | 2050 +++++++++++++++++++ lib/graph/graph_private.h | 15 + lib/graph/meson.build | 4 +- lib/graph/node.c | 23 + lib/graph/rte_graph_feature_arc.h | 644 ++++++ lib/graph/rte_graph_feature_arc_worker.h | 606 ++++++ lib/node/ethdev_ctrl.c | 8 + lib/node/interface_tx_feature.c | 213 ++ lib/node/interface_tx_feature_priv.h | 33 + lib/node/ip4_rewrite.c | 286 ++- lib/node/meson.build | 1 + lib/node/node_private.h | 1 + lib/node/rte_node_ip4_api.h | 4 + 29 files changed, 6907 insertions(+), 9 deletions(-) create mode 100644 app/graph/feature.c create mode 100644 app/graph/feature.h create mode 100644 app/graph/ip4_output_hook.c create mode 100644 app/test/test_graph_feature_arc.c create mode 100644 doc/guides/prog_guide/img/feature_arc-1.svg create mode 100644 doc/guides/prog_guide/img/feature_arc-2.svg create mode 100644 lib/graph/graph_feature_arc.c create mode 100644 lib/graph/rte_graph_feature_arc.h create mode 100644 lib/graph/rte_graph_feature_arc_worker.h create mode 100644 lib/node/interface_tx_feature.c create mode 100644 lib/node/interface_tx_feature_priv.h -- 2.43.0