Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/133595/?format=api
http://patchwork.dpdk.org/api/patches/133595/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20231029163202.216450-22-getelson@nvidia.com/", "project": { "id": 1, "url": "http://patchwork.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": "<20231029163202.216450-22-getelson@nvidia.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20231029163202.216450-22-getelson@nvidia.com", "date": "2023-10-29T16:31:54", "name": "[22/30] net/mlx5/hws: allow jump to TIR over FDB", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "cf776d7087b434e50e85a128eb1e14eca7cf2960", "submitter": { "id": 1882, "url": "http://patchwork.dpdk.org/api/people/1882/?format=api", "name": "Gregory Etelson", "email": "getelson@nvidia.com" }, "delegate": { "id": 3268, "url": "http://patchwork.dpdk.org/api/users/3268/?format=api", "username": "rasland", "first_name": "Raslan", "last_name": "Darawsheh", "email": "rasland@nvidia.com" }, "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20231029163202.216450-22-getelson@nvidia.com/mbox/", "series": [ { "id": 30049, "url": "http://patchwork.dpdk.org/api/series/30049/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=30049", "date": "2023-10-29T16:31:33", "name": "[01/30] net/mlx5/hws: Definer, add mlx5dr context to definer_conv_data", "version": 1, "mbox": "http://patchwork.dpdk.org/series/30049/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/133595/comments/", "check": "warning", "checks": "http://patchwork.dpdk.org/api/patches/133595/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 D0FFD43238;\n\tSun, 29 Oct 2023 17:35:45 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 980A540E96;\n\tSun, 29 Oct 2023 17:33:43 +0100 (CET)", "from NAM10-MW2-obe.outbound.protection.outlook.com\n (mail-mw2nam10on2048.outbound.protection.outlook.com [40.107.94.48])\n by mails.dpdk.org (Postfix) with ESMTP id 061D842DCB\n for <dev@dpdk.org>; Sun, 29 Oct 2023 17:33:42 +0100 (CET)", "from BL1PR13CA0104.namprd13.prod.outlook.com (2603:10b6:208:2b9::19)\n by IA0PR12MB7627.namprd12.prod.outlook.com (2603:10b6:208:437::12)\n with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6933.27; Sun, 29 Oct\n 2023 16:33:39 +0000", "from BL02EPF0001A100.namprd03.prod.outlook.com\n (2603:10b6:208:2b9:cafe::b7) by BL1PR13CA0104.outlook.office365.com\n (2603:10b6:208:2b9::19) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6954.15 via Frontend\n Transport; Sun, 29 Oct 2023 16:33:39 +0000", "from mail.nvidia.com (216.228.117.161) by\n BL02EPF0001A100.mail.protection.outlook.com (10.167.242.107) with Microsoft\n SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id\n 15.20.6933.15 via Frontend Transport; Sun, 29 Oct 2023 16:33:39 +0000", "from rnnvmail201.nvidia.com (10.129.68.8) by mail.nvidia.com\n (10.129.200.67) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.41; Sun, 29 Oct\n 2023 09:33:28 -0700", "from nvidia.com (10.126.231.35) by rnnvmail201.nvidia.com\n (10.129.68.8) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.41; Sun, 29 Oct\n 2023 09:33:25 -0700" ], "ARC-Seal": "i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=WWLiEj4VsGH4LZ9kQ+OK3AoVgtjKAmoKkTIaUMiky6wbMsIYfcUhUwDBUkjyI5dceVb68CdqWCzvAyXXtX5TpSGi8e2kyA1WhNkh8QB2DfjWcdAjcNBed0u0ilPiYusQw7naxV7W4SaKlnPkzMbp2+sP1HHYn/ujAfsN4UHdXwWtM396HFBPL7r2mhe+KllS+eLMQ/Mo55XkI//1d1GDP4EsEW5ydfghy5RS0shmwCoKLgneb8XmNPnzef69LHAfmUwXD3pBMIZSCkbWmgoCVVEdc7txjTToQD4KMwDsvk5PP5axAeUmhIjiIJ7leJ0KEA5oZGl7kO805GDLrIErvA==", "ARC-Message-Signature": "i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector9901;\n 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;\n bh=D0FH8a9CyviaqAyWMFP8igj/NO06p9+bY7HeX9M6+QQ=;\n b=F+GvKxmpoXP2hlP29CEQD+p1WH4UP7v6MCtroxzsokuTJHNvl5dZULmy+nnVz87zgy3181d6IjrHtOUbkPCaVXbQHblSovczXgSD7fqe12pTh7hZJvlIwbholItfau2b1Ysgphvz99P5l/qLr6dAn9V+94SMVeVGNYLehDfrovXZ5dvE8K2vjELMxbyvDVNkt3si6p9QFh5MW4tcWCNgwZIjOokLTGarGktHL68RoozfzJhNmEsMthDx0xW9qAgDnT3bwRqAunc4osTcQ+oFmihd+KFNa5c2RCmfxiXgwcdv1duDaWt2GpdkcefLciNBaN1V8tpUtQo5dN2Yx/UvLA==", "ARC-Authentication-Results": "i=1; mx.microsoft.com 1; spf=pass (sender ip is\n 216.228.117.161) smtp.rcpttodomain=dpdk.org smtp.mailfrom=nvidia.com;\n dmarc=pass (p=reject sp=reject pct=100) action=none header.from=nvidia.com;\n dkim=none (message not signed); arc=none (0)", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com;\n s=selector2;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=D0FH8a9CyviaqAyWMFP8igj/NO06p9+bY7HeX9M6+QQ=;\n b=kzSxn0Y6V3+Rxkd8uHs53h9DqDk+RXkkC8LGhl4Qin2MmL60xLF1/0T2C5Z9Q3KqYy4hiN9B/9ki/l/4Uk1aKJYNhUEG4g9vevmtgYgcw8CE8v3TaYegRJhE5ynu1lizW96L33vDreu66ns2BhVZA1UgRaZqG2khdjwybpxrvF/xb+yBzQ0UZTyfk9707WAu1rGdZkgCtNomlicAGhEyjudEa3TsB1hsaWIB13dn0NIdSFxlTOZzgvRatnVhb1yn1oVVwGsiPzBIGvwbh+GBssTvrzTLaEe7KIyoCgEMHj+ZRrdrobH7Ucm58+cHgnzF9Qf7nYQeK35hIqfKaeBRDw==", "X-MS-Exchange-Authentication-Results": "spf=pass (sender IP is 216.228.117.161)\n smtp.mailfrom=nvidia.com;\n dkim=none (message not signed)\n header.d=none;dmarc=pass action=none header.from=nvidia.com;", "Received-SPF": "Pass (protection.outlook.com: domain of nvidia.com designates\n 216.228.117.161 as permitted sender) receiver=protection.outlook.com;\n client-ip=216.228.117.161; helo=mail.nvidia.com; pr=C", "From": "Gregory Etelson <getelson@nvidia.com>", "To": "<dev@dpdk.org>", "CC": "<getelson@nvidia.com>, <mkashani@nvidia.com>, <rasland@nvidia.com>, \"Alex\n Vesker\" <valex@nvidia.com>, Erez Shitrit <erezsh@nvidia.com>, Matan Azrad\n <matan@nvidia.com>, Viacheslav Ovsiienko <viacheslavo@nvidia.com>, Ori Kam\n <orika@nvidia.com>, Suanming Mou <suanmingm@nvidia.com>", "Subject": "[PATCH 22/30] net/mlx5/hws: allow jump to TIR over FDB", "Date": "Sun, 29 Oct 2023 18:31:54 +0200", "Message-ID": "<20231029163202.216450-22-getelson@nvidia.com>", "X-Mailer": "git-send-email 2.39.2", "In-Reply-To": "<20231029163202.216450-1-getelson@nvidia.com>", "References": "<20231029163202.216450-1-getelson@nvidia.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Content-Type": "text/plain", "X-Originating-IP": "[10.126.231.35]", "X-ClientProxiedBy": "rnnvmail203.nvidia.com (10.129.68.9) To\n rnnvmail201.nvidia.com (10.129.68.8)", "X-EOPAttributedMessage": "0", "X-MS-PublicTrafficType": "Email", "X-MS-TrafficTypeDiagnostic": "BL02EPF0001A100:EE_|IA0PR12MB7627:EE_", "X-MS-Office365-Filtering-Correlation-Id": "41a0a8b5-df7e-4ea5-8831-08dbd89cced5", "X-MS-Exchange-SenderADCheck": "1", "X-MS-Exchange-AntiSpam-Relay": "0", "X-Microsoft-Antispam": "BCL:0;", "X-Microsoft-Antispam-Message-Info": "\n h5ai6X8h6WHFrJVrjHWAfV3vk21Qvv9qPDyySjXTXnfdc2e2Fd/V2Ko/CArJBKSGYAJPoblcJjNHWtY3g6Pp/FxMopKw/rQPBZUIjCSI+yPERnIt0kzwGpcXWmMBdIw0Zz8MFRq+J3r6ClzYX5kTR+c1+O/rDRaCvcruLU3rhE9SfVT/Ki3ZRhke/Pg1dxy9+OYUqP8sbioajapVr7AgE9j+gRQAyRavcki+6+tk5x9CNQZfUu0ORnE2TBYsqChhVaUiBS/CSQCi5N+XONHMH4XhRy44pxmH0OoXQEFPpet2dcyBtn0+ajUmst99dAZdtpUaHLzU2cJU6K88B4Qb7YTNBPPctmJR8rx2CbR1wFcGOBGQG5VRN6d+DvkGv1qcJ2CZAsOsCKHLeDrFq2yShJR75tX5PQzlt3DBRB7y83vo4gu7yUeGB668OMDHQCTJDXkXYfMKgQz7TgfJuTTcRScBAe52f/dLnFJmVNLwjZX+IiVX4p8nrIhZay8Zoqb2XcVKap/L/lS5gzgFwaY7bvsppoYF9qJBI+rqa6m+H081bQfeimE6PM+7gycnHyDdyw/bHYqx4pHKw6utrvkdYR2RSqZ4FyyId20MaX4d5gt14IXtteWVEagVIioTeP36hgyUxeleRuetqbXI/VdJJ0MuMAX8oFvGmFlsPA90GiIHIEO/7jPuA70L9RZV4e5lB91B7iAwLGsg2DK7CqixL3kGLsRBAxTKz3LddUiTZ1M4jurNtBpQB0Odpo3yKgwA", "X-Forefront-Antispam-Report": "CIP:216.228.117.161; CTRY:US; LANG:en; SCL:1;\n SRV:;\n IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:dc6edge2.nvidia.com; CAT:NONE;\n SFS:(13230031)(4636009)(396003)(346002)(39860400002)(376002)(136003)(230922051799003)(451199024)(64100799003)(186009)(1800799009)(82310400011)(40470700004)(46966006)(36840700001)(55016003)(40460700003)(2906002)(36860700001)(6916009)(54906003)(70586007)(70206006)(47076005)(356005)(7636003)(82740400003)(316002)(6666004)(7696005)(478600001)(83380400001)(2616005)(107886003)(336012)(426003)(16526019)(6286002)(1076003)(26005)(41300700001)(5660300002)(8936002)(8676002)(4326008)(86362001)(40480700001)(36756003);\n DIR:OUT; SFP:1101;", "X-OriginatorOrg": "Nvidia.com", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "29 Oct 2023 16:33:39.7624 (UTC)", "X-MS-Exchange-CrossTenant-Network-Message-Id": "\n 41a0a8b5-df7e-4ea5-8831-08dbd89cced5", "X-MS-Exchange-CrossTenant-Id": "43083d15-7273-40c1-b7db-39efd9ccc17a", "X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp": "\n TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.117.161];\n Helo=[mail.nvidia.com]", "X-MS-Exchange-CrossTenant-AuthSource": "\n BL02EPF0001A100.namprd03.prod.outlook.com", "X-MS-Exchange-CrossTenant-AuthAs": "Anonymous", "X-MS-Exchange-CrossTenant-FromEntityHeader": "HybridOnPrem", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "IA0PR12MB7627", "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": "From: Alex Vesker <valex@nvidia.com>\n\nCurrent TIR action is allowed to be used only for NIC RX,\nthis will allow TIR action over FDB for RX traffic in case\nof TX traffic packets will be dropped.\n\nSigned-off-by: Alex Vesker <valex@nvidia.com>\nReviewed-by: Erez Shitrit <erezsh@nvidia.com>\nAcked-by: Matan Azrad <matan@nvidia.com>\n---\n drivers/common/mlx5/mlx5_prm.h | 2 ++\n drivers/net/mlx5/hws/mlx5dr_action.c | 27 ++++++++++++++++++++++-----\n drivers/net/mlx5/hws/mlx5dr_cmd.c | 4 ++++\n drivers/net/mlx5/hws/mlx5dr_cmd.h | 1 +\n 4 files changed, 29 insertions(+), 5 deletions(-)", "diff": "diff --git a/drivers/common/mlx5/mlx5_prm.h b/drivers/common/mlx5/mlx5_prm.h\nindex 40e461cb82..bb2b990d5b 100644\n--- a/drivers/common/mlx5/mlx5_prm.h\n+++ b/drivers/common/mlx5/mlx5_prm.h\n@@ -2418,6 +2418,8 @@ struct mlx5_ifc_wqe_based_flow_table_cap_bits {\n \tu8 reserved_at_180[0x10];\n \tu8 ste_format_gen_wqe[0x10];\n \tu8 linear_match_definer_reg_c3[0x20];\n+\tu8 fdb_jump_to_tir_stc[0x1];\n+\tu8 reserved_at_1c1[0x1f];\n };\n \n union mlx5_ifc_hca_cap_union_bits {\ndiff --git a/drivers/net/mlx5/hws/mlx5dr_action.c b/drivers/net/mlx5/hws/mlx5dr_action.c\nindex 1a6296a728..05b6e97576 100644\n--- a/drivers/net/mlx5/hws/mlx5dr_action.c\n+++ b/drivers/net/mlx5/hws/mlx5dr_action.c\n@@ -445,6 +445,7 @@ mlx5dr_action_fixup_stc_attr(struct mlx5dr_context *ctx,\n \t\tbreak;\n \n \tcase MLX5_IFC_STC_ACTION_TYPE_CRYPTO_IPSEC_ENCRYPTION:\n+\t\t/* Encrypt is allowed on RX side, requires mask in case of FDB */\n \t\tif (fw_tbl_type == FS_FT_FDB_RX) {\n \t\t\tfixup_stc_attr->action_type = MLX5_IFC_STC_ACTION_TYPE_NOP;\n \t\t\tfixup_stc_attr->action_offset = stc_attr->action_offset;\n@@ -454,6 +455,7 @@ mlx5dr_action_fixup_stc_attr(struct mlx5dr_context *ctx,\n \t\tbreak;\n \n \tcase MLX5_IFC_STC_ACTION_TYPE_CRYPTO_IPSEC_DECRYPTION:\n+\t\t/* Decrypt is allowed on TX side, requires mask in case of FDB */\n \t\tif (fw_tbl_type == FS_FT_FDB_TX) {\n \t\t\tfixup_stc_attr->action_type = MLX5_IFC_STC_ACTION_TYPE_NOP;\n \t\t\tfixup_stc_attr->action_offset = stc_attr->action_offset;\n@@ -463,12 +465,10 @@ mlx5dr_action_fixup_stc_attr(struct mlx5dr_context *ctx,\n \t\tbreak;\n \n \tcase MLX5_IFC_STC_ACTION_TYPE_TRAILER:\n-\t\tif (table_type != MLX5DR_TABLE_TYPE_FDB)\n-\t\t\tbreak;\n-\n+\t\t/* Trailer has FDB limitations on RX and TX based on operation */\n \t\tval = stc_attr->reformat_trailer.op;\n-\t\tif ((val == MLX5DR_ACTION_TRAILER_OP_INSERT && !is_mirror) ||\n-\t\t (val == MLX5DR_ACTION_TRAILER_OP_REMOVE && is_mirror)) {\n+\t\tif ((val == MLX5DR_ACTION_TRAILER_OP_INSERT && fw_tbl_type == FS_FT_FDB_RX) ||\n+\t\t (val == MLX5DR_ACTION_TRAILER_OP_REMOVE && fw_tbl_type == FS_FT_FDB_TX)) {\n \t\t\tfixup_stc_attr->action_type = MLX5_IFC_STC_ACTION_TYPE_NOP;\n \t\t\tfixup_stc_attr->action_offset = stc_attr->action_offset;\n \t\t\tfixup_stc_attr->stc_offset = stc_attr->stc_offset;\n@@ -476,6 +476,16 @@ mlx5dr_action_fixup_stc_attr(struct mlx5dr_context *ctx,\n \t\t}\n \t\tbreak;\n \n+\tcase MLX5_IFC_STC_ACTION_TYPE_JUMP_TO_TIR:\n+\t\t/* TIR is allowed on RX side, requires mask in case of FDB */\n+\t\tif (fw_tbl_type == FS_FT_FDB_TX) {\n+\t\t\tfixup_stc_attr->action_type = MLX5_IFC_STC_ACTION_TYPE_DROP;\n+\t\t\tfixup_stc_attr->action_offset = MLX5DR_ACTION_OFFSET_HIT;\n+\t\t\tfixup_stc_attr->stc_offset = stc_attr->stc_offset;\n+\t\t\tuse_fixup = true;\n+\t\t}\n+\t\tbreak;\n+\n \tdefault:\n \t\tbreak;\n \t}\n@@ -976,6 +986,13 @@ mlx5dr_action_create_dest_tir(struct mlx5dr_context *ctx,\n \t\treturn NULL;\n \t}\n \n+\tif ((flags & MLX5DR_ACTION_FLAG_ROOT_FDB) ||\n+\t (flags & MLX5DR_ACTION_FLAG_HWS_FDB && !ctx->caps->fdb_tir_stc)) {\n+\t\tDR_LOG(ERR, \"TIR action not support on FDB\");\n+\t\trte_errno = ENOTSUP;\n+\t\treturn NULL;\n+\t}\n+\n \tif (!is_local) {\n \t\tDR_LOG(ERR, \"TIR should be created on local ibv_device, flags: 0x%x\",\n \t\t flags);\ndiff --git a/drivers/net/mlx5/hws/mlx5dr_cmd.c b/drivers/net/mlx5/hws/mlx5dr_cmd.c\nindex 0ba4774f08..135d31dca1 100644\n--- a/drivers/net/mlx5/hws/mlx5dr_cmd.c\n+++ b/drivers/net/mlx5/hws/mlx5dr_cmd.c\n@@ -1275,6 +1275,10 @@ int mlx5dr_cmd_query_caps(struct ibv_context *ctx,\n \t\tcaps->supp_ste_format_gen_wqe = MLX5_GET(query_hca_cap_out, out,\n \t\t\t\t\t\t\t capability.wqe_based_flow_table_cap.\n \t\t\t\t\t\t\t ste_format_gen_wqe);\n+\n+\t\tcaps->fdb_tir_stc = MLX5_GET(query_hca_cap_out, out,\n+\t\t\t\t\t capability.wqe_based_flow_table_cap.\n+\t\t\t\t\t fdb_jump_to_tir_stc);\n \t}\n \n \tif (caps->eswitch_manager) {\ndiff --git a/drivers/net/mlx5/hws/mlx5dr_cmd.h b/drivers/net/mlx5/hws/mlx5dr_cmd.h\nindex c082157538..cb27212a5b 100644\n--- a/drivers/net/mlx5/hws/mlx5dr_cmd.h\n+++ b/drivers/net/mlx5/hws/mlx5dr_cmd.h\n@@ -241,6 +241,7 @@ struct mlx5dr_cmd_query_caps {\n \tuint8_t log_header_modify_argument_granularity;\n \tuint8_t log_header_modify_argument_max_alloc;\n \tuint8_t sq_ts_format;\n+\tuint8_t fdb_tir_stc;\n \tuint64_t definer_format_sup;\n \tuint32_t trivial_match_definer;\n \tuint32_t vhca_id;\n", "prefixes": [ "22/30" ] }{ "id": 133595, "url": "