Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/117766/?format=api
{ "id": 117766, "url": "http://patches.dpdk.org/api/patches/117766/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1665382142-21684-5-git-send-email-chaoyong.he@corigine.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": "<1665382142-21684-5-git-send-email-chaoyong.he@corigine.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1665382142-21684-5-git-send-email-chaoyong.he@corigine.com", "date": "2022-10-10T06:08:42", "name": "[v2,04/24] net/nfp: add the offload support of basic items", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "31a0670d5b588568694f79323a126196fa7f9cfd", "submitter": { "id": 2554, "url": "http://patches.dpdk.org/api/people/2554/?format=api", "name": "Chaoyong He", "email": "chaoyong.he@corigine.com" }, "delegate": { "id": 319, "url": "http://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/1665382142-21684-5-git-send-email-chaoyong.he@corigine.com/mbox/", "series": [ { "id": 25072, "url": "http://patches.dpdk.org/api/series/25072/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=25072", "date": "2022-10-10T06:08:38", "name": "add the basic rte_flow offload support of nfp PMD", "version": 2, "mbox": "http://patches.dpdk.org/series/25072/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/117766/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/117766/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 A603EA0544;\n\tMon, 10 Oct 2022 08:09:51 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 3244242B6C;\n\tMon, 10 Oct 2022 08:09:32 +0200 (CEST)", "from NAM04-BN8-obe.outbound.protection.outlook.com\n (mail-bn8nam04on2127.outbound.protection.outlook.com [40.107.100.127])\n by mails.dpdk.org (Postfix) with ESMTP id BF73D427F6\n for <dev@dpdk.org>; Mon, 10 Oct 2022 08:09:28 +0200 (CEST)", "from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5)\n by SN4PR13MB5296.namprd13.prod.outlook.com (2603:10b6:806:207::14)\n with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5723.14; Mon, 10 Oct\n 2022 06:09:27 +0000", "from SJ0PR13MB5545.namprd13.prod.outlook.com\n ([fe80::7c26:1a0b:2825:6f4b]) by SJ0PR13MB5545.namprd13.prod.outlook.com\n ([fe80::7c26:1a0b:2825:6f4b%4]) with mapi id 15.20.5723.014; Mon, 10 Oct 2022\n 06:09:27 +0000" ], "ARC-Seal": "i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=SCaNXpmP3cI4S3eD5PYinnAHsBHVeQ+KSffoEzX5yQv7B52IjWIWR1uuZCjvsbhgv60uWvmAbH49y4vFrK1TcdsPna4CGA9s+5Vf6F+2XpFKAy/2t5TeCgdFk/pyWxqhWNwHkBWxpiP5BOO3OC0num0y847kiwmisywJ9FIz5PhuKwYnQZq1yOtVoMIkSNQS0lngme0elPGueOqLnUQTQNjqY8uIm+B5siNneFGckz/oN+dkkIXjg/SNXKsqTD0UOvkK9qzwjsjsbcnYJgSvCOYwnPwPORjdb9EZlC1E4aagNTwC08fRhNsnvFZUHt3zXBr5YPWi7zi3H0tH2feb/Q==", "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=n/typgngfDiz+5Q5xdEyVbgZOwP5sbJT2BbpQnJU0dA=;\n b=flQuFUF23vms/j+OL4nD8MuzGSueymk8EJSH5bLOuIAZighyx9NB9NGud6i3lRBdFPoFcKjgN/W44IEDzupURg3LByymkXS7iDDJG11+akqMUcmGay0zc7xxng6OQNZ/bYNpvE1Ae6feEMC+3h6lJRXZTOIEYMWMX4z87lF55Y4/fR91V260rR2t+RMo8Fm3ljcT/GQrVWPbwnfct1SiZxrksUDW1+k/wdvaDCsQrpQXMxeBGhWQn5hrm0Wpx19UYEQv5mcpRVGCTOKRAUVLXaScA4WFcECgpvv0K8qptXdUWtIViWYLQyqE5DHQxlla98QDhbvF4DNtxGKHdASl+A==", "ARC-Authentication-Results": "i=1; mx.microsoft.com 1; spf=pass\n smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com;\n dkim=pass header.d=corigine.com; arc=none", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=n/typgngfDiz+5Q5xdEyVbgZOwP5sbJT2BbpQnJU0dA=;\n b=A4FHvd1Mfl29Ep7qsFtCBAntOIMBkMHKGwd7bb8DVzFTN4+wYOmh/6qx5lXsVnm8FqbzPTi2AXLIjkCAnFEqfdQSN2oQFJqskNLa6PARI1VH4P9zFrCwqviVBn4p+7UpujbXfBolSMmz3beUIZfh3IMvEQiMnTgDZkne916duR8=", "Authentication-Results": "dkim=none (message not signed)\n header.d=none;dmarc=none action=none header.from=corigine.com;", "From": "Chaoyong He <chaoyong.he@corigine.com>", "To": "dev@dpdk.org", "Cc": "oss-drivers@corigine.com, niklas.soderlund@corigine.com,\n Chaoyong He <chaoyong.he@corigine.com>", "Subject": "[PATCH v2 04/24] net/nfp: add the offload support of basic items", "Date": "Mon, 10 Oct 2022 14:08:42 +0800", "Message-Id": "<1665382142-21684-5-git-send-email-chaoyong.he@corigine.com>", "X-Mailer": "git-send-email 1.8.3.1", "In-Reply-To": "<1665382142-21684-1-git-send-email-chaoyong.he@corigine.com>", "References": "<1665382142-21684-1-git-send-email-chaoyong.he@corigine.com>", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "X-ClientProxiedBy": "SI2PR01CA0051.apcprd01.prod.exchangelabs.com\n (2603:1096:4:193::6) To SJ0PR13MB5545.namprd13.prod.outlook.com\n (2603:10b6:a03:424::5)", "MIME-Version": "1.0", "X-MS-PublicTrafficType": "Email", "X-MS-TrafficTypeDiagnostic": "SJ0PR13MB5545:EE_|SN4PR13MB5296:EE_", "X-MS-Office365-Filtering-Correlation-Id": "a452ec35-e356-471c-1694-08daaa85fcba", "X-MS-Exchange-SenderADCheck": "1", "X-MS-Exchange-AntiSpam-Relay": "0", "X-Microsoft-Antispam": "BCL:0;", "X-Microsoft-Antispam-Message-Info": "\n KZlCdUXyVBjewlLugMT71Ai/VY+jXnvAFTipP3InSMx5bWrA131gnddqjVpIhc9WwGCn5nh/3XGc/pUzqYKnD1cRlagiPDXwueIDVlnVCtSflmPc0zDjsr0qF/HMuI/ibmAk2SzvO/qsOC7Lxc/oB0TfroZ50ydl3+mMgbwn4tR5X3b4+ot2ggeNAunz/4G16ZJqYQ8lJ/0E/Iy4MIKSUmdNAd4W8JdoKmYmUWrt91Efq035DZabDX0CkkxqYWktd3pl25Oel3XYb8cqiWJ2tVqu1qJNckvduVZ/XNU67YwozH3RrFSVdp4z0bti68A0iKGCAlkctIStJm1bNDXnxPvcnmnQJzdCbIWzKBR/ZWFlVTca29vJGjlakqTJkzUllGQih0EUtk9RrMuerXf82gO0bGvw2kJ9VbDVLeoGZetMb9lLjsjPmpe9XzNlXdW21/SmCXmGT2kh0KdMzC6yoEUSVVB4CdrxJOyaCx/5A8MEqrdqGWKbo+vKpjJ04+BM1M1Vn1DIvqKZtBNucJXgAE4FghQlOIgWt3Noq2JcRUaKB8cQzmFhdyK0peS1sqV54PkFcAmzCL29Yh7kmr3xiqXotSLKL1oN1Yut8JK7GT4L4A2rmjfamA324PYynlbk7Ho3lBXUwp3et8PpzdYgwKw3oy0TtwmyUPtD8nkzDc+1YqJTH8+9R3/C4qR3NXa4kFH/WPbH1gU7bEuCRZ5qkxD6CROqI84v6RfaT06WFM8LPUBZcTlYjILpqOcEjcCFx2JS2VtaoOegQ5iT4VdgSbTxxuvdnw6Gz3CRjjcrJbVk+WYXLfjOVTblKMpLgoyD", "X-Forefront-Antispam-Report": "CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:;\n IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE;\n SFS:(13230022)(4636009)(366004)(396003)(346002)(376002)(136003)(39830400003)(451199015)(6916009)(316002)(2906002)(186003)(6486002)(2616005)(66476007)(6666004)(107886003)(66556008)(41300700001)(66946007)(83380400001)(30864003)(8936002)(8676002)(4326008)(5660300002)(66574015)(44832011)(26005)(6506007)(6512007)(52116002)(86362001)(478600001)(36756003)(38350700002)(38100700002);\n DIR:OUT; SFP:1102;", "X-MS-Exchange-AntiSpam-MessageData-ChunkCount": "1", "X-MS-Exchange-AntiSpam-MessageData-0": "=?utf-8?q?90xf3qd6B2pMbn3nKj/La703sQ6B?=\n\t=?utf-8?q?D3kxBerRaLiOc3n1SZFC6SZNfzojNLv96k60UQkDJKdh1WclyIkftAT3eQtJF8wS6?=\n\t=?utf-8?q?+BkA3DJJpO4TgfZeMFyMWKRd4GUpM3NhJ1TdrNhQfHMK7yTuRwvof6GZoB/DErohE?=\n\t=?utf-8?q?QpJF9dWQHEHqNcg8yDdmVscetkcP9il6cCQik1OhSuKvzutrWFuycJ0LldIo/PgMx?=\n\t=?utf-8?q?2m9TUgUXX/zIGy2io0BiIdKkQ9D16TSXHtaQgg8UGllIeW8u5JbLIRO8ErZV+vwAk?=\n\t=?utf-8?q?M7Ci5H2zHLADxBFXkwwYiptcYkKVYTXvSf/XQMgf9oWgwR409KR43naFF+nLNBMwM?=\n\t=?utf-8?q?4l+MTY1b1GqyIVugyx75vnNeDsx7D7ibJ9JdUmMnrDpVTDzsVsU1rRm7CXxvRbjcD?=\n\t=?utf-8?q?ppua+SUhvmepGz0jBaiHLJUdIq5MMwvwnxyNCupOy+YfvZAbltQao1yeE0LEaMX3v?=\n\t=?utf-8?q?RziRQAtxUYGEYjNH4ZNvWfrhaAHW8Op/dggkEmHlK9ypojuL1/kps84wF0oG7Yd1c?=\n\t=?utf-8?q?OtcxP4n03ozE/hgnXs6EaT6KNHsN9GANfg7ir4BI98WnFfxxU5oDf54tWvCzjuLh2?=\n\t=?utf-8?q?m6QIWoOPiBH5me89sQAQxnNdKk8t4xaZ41ijagn39FwVSv3ekd6CgJsqJfHEqoUWb?=\n\t=?utf-8?q?uv3cZQXzNnZD2/f3qKiZ4p1HhlrYfVnpQT0Q+xZfa4WY9Sktb3pDBLsyukMJS2xx3?=\n\t=?utf-8?q?zCYE+/W8qs1Zq/QSSCXkIA2NWzdjzOs/3I9CwVoT4zQeUcTElS9KwZkbOIaraCkOb?=\n\t=?utf-8?q?D9de5FrxuMI7GiOVTv+P8sEBVA1oo7GS39QZ4/D8++FcikpVJ5xAaOy66LeZoYd/u?=\n\t=?utf-8?q?xWmTEe7QUPHZEawN8doD0uVapKiMmPI5Lfm6coPBOzYGKAVObFaclyAtB3cwh83bD?=\n\t=?utf-8?q?7rW3XNsiFOCL9pYcxbbKFx71D/XnqkWjFigrDrjLwOv1JzQtcpFs3vXQlNKOp2dLX?=\n\t=?utf-8?q?uFUcuRjqCyaEa/gOYEmjBU9Q19lN5G8x1lH20lIFnY7jkFsU92yvuOYShHC69eSPQ?=\n\t=?utf-8?q?vSU4bwoqMc68tp1+61zgsugvxmsJHCOskVMd6nvp2xaKhP2lNUSCjBnzyHTqpLwDE?=\n\t=?utf-8?q?pp27PUlczEd3oQrXrt0nWiQYP8bKa4/Eh5D8/4dddPTODj3IBt2FDHGB493Op0pp1?=\n\t=?utf-8?q?EEZo1iPVhZ/3J26kB/j30a01iKGjVuMdaYUpmtXdTZwwOoTt1x1s10drV1s8MrW4r?=\n\t=?utf-8?q?YUInITmdRQmFoI8cEsZ2IAO01GX0rQ4l4sVAb/XAE2rYEnhoqt5yXzmh1YY9Eu07S?=\n\t=?utf-8?q?imXMyLH/YONs+xs0qCDeVKmc3mVSH+KqxgMA/f3DRnDlBTVJ0fCkLsDWiVbaYcawN?=\n\t=?utf-8?q?x8X0l+1wkeEHBpOKHFuBCZNKpOhUEERN6LZCzgDDXF/s5ULWHSfMUT37npYiQDAoL?=\n\t=?utf-8?q?icgSWaA5KLLrejFSfOSXyTxOrKSHM1S+kDjNj0woIFcNIWyh6dvxdXTsZ+08VnelJ?=\n\t=?utf-8?q?zDA9XobMPCCSOibok4nu4kavHKf7ubLw8nH7GqXKuQOl+Qq3Epe0TyyyVnM/yFOgY?=\n\t=?utf-8?q?KBVFDhJetOEEeKoLhKIxA/FF+JR5h9bI6A=3D=3D?=", "X-OriginatorOrg": "corigine.com", "X-MS-Exchange-CrossTenant-Network-Message-Id": "\n a452ec35-e356-471c-1694-08daaa85fcba", "X-MS-Exchange-CrossTenant-AuthSource": "SJ0PR13MB5545.namprd13.prod.outlook.com", "X-MS-Exchange-CrossTenant-AuthAs": "Internal", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "10 Oct 2022 06:09:27.5021 (UTC)", "X-MS-Exchange-CrossTenant-FromEntityHeader": "Hosted", "X-MS-Exchange-CrossTenant-Id": "fe128f2c-073b-4c20-818e-7246a585940c", "X-MS-Exchange-CrossTenant-MailboxType": "HOSTED", "X-MS-Exchange-CrossTenant-UserPrincipalName": "\n 62wyA6A5Lx0/ySUK3pnQYnmJaWVpkZ3bKHlEMK+y8bjUQB7IbkcRHOWLsFY9dM4PmwqK35whRKTEZZaijDgO6ckF1rQURd7i2by+mhYE0Ow=", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "SN4PR13MB5296", "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": "Add the offload support of very basic items: ethernet and\nport id.\n\nSigned-off-by: Chaoyong He <chaoyong.he@corigine.com>\nReviewed-by: Niklas Söderlund <niklas.soderlund@corigine.com>\n---\n doc/guides/nics/features/nfp.ini | 4 +\n doc/guides/rel_notes/release_22_11.rst | 4 +\n drivers/net/nfp/flower/nfp_flower_cmsg.h | 20 +++\n drivers/net/nfp/nfp_flow.c | 252 ++++++++++++++++++++++++++++++-\n 4 files changed, 279 insertions(+), 1 deletion(-)", "diff": "diff --git a/doc/guides/nics/features/nfp.ini b/doc/guides/nics/features/nfp.ini\nindex f7a0362..4460cf0 100644\n--- a/doc/guides/nics/features/nfp.ini\n+++ b/doc/guides/nics/features/nfp.ini\n@@ -25,3 +25,7 @@ Linux = Y\n Multiprocess aware = Y\n x86-64 = Y\n Usage doc = Y\n+\n+[rte_flow items]\n+eth = Y\n+port_id = Y\ndiff --git a/doc/guides/rel_notes/release_22_11.rst b/doc/guides/rel_notes/release_22_11.rst\nindex fbb5752..cd22d95 100644\n--- a/doc/guides/rel_notes/release_22_11.rst\n+++ b/doc/guides/rel_notes/release_22_11.rst\n@@ -151,6 +151,10 @@ New Features\n * Added the control message interactive channels between PMD and firmware.\n * Added the support of representor port.\n \n+ Add the support of rte_flow pattern items as follow:\n+\n+ * Ethernet\n+\n * **Updated NXP dpaa2 driver.**\n \n * Added support for flow action REPRESENTED_PORT.\ndiff --git a/drivers/net/nfp/flower/nfp_flower_cmsg.h b/drivers/net/nfp/flower/nfp_flower_cmsg.h\nindex 6045bb0..75a3b91 100644\n--- a/drivers/net/nfp/flower/nfp_flower_cmsg.h\n+++ b/drivers/net/nfp/flower/nfp_flower_cmsg.h\n@@ -233,6 +233,26 @@ struct nfp_flower_in_port {\n \trte_be32_t in_port;\n };\n \n+/*\n+ * L2 details (4W/16B)\n+ * 3 2 1\n+ * 1 0 9 8 7 6 5 4 3 2 1 0 9 8 7 6 5 4 3 2 1 0 9 8 7 6 5 4 3 2 1 0\n+ * +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+\n+ * | mac_addr_dst, 31 - 0 |\n+ * +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+\n+ * | mac_addr_dst, 47 - 32 | mac_addr_src, 15 - 0 |\n+ * +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+\n+ * | mac_addr_src, 47 - 16 |\n+ * +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+\n+ * | mpls outermost label | TC |B| reserved |q|\n+ * +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+\n+ */\n+struct nfp_flower_mac_mpls {\n+\tuint8_t mac_dst[6];\n+\tuint8_t mac_src[6];\n+\trte_be32_t mpls_lse;\n+};\n+\n int nfp_flower_cmsg_mac_repr(struct nfp_app_fw_flower *app_fw_flower);\n int nfp_flower_cmsg_repr_reify(struct nfp_app_fw_flower *app_fw_flower,\n \t\tstruct nfp_flower_representor *repr);\ndiff --git a/drivers/net/nfp/nfp_flow.c b/drivers/net/nfp/nfp_flow.c\nindex 114686f..a077423 100644\n--- a/drivers/net/nfp/nfp_flow.c\n+++ b/drivers/net/nfp/nfp_flow.c\n@@ -19,6 +19,30 @@\n #include \"nfpcore/nfp_mip.h\"\n #include \"nfpcore/nfp_rtsym.h\"\n \n+/* Static initializer for a list of subsequent item types */\n+#define NEXT_ITEM(...) \\\n+\t((const enum rte_flow_item_type []){ \\\n+\t\t__VA_ARGS__, RTE_FLOW_ITEM_TYPE_END, \\\n+\t})\n+\n+/* Process structure associated with a flow item */\n+struct nfp_flow_item_proc {\n+\t/* Bit-mask for fields supported by this PMD. */\n+\tconst void *mask_support;\n+\t/* Bit-mask to use when @p item->mask is not provided. */\n+\tconst void *mask_default;\n+\t/* Size in bytes for @p mask_support and @p mask_default. */\n+\tconst unsigned int mask_sz;\n+\t/* Merge a pattern item into a flow rule handle. */\n+\tint (*merge)(struct rte_flow *nfp_flow,\n+\t\t\tchar **mbuf_off,\n+\t\t\tconst struct rte_flow_item *item,\n+\t\t\tconst struct nfp_flow_item_proc *proc,\n+\t\t\tbool is_mask);\n+\t/* List of possible subsequent items. */\n+\tconst enum rte_flow_item_type *const next_item;\n+};\n+\n struct nfp_mask_id_entry {\n \tuint32_t hash_key;\n \tuint32_t ref_cnt;\n@@ -476,12 +500,36 @@ struct nfp_mask_id_entry {\n \n static int\n nfp_flow_key_layers_calculate_items(const struct rte_flow_item items[],\n-\t\t__rte_unused struct nfp_fl_key_ls *key_ls)\n+\t\tstruct nfp_fl_key_ls *key_ls)\n {\n+\tstruct rte_eth_dev *ethdev;\n \tconst struct rte_flow_item *item;\n+\tstruct nfp_flower_representor *representor;\n+\tconst struct rte_flow_item_port_id *port_id;\n \n \tfor (item = items; item->type != RTE_FLOW_ITEM_TYPE_END; ++item) {\n \t\tswitch (item->type) {\n+\t\tcase RTE_FLOW_ITEM_TYPE_ETH:\n+\t\t\tPMD_DRV_LOG(DEBUG, \"RTE_FLOW_ITEM_TYPE_ETH detected\");\n+\t\t\t/*\n+\t\t\t * eth is set with no specific params.\n+\t\t\t * NFP does not need this.\n+\t\t\t */\n+\t\t\tif (item->spec == NULL)\n+\t\t\t\tcontinue;\n+\t\t\tkey_ls->key_layer |= NFP_FLOWER_LAYER_MAC;\n+\t\t\tkey_ls->key_size += sizeof(struct nfp_flower_mac_mpls);\n+\t\t\tbreak;\n+\t\tcase RTE_FLOW_ITEM_TYPE_PORT_ID:\n+\t\t\tPMD_DRV_LOG(DEBUG, \"RTE_FLOW_ITEM_TYPE_PORT_ID detected\");\n+\t\t\tport_id = item->spec;\n+\t\t\tif (port_id->id >= RTE_MAX_ETHPORTS)\n+\t\t\t\treturn -ERANGE;\n+\t\t\tethdev = &rte_eth_devices[port_id->id];\n+\t\t\trepresentor = (struct nfp_flower_representor *)\n+\t\t\t\t\tethdev->data->dev_private;\n+\t\t\tkey_ls->port = rte_cpu_to_be_32(representor->port_id);\n+\t\t\tbreak;\n \t\tdefault:\n \t\t\tPMD_DRV_LOG(ERR, \"Item type %d not supported.\", item->type);\n \t\t\treturn -ENOTSUP;\n@@ -541,6 +589,202 @@ struct nfp_mask_id_entry {\n \treturn ret;\n }\n \n+static int\n+nfp_flow_merge_eth(__rte_unused struct rte_flow *nfp_flow,\n+\t\tchar **mbuf_off,\n+\t\tconst struct rte_flow_item *item,\n+\t\tconst struct nfp_flow_item_proc *proc,\n+\t\tbool is_mask)\n+{\n+\tstruct nfp_flower_mac_mpls *eth;\n+\tconst struct rte_flow_item_eth *spec;\n+\tconst struct rte_flow_item_eth *mask;\n+\n+\tspec = item->spec;\n+\tif (spec == NULL) {\n+\t\tPMD_DRV_LOG(DEBUG, \"nfp flow merge eth: no item->spec!\");\n+\t\tgoto eth_end;\n+\t}\n+\n+\tmask = item->mask ? item->mask : proc->mask_default;\n+\teth = (void *)*mbuf_off;\n+\n+\tif (is_mask) {\n+\t\tmemcpy(eth->mac_src, mask->src.addr_bytes, RTE_ETHER_ADDR_LEN);\n+\t\tmemcpy(eth->mac_dst, mask->dst.addr_bytes, RTE_ETHER_ADDR_LEN);\n+\t} else {\n+\t\tmemcpy(eth->mac_src, spec->src.addr_bytes, RTE_ETHER_ADDR_LEN);\n+\t\tmemcpy(eth->mac_dst, spec->dst.addr_bytes, RTE_ETHER_ADDR_LEN);\n+\t}\n+\n+\teth->mpls_lse = 0;\n+\n+eth_end:\n+\t*mbuf_off += sizeof(struct nfp_flower_mac_mpls);\n+\n+\treturn 0;\n+}\n+\n+/* Graph of supported items and associated process function */\n+static const struct nfp_flow_item_proc nfp_flow_item_proc_list[] = {\n+\t[RTE_FLOW_ITEM_TYPE_END] = {\n+\t\t.next_item = NEXT_ITEM(RTE_FLOW_ITEM_TYPE_ETH),\n+\t},\n+\t[RTE_FLOW_ITEM_TYPE_ETH] = {\n+\t\t.mask_support = &(const struct rte_flow_item_eth){\n+\t\t\t.hdr = {\n+\t\t\t\t.dst_addr.addr_bytes = \"\\xff\\xff\\xff\\xff\\xff\\xff\",\n+\t\t\t\t.src_addr.addr_bytes = \"\\xff\\xff\\xff\\xff\\xff\\xff\",\n+\t\t\t\t.ether_type = RTE_BE16(0xffff),\n+\t\t\t},\n+\t\t\t.has_vlan = 1,\n+\t\t},\n+\t\t.mask_default = &rte_flow_item_eth_mask,\n+\t\t.mask_sz = sizeof(struct rte_flow_item_eth),\n+\t\t.merge = nfp_flow_merge_eth,\n+\t},\n+};\n+\n+static int\n+nfp_flow_item_check(const struct rte_flow_item *item,\n+\t\tconst struct nfp_flow_item_proc *proc)\n+{\n+\tint ret = 0;\n+\tunsigned int i;\n+\tconst uint8_t *mask;\n+\n+\t/* item->last and item->mask cannot exist without item->spec. */\n+\tif (item->spec == NULL) {\n+\t\tif (item->mask || item->last) {\n+\t\t\tPMD_DRV_LOG(ERR, \"'mask' or 'last' field provided\"\n+\t\t\t\t\t\" without a corresponding 'spec'.\");\n+\t\t\treturn -EINVAL;\n+\t\t}\n+\t\t/* No spec, no mask, no problem. */\n+\t\treturn 0;\n+\t}\n+\n+\tmask = item->mask ?\n+\t\t(const uint8_t *)item->mask :\n+\t\t(const uint8_t *)proc->mask_default;\n+\n+\t/*\n+\t * Single-pass check to make sure that:\n+\t * - Mask is supported, no bits are set outside proc->mask_support.\n+\t * - Both item->spec and item->last are included in mask.\n+\t */\n+\tfor (i = 0; i != proc->mask_sz; ++i) {\n+\t\tif (mask[i] == 0)\n+\t\t\tcontinue;\n+\n+\t\tif ((mask[i] | ((const uint8_t *)proc->mask_support)[i]) !=\n+\t\t\t\t((const uint8_t *)proc->mask_support)[i]) {\n+\t\t\tPMD_DRV_LOG(ERR, \"Unsupported field found in 'mask'.\");\n+\t\t\tret = -EINVAL;\n+\t\t\tbreak;\n+\t\t}\n+\n+\t\tif (item->last && (((const uint8_t *)item->spec)[i] & mask[i]) !=\n+\t\t\t\t(((const uint8_t *)item->last)[i] & mask[i])) {\n+\t\t\tPMD_DRV_LOG(ERR, \"Range between 'spec' and 'last'\"\n+\t\t\t\t\t\" is larger than 'mask'.\");\n+\t\t\tret = -ERANGE;\n+\t\t\tbreak;\n+\t\t}\n+\t}\n+\n+\treturn ret;\n+}\n+\n+static int\n+nfp_flow_compile_item_proc(const struct rte_flow_item items[],\n+\t\tstruct rte_flow *nfp_flow,\n+\t\tchar **mbuf_off_exact,\n+\t\tchar **mbuf_off_mask)\n+{\n+\tint i;\n+\tint ret = 0;\n+\tconst struct rte_flow_item *item;\n+\tconst struct nfp_flow_item_proc *proc_list;\n+\n+\tproc_list = nfp_flow_item_proc_list;\n+\tfor (item = items; item->type != RTE_FLOW_ITEM_TYPE_END; ++item) {\n+\t\tconst struct nfp_flow_item_proc *proc = NULL;\n+\n+\t\tfor (i = 0; proc_list->next_item && proc_list->next_item[i]; ++i) {\n+\t\t\tif (proc_list->next_item[i] == item->type) {\n+\t\t\t\tproc = &nfp_flow_item_proc_list[item->type];\n+\t\t\t\tbreak;\n+\t\t\t}\n+\t\t}\n+\n+\t\tif (proc == NULL) {\n+\t\t\tPMD_DRV_LOG(ERR, \"No next item provided for %d\", item->type);\n+\t\t\tret = -ENOTSUP;\n+\t\t\tbreak;\n+\t\t}\n+\n+\t\t/* Perform basic sanity checks */\n+\t\tret = nfp_flow_item_check(item, proc);\n+\t\tif (ret != 0) {\n+\t\t\tPMD_DRV_LOG(ERR, \"nfp flow item %d check failed\", item->type);\n+\t\t\tret = -EINVAL;\n+\t\t\tbreak;\n+\t\t}\n+\n+\t\tif (proc->merge == NULL) {\n+\t\t\tPMD_DRV_LOG(ERR, \"nfp flow item %d no proc function\", item->type);\n+\t\t\tret = -ENOTSUP;\n+\t\t\tbreak;\n+\t\t}\n+\n+\t\tret = proc->merge(nfp_flow, mbuf_off_exact, item,\n+\t\t\t\tproc, false);\n+\t\tif (ret != 0) {\n+\t\t\tPMD_DRV_LOG(ERR, \"nfp flow item %d exact merge failed\", item->type);\n+\t\t\tbreak;\n+\t\t}\n+\n+\t\tret = proc->merge(nfp_flow, mbuf_off_mask, item,\n+\t\t\t\tproc, true);\n+\t\tif (ret != 0) {\n+\t\t\tPMD_DRV_LOG(ERR, \"nfp flow item %d mask merge failed\", item->type);\n+\t\t\tbreak;\n+\t\t}\n+\n+\t\tproc_list = proc;\n+\t}\n+\n+\treturn ret;\n+}\n+\n+static int\n+nfp_flow_compile_items(__rte_unused struct nfp_flower_representor *representor,\n+\t\tconst struct rte_flow_item items[],\n+\t\tstruct rte_flow *nfp_flow)\n+{\n+\tint ret;\n+\tchar *mbuf_off_mask;\n+\tchar *mbuf_off_exact;\n+\n+\tmbuf_off_exact = nfp_flow->payload.unmasked_data +\n+\t\t\tsizeof(struct nfp_flower_meta_tci) +\n+\t\t\tsizeof(struct nfp_flower_in_port);\n+\tmbuf_off_mask = nfp_flow->payload.mask_data +\n+\t\t\tsizeof(struct nfp_flower_meta_tci) +\n+\t\t\tsizeof(struct nfp_flower_in_port);\n+\n+\t/* Go over items */\n+\tret = nfp_flow_compile_item_proc(items, nfp_flow,\n+\t\t\t&mbuf_off_exact, &mbuf_off_mask);\n+\tif (ret != 0) {\n+\t\tPMD_DRV_LOG(ERR, \"nfp flow item compile failed.\");\n+\t\treturn -EINVAL;\n+\t}\n+\n+\treturn 0;\n+}\n+\n static struct rte_flow *\n nfp_flow_process(struct nfp_flower_representor *representor,\n \t\tconst struct rte_flow_item items[],\n@@ -583,6 +827,12 @@ struct nfp_mask_id_entry {\n \n \tnfp_flow_compile_metadata(priv, nfp_flow, &key_layer, stats_ctx);\n \n+\tret = nfp_flow_compile_items(representor, items, nfp_flow);\n+\tif (ret != 0) {\n+\t\tPMD_DRV_LOG(ERR, \"nfp flow item process failed.\");\n+\t\tgoto free_flow;\n+\t}\n+\n \tnfp_flow_meta = nfp_flow->payload.meta;\n \tmask_data = nfp_flow->payload.mask_data;\n \tmask_len = key_layer.key_size;\n", "prefixes": [ "v2", "04/24" ] }