From patchwork Mon Jun 17 14:07:46 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gregory Etelson X-Patchwork-Id: 141205 X-Patchwork-Delegate: rasland@nvidia.com Return-Path: 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]) by inbox.dpdk.org (Postfix) with ESMTP id A00B745482; Mon, 17 Jun 2024 16:08:35 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1797340656; Mon, 17 Jun 2024 16:08:35 +0200 (CEST) Received: from NAM12-MW2-obe.outbound.protection.outlook.com (mail-mw2nam12on2055.outbound.protection.outlook.com [40.107.244.55]) by mails.dpdk.org (Postfix) with ESMTP id C16BC4028A for ; Mon, 17 Jun 2024 16:08:33 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GiHOiMlR9uzfVVViSf2L7TwlN8q8xDFB2+Y4RnQGwrUqhJ0jjvyDoglW3uJ8VMraf0CKJHn1hHUnEBsR2XaOXlLDfgL3za5Lcdu3VSNq4vXSan/pRNtdaTsjVif8Ik4W6RWqbDlRTQAoetO26RV7qwoCPBNHNSfaGUgXsowJiRm4YeVDQV23/hDXqwOo8YDU8Fn0qHNErj7DYdkMj0g+07USCCRhcNRRMsjFuNu8/eytzblmG4Mp0jSNR1x0OFPTQn/LV26bxbPUB6beHLT/iRli5m8bqqtjD8xo3Jm+McYWBcpUjWJbvGwfMms4Ap4u8+ZpAS9iriQ2jDW8RmMXuw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=nr0KcsSZe/Yofd4P5yXAdU9+Bn+ufm6Ij1thUjYLEwQ=; b=DjqXU361pjI/4ONq/a/Jv03YqmqBjToWnGlxHyFpvBz3C4tq2ySArrBjyjjfK1DIQb0YVj46OrspzYzfOMcD6QIquNXUzB6bTQsmxB8ViKY/AX/USuIC3Ic744RALqvjWXUQCow4feZ81BXyVt/mC0H7gG9ocxI7MsGON/A7w33FUWgBKzy20kzG8xSo4yQggvz1iQqcH7G1lML+w9ZezEjJGp8+6QCnytpSvI7JzT7mQgK4+ElcGhI9bYpjjhrCGWxdEHSl6HPrjvcKb/ftdUqEIIblPu1y47ttu7y0c7HzMAyh+QPXcY0HTQGVnPRMcGlQ/snjSw3pMRo2Jmz3pA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.161) smtp.rcpttodomain=dpdk.org smtp.mailfrom=nvidia.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=nr0KcsSZe/Yofd4P5yXAdU9+Bn+ufm6Ij1thUjYLEwQ=; b=eA5HuAK/wP2j8V6lCmwo54RS0M2liAAOLuI29omOVfx1mVCay5OHQrpzhwBwb9/vEN6T4rlwzDGeaxIi1h9vwDjbAjb/12xL83BjcpFoin/VsvfprYYDdlTSrxg+prDUxua938t46ENRxj8teIPROz1zweqWoAmoUxnRo4NujNILZNMwe60ORnWkq+DXUxWIYeqkDTQZJSJ6+vqHFK/taxTjEI0/Zs/a0aFCaGJv0DtNiLQWtodCf5az3NSiXMzC79MSW7LiJZq1uw46DOaB7xrJDuBmTzk2v2rosHU0B2Qt/GPoknVmD2AHF0RwGojalgX89ItClK5R0yiW0FB+uQ== Received: from DM6PR02CA0115.namprd02.prod.outlook.com (2603:10b6:5:1b4::17) by IA0PR12MB8421.namprd12.prod.outlook.com (2603:10b6:208:40f::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7677.30; Mon, 17 Jun 2024 14:08:30 +0000 Received: from DS1PEPF0001708E.namprd03.prod.outlook.com (2603:10b6:5:1b4:cafe::61) by DM6PR02CA0115.outlook.office365.com (2603:10b6:5:1b4::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7677.31 via Frontend Transport; Mon, 17 Jun 2024 14:08:30 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.117.161) smtp.mailfrom=nvidia.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.117.161 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.117.161; helo=mail.nvidia.com; pr=C Received: from mail.nvidia.com (216.228.117.161) by DS1PEPF0001708E.mail.protection.outlook.com (10.167.17.134) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7677.15 via Frontend Transport; Mon, 17 Jun 2024 14:08:30 +0000 Received: from rnnvmail201.nvidia.com (10.129.68.8) by mail.nvidia.com (10.129.200.67) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.4; Mon, 17 Jun 2024 07:08:13 -0700 Received: from nvidia.com (10.126.231.35) by rnnvmail201.nvidia.com (10.129.68.8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.4; Mon, 17 Jun 2024 07:08:10 -0700 From: Gregory Etelson To: CC: , =?utf-8?b?wqA=?= , , Dariusz Sosnowski , "Viacheslav Ovsiienko" , Ori Kam , Suanming Mou , Matan Azrad Subject: [PATCH 1/2] net/mlx5: refactor non-template RSS expansion Date: Mon, 17 Jun 2024 17:07:46 +0300 Message-ID: <20240617140747.169197-2-getelson@nvidia.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240617140747.169197-1-getelson@nvidia.com> References: <20240617140747.169197-1-getelson@nvidia.com> MIME-Version: 1.0 X-Originating-IP: [10.126.231.35] X-ClientProxiedBy: rnnvmail201.nvidia.com (10.129.68.8) To rnnvmail201.nvidia.com (10.129.68.8) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS1PEPF0001708E:EE_|IA0PR12MB8421:EE_ X-MS-Office365-Filtering-Correlation-Id: a9269692-eb31-465f-4c91-08dc8ed6f75a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230037|82310400023|1800799021|376011|36860700010; X-Microsoft-Antispam-Message-Info: qDRvUo+OHhaZfI9P0EU4goXo6IEFWv0m97+svcsjv8W91N9Hfhx9FjnTKjuzOz+DiZ/Z3vgqxA3N56qgOUJwQodGCvM+tyr9lDCyCiVNCBrn3kDAu4Caa7jZZhd8ohYIzj3KBZbSu0TnG4bz2JvP4Xxm8VoU/t7Zcj3rF/uy+FZNBa4whkgoTMVeukGvAOEHpQlhZEsTiiPRzF1BQyFNzgNYysk9MDCRoBHeM6lc/Fi22UOA3Cq5nQTdbcDecZfCl0h8cALz4GhVU9qHyjeJmdtB6IiY84glbMrI1ozsMh5enAdIFP7bvNmXta0/XG+iVVaBqbNxrR5QYNOx+mcH+sMV0l1D7cDClDlQnewTOBaUy6lc8WZRafwvJSyoSAe0rvuhjDe3MOnQzo0DedVyGBQDbPAQy/ltKDrIfMODWURMSSZuAjrn68LaZGlRIBm+9Hj3TtmD7lDpJSKADv07KzkHxYy/gD4MaJNX2KfZ6pJWD0z+mBkGiJLte+Kin541kQs/0FGup6QPczZNehaV/4YA5/JYUDgkzXLJL6SvpDq8ZjQXiw4s4O3e2HhG6dg1RHprrEne03I6/L1evmsmV48ZqDAhaSioZUZ8sK1gNwGIcuhhcOG3kZUr5TlHfi1DJEYiAg+XWmaZwPr7QYuoH+g99qGVNvQogSf4MNAbrwb2XnMItG0kHVh8mxowSZW4pTQIYYy1uNJCFfBq76V85MB/nrKQTnRfPUAC2XzRVHEQgVlAdynVef9AhTsurcaPWoMIzVMz4PND+Gc2Znf8HMv3LNuWRio1dLOW3Hr3xvEK14EzeIBf/FiPAd1QlK7p/Z/YkZTD9G25TUlb0D0Gcx9AahwMiOqOtgxXASm3AKvjrZ1l/Q2B3Px+2r2sHvudpShes+paaq/TW38hsUUXF2WvOzVnXQkJHqjdFAyVOmfDOzdeBx2ZK0LES+GOh7IdF5Jc8UzMbmLyw1gdinpaPa9UnuzsX9xDiC+RHno7LOi+rBE7I5b2j15RJjmtx2BPEHldsT6sXlniT5BaxRp0nnmOV3o01WVQEKIC60ETxDiVyLU5l7l+fP0kPXyD4Xlxg/WJC+HBZGJu786JJhLazF4Nwqtc6wgLWJUSvtKaCQKfdKkYSADZ8GgCj87gnJTQu52F6rqMSj91UAmiUQ5K3rz5jgB13hCA0WO3AAjdr3YABT2pV8Ja1/Jv06u57DIn407IY17UAQHUPv8ah1EuEB8HhoYbcSs3jlyDB5AowkJ8kBYuilhG0tE5hJEs7+DfR4g7R6CRdw2M02ZfbxnJxQNJO5GDiNXBUxXUA0nMzamKtsEYRMySp4I7+HZ4BuoDiuan8smlwzApjISERVYZ02IVIHVIr3L0h5nxtLxUhAd+NrosD77wIlmBiZw2NcvtqYvVCwTsSXFOdFcFnnZmXw== X-Forefront-Antispam-Report: CIP:216.228.117.161; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:dc6edge2.nvidia.com; CAT:NONE; SFS:(13230037)(82310400023)(1800799021)(376011)(36860700010); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Jun 2024 14:08:30.2084 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: a9269692-eb31-465f-4c91-08dc8ed6f75a X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.117.161]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: DS1PEPF0001708E.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA0PR12MB8421 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 The current PMD handled non-template RSS expansion in the `flow_nta_handle_rss()` function. The function returned flow handle for expanded RSS flow rule or NULL if there was no RSS expansion. The NULL result with non-zero rte_errno value pointed to an error during RSS expansion. If rte_errno value was 0, PMD resumed normal flow creation for the RSS rule without expansion. The patch changes `flow_nta_handle_rss()` behavior. On success, the function always creates a flow handle. The flow handle references multiple flows if RSS expansion was required and single flow if no RSS expansion was not required. The function returns NULL if it failed to create RSS flow. Signed-off-by: Gregory Etelson Acked-by: Dariusz Sosnowski --- drivers/net/mlx5/mlx5_flow_hw.c | 6 +----- drivers/net/mlx5/mlx5_nta_rss.c | 24 ++++++++++++++---------- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/net/mlx5/mlx5_flow_hw.c b/drivers/net/mlx5/mlx5_flow_hw.c index eb89dcf454..78b7014b73 100644 --- a/drivers/net/mlx5/mlx5_flow_hw.c +++ b/drivers/net/mlx5/mlx5_flow_hw.c @@ -13766,11 +13766,7 @@ static uintptr_t flow_hw_list_create(struct rte_eth_dev *dev, flow = flow_nta_handle_rss(dev, attr, items, actions, rss_conf, item_flags, action_flags, external, type, error); - if (flow) - return (uintptr_t)flow; - if (error->type != RTE_FLOW_ERROR_TYPE_NONE) - return 0; - /* Fall Through to non-expanded RSS flow */ + return (uintptr_t)flow; } /*TODO: Handle split/expand to num_flows. */ diff --git a/drivers/net/mlx5/mlx5_nta_rss.c b/drivers/net/mlx5/mlx5_nta_rss.c index 1f0085ff06..bcba2f98ed 100644 --- a/drivers/net/mlx5/mlx5_nta_rss.c +++ b/drivers/net/mlx5/mlx5_nta_rss.c @@ -439,6 +439,7 @@ flow_nta_handle_rss(struct rte_eth_dev *dev, struct rte_flow_action_rss ptype_rss_conf; struct mlx5_nta_rss_ctx rss_ctx; uint64_t rss_types = rte_eth_rss_hf_refine(rss_conf->types); + bool expand = true; bool inner_rss = rss_conf->level > 1; bool outer_rss = !inner_rss; bool l3_item = (outer_rss && (item_flags & MLX5_FLOW_LAYER_OUTER_L3)) || @@ -484,28 +485,31 @@ flow_nta_handle_rss(struct rte_eth_dev *dev, * L4 is the maximal expansion level. * Original pattern does not need expansion. */ - rte_flow_error_set(error, 0, RTE_FLOW_ERROR_TYPE_NONE, NULL, NULL); - return NULL; - } - if (!l4_hash) { + expand = false; + } else if (!l4_hash) { if (!l3_hash) { /* * RSS action was not configured to hash L3 or L4. * No expansion needed. */ - rte_flow_error_set(error, 0, RTE_FLOW_ERROR_TYPE_NONE, NULL, NULL); - return NULL; - } - if (l3_item) { + expand = false; + } else if (l3_item) { /* * Original flow pattern extended up to L3 level. * RSS action was not set for L4 hash. * Original pattern does not need expansion. */ - rte_flow_error_set(error, 0, RTE_FLOW_ERROR_TYPE_NONE, NULL, NULL); - return NULL; + expand = false; } } + if (!expand) { + struct rte_flow_hw *flow = NULL; + + flow_hw_create_flow(dev, flow_type, attr, items, + actions, item_flags, action_flags, + external, &flow, error); + return flow; + } /* Create RSS expansions in dedicated PTYPE flow group */ ptype_attr.group = mlx5_hw_get_rss_ptype_group(dev); if (!ptype_attr.group) { From patchwork Mon Jun 17 14:07:47 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gregory Etelson X-Patchwork-Id: 141206 X-Patchwork-Delegate: rasland@nvidia.com Return-Path: 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]) by inbox.dpdk.org (Postfix) with ESMTP id 45B6A45482; Mon, 17 Jun 2024 16:08:45 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BF4354065E; Mon, 17 Jun 2024 16:08:41 +0200 (CEST) Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on2063.outbound.protection.outlook.com [40.107.220.63]) by mails.dpdk.org (Postfix) with ESMTP id 049B14065D for ; Mon, 17 Jun 2024 16:08:40 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Ttlvjq2/Kp35pbGkgvXL9VZmAoREJi/mD/uXyrCY7VUbGeY90MiCyEc/4CtD6zmzg8ESQQr2jeJlvSqpWGxWSIK3AMgutruQ3zmcnfRot3Qk7txFTcxKIgm7GBiRYUCoK8hqmaR80YyDHSl5p6ZMLPwn9KoAE5XTrT3SMFXHp/ttpzaPE/RPqjh2vF7iMK1bnPgO0w+hT5t9M5GoUvT49J5h2fclwJcSL4TcnIciEkago2kb+zDhw/6ZoyUp/BJ5wMdLoU/z8ObrgJ9WkEchIP2N4PCdSLcBG8WCREHR3Fr85beqWil26CBoXGqoSprJhOgq75SYlqxDJWsjxMoYRA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=aBqHMbDVCQB5qevBkudxUSnuhkaMwatoeIBLXylvDEA=; b=KgThhqotK7CxsManfMU8z/e8JrcJopkb31oAJWYx6d9Q4rkYK16j+UcjFUa8fXE5qw6xeQJTRlyWhpuU7OlrJwpg+q6luRfCvmreql9gFuAVLtxG8+uwhYlee5OJlvDpWOyHy1lwdK4u9oWjB+KQWNcA8MwYpZv0XlPtKhgepQRRQv5BMpqQDzvIyAkd2If6JRvx/rnQnqdPPi7FvUBXg8pxQ8Ea7bbhAiT+mqiWWto3cb450Hs+GEmEQ32si++p/8gbxTrofLVVM2fgXWP3QY5M4BLP7iaz3FGdr/2BB43YTKJe96EfoN3OBYlwRyXA8KzcyJL/d7Zm5LrV8s93OA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.160) smtp.rcpttodomain=dpdk.org smtp.mailfrom=nvidia.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=aBqHMbDVCQB5qevBkudxUSnuhkaMwatoeIBLXylvDEA=; b=QYWztmLnavFB3C+14qAN2cHE3QtqJKSWcUlk37iUpcnkdwcfFta44NAAO5A3o5EyDvT3o84zrxzURFCXIUbtnBYoXTRo35exNKcHip9TUM91VpHj+Qdij/ktHvUfdIlKUc2ppgO/iTivuN6LozY68Ho2VQLk3Q1A1F2Qnsla1FELSGo2qqovF7EQ6mxvistipcwsNVeIq3BJFHdHI7+Cc+0LOO8L1Ayjwv0KTIV5Q9VCnxjr/j1TdIsjzDTcQ0m5TjllJVfT4C6jOGw5Pfjd9Szeri8O2mXZ699utiw3uYE9LwNxMbpUX6AeVOEQrtcki7fVb9KUw/VypJN80REf7w== Received: from CH5PR05CA0004.namprd05.prod.outlook.com (2603:10b6:610:1f0::9) by PH7PR12MB6883.namprd12.prod.outlook.com (2603:10b6:510:1b9::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7677.30; Mon, 17 Jun 2024 14:08:32 +0000 Received: from CH1PEPF0000AD7B.namprd04.prod.outlook.com (2603:10b6:610:1f0:cafe::34) by CH5PR05CA0004.outlook.office365.com (2603:10b6:610:1f0::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7677.31 via Frontend Transport; Mon, 17 Jun 2024 14:08:32 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.117.160) smtp.mailfrom=nvidia.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.117.160 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.117.160; helo=mail.nvidia.com; pr=C Received: from mail.nvidia.com (216.228.117.160) by CH1PEPF0000AD7B.mail.protection.outlook.com (10.167.244.58) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7677.15 via Frontend Transport; Mon, 17 Jun 2024 14:08:32 +0000 Received: from rnnvmail201.nvidia.com (10.129.68.8) by mail.nvidia.com (10.129.200.66) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.4; Mon, 17 Jun 2024 07:08:16 -0700 Received: from nvidia.com (10.126.231.35) by rnnvmail201.nvidia.com (10.129.68.8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.4; Mon, 17 Jun 2024 07:08:13 -0700 From: Gregory Etelson To: CC: , =?utf-8?b?wqA=?= , , Dariusz Sosnowski , "Viacheslav Ovsiienko" , Ori Kam , Suanming Mou , Matan Azrad Subject: [PATCH 2/2] net/mlx5: fix non-template IPv6 RSS hash Date: Mon, 17 Jun 2024 17:07:47 +0300 Message-ID: <20240617140747.169197-3-getelson@nvidia.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240617140747.169197-1-getelson@nvidia.com> References: <20240617140747.169197-1-getelson@nvidia.com> MIME-Version: 1.0 X-Originating-IP: [10.126.231.35] X-ClientProxiedBy: rnnvmail201.nvidia.com (10.129.68.8) To rnnvmail201.nvidia.com (10.129.68.8) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH1PEPF0000AD7B:EE_|PH7PR12MB6883:EE_ X-MS-Office365-Filtering-Correlation-Id: de47cd80-edf7-42d0-a22b-08dc8ed6f8be X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230037|36860700010|1800799021|82310400023|376011; X-Microsoft-Antispam-Message-Info: /uybyFFSYnFhtZ/PxmAGIUP6yr432yzFB54vCb7gnQSR0/1O6kGOHBTRARHbKx8f0Lr/Kmi8uwCMhTWo+PYwNzL5FYhly9eBRHxH/BOIdTky2voJQfdzQfZX30fkZHjbQhQxIKpqnL5l915HgANmTgNFyOHw/WxSoWgi6oW8anTQFHxxCeVJjD+3mvS2dlnoaifdiCnq6043y8UfgbWYBNlVtTWKOlD86Gbr8pRSS8AM0DpmHHgMp1KR6R+zbtLTHOxaO0R/tLZg2rt0g+gwGj8uow7NqHb/pPJvDYOJiZpHdDRyOgbQkqv8I+Aze1GXCQtD1+M4g22YAyuBEM6DnuZqpWaLIkRlTjzqpXNhjS+7BQa4/S4jTXKZ4kF+XDuXqLPFM7DltivP8WLGNSlHTCGe6hzWentqxGFa0XmDsuE1ZhrajMN7Wb9qrkxvP9tBbscM5i4CL8COHv/AnGExrXG/lJuKcLy1J4CX7DZWdDBPhw4FK3Xe0CuK/1hMurDKouKwl+jYojaBc0S3Mlq4mJ0usyGknbgKLZ868DC/DV9IP/RGDS8G8RQF+JHApC/0E07m2eNY1KK5uEo/pMZqfn/CONUiOoyftF3QOIdJJHcx9RK4RqgSBka40KM5G8MJ+L03q7JAB6hp8mPyih9Ol+sQFhOgah1lXOetGgtreqXNZhKXWxeR11TwW6cx/2XVZE4/h2gji013sty7Ho4b8kubeG3d4oXUtU6We9Bi2lYsam2WO5a6+2N9O6iAK+sMdsQw2eiuDRssENOBq21iDanMML0m0CoOgGLcM9NBMcsF4RH6H+06/ailotLMmv3hPFGAc2C96N/RHvdoU6LS+i2zpgWprThCdmep/8q4N+Q9mt3/9asw1D975KrA/eVqW+SH+sk4zKDQeVTsU6YiZhiAzNHPOMqtmqoCLFcmsdfYjNLEkDDnbSwBzy97gpwz8TIoF9bNhcInXEiBdXKKmeLffRfXyQ1PXIcNnulepTsVtlFy5FAGnpr2Xvikaiq4ilWZalyegCwnUzOhm46mlpJspEJAChuYk5gngbYD5wbOPOgkS1PcCKd37tUatcF0g/e4+LDAdsKSOMZfCLy6Yr1SDJuY4QO6Iy4kUoV5JIREHU2pc23pwG0IquBzxcrVWTTx/FCC9Fy/RS0nMA4gi/i3wm7Hrz6+BYd8dcZ0syR/NyJSxa7Z6z9Uvqj2r2ucf4YHArLDb9tSWw7igPcPDvU2P0iQrrDXPtz8fLUpIxmrGKqfmqHEFLNB5nXMJsvR0OMhgL+ivKi3CzfwTyHDMUZ8Wx2Ztg+WvJwElbuThnYmGEqftKgmmys64Tj34rNN8ITesFWCqe7/O3iDYJFk8+PejM2xylhgghZuX8piNvVLdU/rVlj66Ei//cga8qOeWexMUAk3qCxztp87nYYZzg== X-Forefront-Antispam-Report: CIP:216.228.117.160; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:dc6edge1.nvidia.com; CAT:NONE; SFS:(13230037)(36860700010)(1800799021)(82310400023)(376011); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Jun 2024 14:08:32.5593 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: de47cd80-edf7-42d0-a22b-08dc8ed6f8be X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.117.160]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: CH1PEPF0000AD7B.namprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB6883 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 MLX5 HW prioritizes IPv4 hash over IPv6 in TIR setup. If TIR was configured to hash both IPv4 and IPv6 the MLX5 HW will use IPv4 hash even if packet matched IPv6 flow item. The patch removes IPv4 hash from RSS action configuration if flow rule matched IPV6 item in the non-template PMD setup. Fixes: 1ca9ec661a02 ("net/mlx5: support RSS expansion in non-template HWS setup") Signed-off-by: Gregory Etelson Acked-by: Dariusz Sosnowski --- drivers/net/mlx5/mlx5_nta_rss.c | 91 ++++++++++++++++++++++++++++----- 1 file changed, 78 insertions(+), 13 deletions(-) diff --git a/drivers/net/mlx5/mlx5_nta_rss.c b/drivers/net/mlx5/mlx5_nta_rss.c index bcba2f98ed..602df301ac 100644 --- a/drivers/net/mlx5/mlx5_nta_rss.c +++ b/drivers/net/mlx5/mlx5_nta_rss.c @@ -413,6 +413,54 @@ mlx5_nta_rss_init_ptype_ctx(struct mlx5_nta_rss_ctx *rss_ctx, rss_ctx->external = external; } +static struct rte_flow_hw * +flow_nta_create_single(struct rte_eth_dev *dev, + const struct rte_flow_attr *attr, + const struct rte_flow_item items[], + const struct rte_flow_action actions[], + struct rte_flow_action_rss *rss_conf, + int64_t item_flags, uint64_t action_flags, + bool external, bool copy_actions, + enum mlx5_flow_type flow_type, + struct rte_flow_error *error) +{ + struct rte_flow_hw *flow = NULL; + struct rte_flow_action copy[MLX5_HW_MAX_ACTS]; + const struct rte_flow_action *_actions; + + if (copy_actions) { + int i; + + _actions = copy; + for (i = 0; ; i++) { + copy[i] = actions[i]; + switch (actions[i].type) { + case RTE_FLOW_ACTION_TYPE_RSS: + copy[i].conf = rss_conf; + break; + case RTE_FLOW_ACTION_TYPE_INDIRECT: + if (MLX5_INDIRECT_ACTION_TYPE_GET(actions[i].conf) == + MLX5_INDIRECT_ACTION_TYPE_RSS) { + copy[i].type = RTE_FLOW_ACTION_TYPE_RSS; + copy[i].conf = rss_conf; + } + break; + case RTE_FLOW_ACTION_TYPE_END: + goto end; + default: + break; + } + } + } else { + _actions = actions; + } +end: + flow_hw_create_flow(dev, flow_type, attr, items, + _actions, item_flags, action_flags, + external, &flow, error); + return flow; +} + /* * MLX5 HW hashes IPv4 and IPv6 L3 headers and UDP, TCP, ESP L4 headers. * RSS expansion is required when RSS action was configured to hash @@ -436,10 +484,11 @@ flow_nta_handle_rss(struct rte_eth_dev *dev, struct rte_flow_error *error) { struct rte_flow_hw *rss_base = NULL, *rss_next = NULL, *rss_miss = NULL; - struct rte_flow_action_rss ptype_rss_conf; + struct rte_flow_action_rss ptype_rss_conf = *rss_conf; struct mlx5_nta_rss_ctx rss_ctx; uint64_t rss_types = rte_eth_rss_hf_refine(rss_conf->types); bool expand = true; + bool copy_actions = false; bool inner_rss = rss_conf->level > 1; bool outer_rss = !inner_rss; bool l3_item = (outer_rss && (item_flags & MLX5_FLOW_LAYER_OUTER_L3)) || @@ -479,6 +528,7 @@ flow_nta_handle_rss(struct rte_eth_dev *dev, [MLX5_RSS_PTYPE_ACTION_INDEX + 1] = { .type = RTE_FLOW_ACTION_TYPE_END } }; + ptype_rss_conf.types = rss_types; if (l4_item) { /* * Original flow pattern extended up to L4 level. @@ -497,19 +547,35 @@ flow_nta_handle_rss(struct rte_eth_dev *dev, /* * Original flow pattern extended up to L3 level. * RSS action was not set for L4 hash. - * Original pattern does not need expansion. */ + bool ip4_item = + (outer_rss && (item_flags & MLX5_FLOW_LAYER_OUTER_L3_IPV4)) || + (inner_rss && (item_flags & MLX5_FLOW_LAYER_INNER_L3_IPV4)); + bool ip6_item = + (outer_rss && (item_flags & MLX5_FLOW_LAYER_OUTER_L3_IPV6)) || + (inner_rss && (item_flags & MLX5_FLOW_LAYER_INNER_L3_IPV6)); + bool ip4_hash = rss_types & MLX5_IPV4_LAYER_TYPES; + bool ip6_hash = rss_types & MLX5_IPV6_LAYER_TYPES; + expand = false; + if (ip4_item && ip4_hash) { + ptype_rss_conf.types &= ~MLX5_IPV6_LAYER_TYPES; + copy_actions = true; + } else if (ip6_item && ip6_hash) { + /* + * MLX5 HW will not activate TIR IPv6 hash + * if that TIR has also IPv4 hash + */ + ptype_rss_conf.types &= ~MLX5_IPV4_LAYER_TYPES; + copy_actions = true; + } } } - if (!expand) { - struct rte_flow_hw *flow = NULL; - - flow_hw_create_flow(dev, flow_type, attr, items, - actions, item_flags, action_flags, - external, &flow, error); - return flow; - } + if (!expand) + return flow_nta_create_single(dev, attr, items, actions, + &ptype_rss_conf, item_flags, + action_flags, external, + copy_actions, flow_type, error); /* Create RSS expansions in dedicated PTYPE flow group */ ptype_attr.group = mlx5_hw_get_rss_ptype_group(dev); if (!ptype_attr.group) { @@ -517,11 +583,10 @@ flow_nta_handle_rss(struct rte_eth_dev *dev, NULL, "cannot get RSS PTYPE group"); return NULL; } - ptype_rss_conf = *rss_conf; mlx5_nta_rss_init_ptype_ctx(&rss_ctx, dev, &ptype_attr, ptype_pattern, - ptype_actions, rss_conf, &expansion_head, + ptype_actions, &ptype_rss_conf, &expansion_head, error, item_flags, flow_type, external); - rss_miss = mlx5_hw_rss_ptype_create_miss_flow(dev, rss_conf, ptype_attr.group, + rss_miss = mlx5_hw_rss_ptype_create_miss_flow(dev, &ptype_rss_conf, ptype_attr.group, external, error); if (!rss_miss) goto error;