Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/92279/?format=api
http://patchwork.dpdk.org/api/patches/92279/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20210427153811.11554-15-bingz@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": "<20210427153811.11554-15-bingz@nvidia.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20210427153811.11554-15-bingz@nvidia.com", "date": "2021-04-27T15:38:08", "name": "[14/17] net/mlx5: validation of CT action", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "5877de2460fcbc2357641a4ba53d3ffc823358bf", "submitter": { "id": 1976, "url": "http://patchwork.dpdk.org/api/people/1976/?format=api", "name": "Bing Zhao", "email": "bingz@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/20210427153811.11554-15-bingz@nvidia.com/mbox/", "series": [ { "id": 16705, "url": "http://patchwork.dpdk.org/api/series/16705/?format=api", "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=16705", "date": "2021-04-27T15:37:54", "name": "conntrack support in mlx5 PMD", "version": 1, "mbox": "http://patchwork.dpdk.org/series/16705/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/92279/comments/", "check": "success", "checks": "http://patchwork.dpdk.org/api/patches/92279/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 7E7EBA0A02;\n\tTue, 27 Apr 2021 17:40:18 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 54685412AB;\n\tTue, 27 Apr 2021 17:39:19 +0200 (CEST)", "from NAM04-BN8-obe.outbound.protection.outlook.com\n (mail-bn8nam08on2087.outbound.protection.outlook.com [40.107.100.87])\n by mails.dpdk.org (Postfix) with ESMTP id 9F14341273\n for <dev@dpdk.org>; Tue, 27 Apr 2021 17:39:16 +0200 (CEST)", "from MW4PR03CA0318.namprd03.prod.outlook.com (2603:10b6:303:dd::23)\n by DM4PR12MB5136.namprd12.prod.outlook.com (2603:10b6:5:393::23) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4065.22; Tue, 27 Apr\n 2021 15:39:15 +0000", "from CO1NAM11FT066.eop-nam11.prod.protection.outlook.com\n (2603:10b6:303:dd:cafe::b0) by MW4PR03CA0318.outlook.office365.com\n (2603:10b6:303:dd::23) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4087.25 via Frontend\n Transport; Tue, 27 Apr 2021 15:39:15 +0000", "from mail.nvidia.com (216.228.112.34) by\n CO1NAM11FT066.mail.protection.outlook.com (10.13.175.18) with Microsoft SMTP\n Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id\n 15.20.4065.21 via Frontend Transport; Tue, 27 Apr 2021 15:39:14 +0000", "from nvidia.com (172.20.145.6) by HQMAIL107.nvidia.com\n (172.20.187.13) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Tue, 27 Apr\n 2021 15:39:13 +0000" ], "ARC-Seal": "i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=Xw8FlpaAzuw6AyPhcJc0vL9THS7kwoEurC4UqZOAysN1kqYjjJ5sqf1XWVlmR6umFWI7d3Z+5/xm9C6ldgVCab9L6qObbpv2tzQCM6EFzPnh6pbf5XocBOHmstE19Kvd5nimngqAulSnYjna/mEKozgr+ApA7ebDPj3ygM+RvaaWAbNnaFpDmBogyV/T/eWNX2de5isY9J71+Dxxc1Uxe6ejhqCVIDdbhka5P4KVRHMi0IEfjOyCGGDd45H4kvsEWh6/OR077BcQbhcsrFj6Gydxc4xJZexDn3q0ZtT6S9oG263NFLuK4TQ1+kUHW4xzy0daQffD10mPjcZ2AocpBw==", "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-SenderADCheck;\n bh=7PLdZW3goDRrnVAs9kIhM4OYia0Q6TQNDTC6dy6Ypu8=;\n b=KphIybyVzVddF+AUmW+umyEWqBktwdy9BOw6Ako7hRfqcPgxfXms20dKXD2Bh5kcdhB8QmY4RLhM6bA2qr9Id9M1ysLcP7VHFgGYzDeIq6A0fy5a+BUstHMDkPAs1VRbmq93cQn25VMs1rg+/47YP3KQJovgQr5RqE9+fLCz37wetup3jVYbfspnD15M5DsDsBjyny6FeyoF5OemqLqwv8hmf1CkS1RmsMfDOZyEMxeDS1Alf1bbPqqNeXDmKBgB3IvevDCgUoYRD2V7huTqCNGUGBAiaeYZUFpQleClAATTHVoNkNsZ3Oc2d+WF3MtadHsBkkUw4mTDwKSZz3W9zw==", "ARC-Authentication-Results": "i=1; mx.microsoft.com 1; spf=pass (sender ip is\n 216.228.112.34) smtp.rcpttodomain=dpdk.org smtp.mailfrom=nvidia.com;\n dmarc=pass (p=none sp=none pct=100) action=none header.from=nvidia.com;\n dkim=none (message not signed); arc=none", "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=7PLdZW3goDRrnVAs9kIhM4OYia0Q6TQNDTC6dy6Ypu8=;\n b=pJIvY4aL4UM5ot9U5Hoftm7NampmNA4C0oWOEHfrCmKaCJNQhm+u55Ywr7JUQtDzDzsBzY4xRL4OUogzrufm70qNRAZ3Kos2H490NZyuKkwET1R+Iy3hwcrzobtKaugAX2NX7j0vKPUyHhGuE2yVH9bCuooohU9qQ4+iLpcW04f88CQ9HMOTMtiREzNWZ0DzJ/1et9/kqXmDfuOvnHjAHtRKA0pu70Ra1swdEtF1tcT+cGde1/vf7+nAdsiT6fOVyhE/iDg6bA9ULVOe+1t3QgG8Sd7QCVvCVNpdc7crDxx/Hji1XNpETiuELx1WWNvInwTvRwGZfKYOe8IJ2uARDA==", "X-MS-Exchange-Authentication-Results": "spf=pass (sender IP is 216.228.112.34)\n smtp.mailfrom=nvidia.com; dpdk.org; dkim=none (message not signed)\n header.d=none;dpdk.org; dmarc=pass action=none header.from=nvidia.com;", "Received-SPF": "Pass (protection.outlook.com: domain of nvidia.com designates\n 216.228.112.34 as permitted sender) receiver=protection.outlook.com;\n client-ip=216.228.112.34; helo=mail.nvidia.com;", "From": "Bing Zhao <bingz@nvidia.com>", "To": "<viacheslavo@nvidia.com>, <matan@nvidia.com>", "CC": "<dev@dpdk.org>, <orika@nvidia.com>, <rasland@nvidia.com>", "Date": "Tue, 27 Apr 2021 18:38:08 +0300", "Message-ID": "<20210427153811.11554-15-bingz@nvidia.com>", "X-Mailer": "git-send-email 2.27.0", "In-Reply-To": "<20210427153811.11554-1-bingz@nvidia.com>", "References": "<20210427153811.11554-1-bingz@nvidia.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Content-Type": "text/plain", "X-Originating-IP": "[172.20.145.6]", "X-ClientProxiedBy": "HQMAIL107.nvidia.com (172.20.187.13) To\n HQMAIL107.nvidia.com (172.20.187.13)", "X-EOPAttributedMessage": "0", "X-MS-PublicTrafficType": "Email", "X-MS-Office365-Filtering-Correlation-Id": "89523ce2-0aa7-43d0-09cc-08d909929cc7", "X-MS-TrafficTypeDiagnostic": "DM4PR12MB5136:", "X-Microsoft-Antispam-PRVS": "\n <DM4PR12MB51362AAB5DDE509BCE4EEACED0419@DM4PR12MB5136.namprd12.prod.outlook.com>", "X-MS-Oob-TLC-OOBClassifiers": "OLM:8273;", "X-MS-Exchange-SenderADCheck": "1", "X-Microsoft-Antispam": "BCL:0;", "X-Microsoft-Antispam-Message-Info": "\n TgTSPbNrfUPyUZNRsEsBsA7X20N9VsQikZOVJ5XTvRk+MchODjVadCHiWJg7z5X3GueyRJXfVsSwNiqS6vxpTxpbo/5BaxjZKgZcNoaKBqxZOZe/JF/qvLiJZEs6El7U1/fCzCKKwwfucp/EdRV2zWEFtEUgfFMOMr6syDK+xpGU6Q1wKHdWEafommfLGqbwbqnLHizk7M6WEQNbXqWvZAb1k2Eia/M+N8irECs6niCH8muUjBpIZv05NigcxUdDJ+reDh8CanfATl0Vi+vnb6xqpbGYwNbFRyhkLk8AAGPhuRgrTOKLQ3rq3hpWkA88uj+NHCyEmhx7tuueEKJ3veAGUtEe6pVRWGWRd3uT9l52LX+sROJtyGVnUm1jOhvTloFEArZpsXm4Pn0iBpJi8sRAPGu4hQvd41MpuOI2zr3ScpBVWuqpM4vp5Eg4g2FD+gwYbFLjiE3BrqcJFgb9x0kIf4rFltbX+0Rdq9xO2JzWRpoJcCcr7hRpj5ihvsIu9XyNgay0olnH6O7hwaAhRDQoOO41h3DVNwqsDP3MuDwafBJy0KfLOEBIMWWITIFHMNE1Qmj8N4XQ/bLxCsDiL7sF2iL4rAHjyKAyakUnz54J+KBm7sYaCJ9F1O533yNm2fxllGUojOwMUBdB7dLDSjkVbQI6ITRIHeIQIz5GZdmDtpT2WgLPgSxB8Tp1aSxN", "X-Forefront-Antispam-Report": "CIP:216.228.112.34; CTRY:US; LANG:en; SCL:1;\n SRV:;\n IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:schybrid03.nvidia.com; CAT:NONE;\n SFS:(4636009)(396003)(136003)(376002)(346002)(39860400002)(36840700001)(46966006)(107886003)(55016002)(6286002)(6636002)(26005)(7696005)(110136005)(70586007)(5660300002)(478600001)(16526019)(47076005)(82310400003)(70206006)(1076003)(4326008)(426003)(186003)(8936002)(2616005)(336012)(82740400003)(6666004)(83380400001)(2906002)(54906003)(7636003)(86362001)(36756003)(356005)(36906005)(36860700001)(8676002)(316002)(309714004);\n DIR:OUT; SFP:1101;", "X-OriginatorOrg": "Nvidia.com", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "27 Apr 2021 15:39:14.8671 (UTC)", "X-MS-Exchange-CrossTenant-Network-Message-Id": "\n 89523ce2-0aa7-43d0-09cc-08d909929cc7", "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.112.34];\n Helo=[mail.nvidia.com]", "X-MS-Exchange-CrossTenant-AuthSource": "\n CO1NAM11FT066.eop-nam11.prod.protection.outlook.com", "X-MS-Exchange-CrossTenant-AuthAs": "Anonymous", "X-MS-Exchange-CrossTenant-FromEntityHeader": "HybridOnPrem", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "DM4PR12MB5136", "Subject": "[dpdk-dev] [PATCH 14/17] net/mlx5: validation of CT action", "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", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "The validation of a CT action contains two parts. The first is the\nCT action configurations parameter. When creating a CT action\ncontext, some members need to be verified.\n\nThe second is that when creating a flow, the DR action of CT should\nbe validated with other actions and items as well. Currently, only\nthe TCP protocol support connection tracking.\n\nSigned-off-by: Bing Zhao <bingz@nvidia.com>\n---\n drivers/net/mlx5/mlx5.h | 4 +++\n drivers/net/mlx5/mlx5_flow.c | 31 +++++++++++++++++++\n drivers/net/mlx5/mlx5_flow_dv.c | 68 +++++++++++++++++++++++++++++++++++++++++\n 3 files changed, 103 insertions(+)", "diff": "diff --git a/drivers/net/mlx5/mlx5.h b/drivers/net/mlx5/mlx5.h\nindex 3b67706..e983897 100644\n--- a/drivers/net/mlx5/mlx5.h\n+++ b/drivers/net/mlx5/mlx5.h\n@@ -1617,6 +1617,10 @@ int mlx5_flow_dev_dump(struct rte_eth_dev *dev, struct rte_flow *flow,\n void mlx5_flow_rxq_dynf_metadata_set(struct rte_eth_dev *dev);\n int mlx5_flow_get_aged_flows(struct rte_eth_dev *dev, void **contexts,\n \t\t\tuint32_t nb_contexts, struct rte_flow_error *error);\n+int mlx5_validate_action_ct(struct rte_eth_dev *dev,\n+\t\t\t const struct rte_flow_action_conntrack *conntrack,\n+\t\t\t struct rte_flow_error *error);\n+\n \n /* mlx5_mp_os.c */\n \ndiff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c\nindex 7b9f055..ee82243 100644\n--- a/drivers/net/mlx5/mlx5_flow.c\n+++ b/drivers/net/mlx5/mlx5_flow.c\n@@ -1687,6 +1687,37 @@ mlx5_flow_validate_action_count(struct rte_eth_dev *dev __rte_unused,\n \treturn 0;\n }\n \n+/*\n+ * Validate the ASO CT action.\n+ *\n+ * @param[in] dev\n+ * Pointer to the Ethernet device structure.\n+ * @param[in] conntrack\n+ * Pointer to the CT action profile.\n+ * @param[out] error\n+ * Pointer to error structure.\n+ *\n+ * @return\n+ * 0 on success, a negative errno value otherwise and rte_errno is set.\n+ */\n+int\n+mlx5_validate_action_ct(struct rte_eth_dev *dev,\n+\t\t\tconst struct rte_flow_action_conntrack *conntrack,\n+\t\t\tstruct rte_flow_error *error)\n+{\n+\tRTE_SET_USED(dev);\n+\n+\tif (conntrack->state > RTE_FLOW_CONNTRACK_STATE_TIME_WAIT)\n+\t\treturn rte_flow_error_set(error, EINVAL,\n+\t\t\t\t\t RTE_FLOW_ERROR_TYPE_ACTION, NULL,\n+\t\t\t\t\t \"Invalid CT state\");\n+\tif (conntrack->last_index > RTE_FLOW_CONNTRACK_FLAG_RST)\n+\t\treturn rte_flow_error_set(error, EINVAL,\n+\t\t\t\t\t RTE_FLOW_ERROR_TYPE_ACTION, NULL,\n+\t\t\t\t\t \"Invalid last TCP packet flag\");\n+\treturn 0;\n+}\n+\n /**\n * Verify the @p attributes will be correctly understood by the NIC and store\n * them in the @p flow if everything is correct.\ndiff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_dv.c\nindex a27aff6..2bf966c 100644\n--- a/drivers/net/mlx5/mlx5_flow_dv.c\n+++ b/drivers/net/mlx5/mlx5_flow_dv.c\n@@ -3400,6 +3400,57 @@ flow_dv_validate_action_raw_encap_decap\n \treturn 0;\n }\n \n+/*\n+ * Validate the ASO CT action.\n+ *\n+ * @param[in] dev\n+ * Pointer to the rte_eth_dev structure.\n+ * @param[in] action_flags\n+ * Holds the actions detected until now.\n+ * @param[in] item_flags\n+ * The items found in this flow rule.\n+ * @param[in] attr\n+ * Pointer to flow attributes.\n+ * @param[out] error\n+ * Pointer to error structure.\n+ *\n+ * @return\n+ * 0 on success, a negative errno value otherwise and rte_errno is set.\n+ */\n+static int\n+flow_dv_validate_action_aso_ct(struct rte_eth_dev *dev,\n+\t\t\t uint64_t action_flags,\n+\t\t\t uint64_t item_flags,\n+\t\t\t const struct rte_flow_attr *attr,\n+\t\t\t struct rte_flow_error *error)\n+{\n+\tRTE_SET_USED(dev);\n+\n+\tif (attr->group == 0 && !attr->transfer)\n+\t\treturn rte_flow_error_set(error, ENOTSUP,\n+\t\t\t\t\t RTE_FLOW_ERROR_TYPE_UNSPECIFIED,\n+\t\t\t\t\t NULL,\n+\t\t\t\t\t \"Only support non-root table\");\n+\tif (action_flags & MLX5_FLOW_FATE_ACTIONS)\n+\t\treturn rte_flow_error_set(error, ENOTSUP,\n+\t\t\t\t\t RTE_FLOW_ERROR_TYPE_ACTION, NULL,\n+\t\t\t\t\t \"CT cannot follow a fate action\");\n+\tif ((action_flags & MLX5_FLOW_ACTION_METER) ||\n+\t (action_flags & MLX5_FLOW_ACTION_AGE))\n+\t\treturn rte_flow_error_set(error, EINVAL,\n+\t\t\t\t\t RTE_FLOW_ERROR_TYPE_ACTION, NULL,\n+\t\t\t\t\t \"Only one ASO action is supported\");\n+\tif (action_flags & MLX5_FLOW_ACTION_ENCAP)\n+\t\treturn rte_flow_error_set(error, EINVAL,\n+\t\t\t\t\t RTE_FLOW_ERROR_TYPE_ACTION, NULL,\n+\t\t\t\t\t \"Encap cannot exist before CT\");\n+\tif (!(item_flags & MLX5_FLOW_LAYER_OUTER_L4_TCP))\n+\t\treturn rte_flow_error_set(error, EINVAL,\n+\t\t\t\t\t RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,\n+\t\t\t\t\t \"Not a outer TCP packet\");\n+\treturn 0;\n+}\n+\n /**\n * Match encap_decap resource.\n *\n@@ -7204,6 +7255,14 @@ flow_dv_validate(struct rte_eth_dev *dev, const struct rte_flow_attr *attr,\n \t\t\taction_flags |= MLX5_FLOW_ACTION_MODIFY_FIELD;\n \t\t\trw_act_num += ret;\n \t\t\tbreak;\n+\t\tcase RTE_FLOW_ACTION_TYPE_CONNTRACK:\n+\t\t\tret = flow_dv_validate_action_aso_ct(dev, action_flags,\n+\t\t\t\t\t\t\t item_flags, attr,\n+\t\t\t\t\t\t\t error);\n+\t\t\tif (ret < 0)\n+\t\t\t\treturn ret;\n+\t\t\taction_flags |= MLX5_FLOW_ACTION_CT;\n+\t\t\tbreak;\n \t\tdefault:\n \t\t\treturn rte_flow_error_set(error, ENOTSUP,\n \t\t\t\t\t\t RTE_FLOW_ERROR_TYPE_ACTION,\n@@ -13838,6 +13897,9 @@ __flow_dv_action_ct_update(struct rte_eth_dev *dev, uint32_t idx,\n \tif (update->direction)\n \t\tct->is_original = !!new_prf->is_original_dir;\n \tif (update->state) {\n+\t\tret = mlx5_validate_action_ct(dev, new_prf, error);\n+\t\tif (ret)\n+\t\t\treturn ret;\n \t\tret = mlx5_aso_ct_update_by_wqe(priv->sh, ct, new_prf);\n \t\t/* Block until ready or a failure. */\n \t\tif (!ret)\n@@ -15691,6 +15753,12 @@ flow_dv_action_validate(struct rte_eth_dev *dev,\n \t\t\t\t\t\tNULL,\n \t\t\t\t\t \"shared age action not supported\");\n \t\treturn flow_dv_validate_action_age(0, action, dev, err);\n+\tcase RTE_FLOW_ACTION_TYPE_CONNTRACK:\n+\t\tif (!priv->sh->ct_aso_en)\n+\t\t\treturn rte_flow_error_set(err, ENOTSUP,\n+\t\t\t\t\tRTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,\n+\t\t\t\t\t\"ASO CT is not supported\");\n+\t\treturn mlx5_validate_action_ct(dev, action->conf, err);\n \tdefault:\n \t\treturn rte_flow_error_set(err, ENOTSUP,\n \t\t\t\t\t RTE_FLOW_ERROR_TYPE_ACTION,\n", "prefixes": [ "14/17" ] }{ "id": 92279, "url": "