Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/117509/?format=api
{ "id": 117509, "url": "http://patches.dpdk.org/api/patches/117509/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1665109126-16201-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": "<1665109126-16201-5-git-send-email-chaoyong.he@corigine.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1665109126-16201-5-git-send-email-chaoyong.he@corigine.com", "date": "2022-10-07T02:18:26", "name": "[04/24] net/nfp: add the offload support of basic items", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "dea4936897562138e2683549f7b02c4717685f4e", "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/1665109126-16201-5-git-send-email-chaoyong.he@corigine.com/mbox/", "series": [ { "id": 25023, "url": "http://patches.dpdk.org/api/series/25023/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=25023", "date": "2022-10-07T02:18:22", "name": "add the basic rte_flow offload support of nfp PMD", "version": 1, "mbox": "http://patches.dpdk.org/series/25023/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/117509/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/117509/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 8BC17A0093;\n\tFri, 7 Oct 2022 04:19:42 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 402E442B72;\n\tFri, 7 Oct 2022 04:19:20 +0200 (CEST)", "from NAM11-CO1-obe.outbound.protection.outlook.com\n (mail-co1nam11on2124.outbound.protection.outlook.com [40.107.220.124])\n by mails.dpdk.org (Postfix) with ESMTP id 8A8A44280B\n for <dev@dpdk.org>; Fri, 7 Oct 2022 04:19:18 +0200 (CEST)", "from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5)\n by SA1PR13MB5465.namprd13.prod.outlook.com (2603:10b6:806:230::12)\n with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5709.10; Fri, 7 Oct\n 2022 02:19:16 +0000", "from SJ0PR13MB5545.namprd13.prod.outlook.com\n ([fe80::a192:2720:1ef8:2773]) by SJ0PR13MB5545.namprd13.prod.outlook.com\n ([fe80::a192:2720:1ef8:2773%4]) with mapi id 15.20.5723.010; Fri, 7 Oct 2022\n 02:19:16 +0000" ], "ARC-Seal": "i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=Gmio5WoCikUlJ8hhQoay+YFCT0TEK1ltGWA1h7MI3QsxPARelikYCfYaeL+GZIw70WyrrjBwhzRxmYca7fvnJ8WnPTP9BRfIiEHoAeA4MbKxF4IW+06hWKJ5ljzBZsVKN3Tp9+MQhUkBbbdsDdB6RM2lRw5PvtmLCsSafGQdNvCvwl34TRIt4p349MFf7LGwOVrxPMGA5joayW+8WrRKikZHNnLPavXS8wZsF0Jt0Zczphi9WBMJDyB0gQB7HkFy5Xb5IGE3iOuwZgV9xVOucIogHYy23BPJBfF2AHDWk4yeAsoOYe+DcuTuoz+4nyIwqyNbyDWZPWDrteWExNiNpA==", "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=soRSspkOxg1Agie1+bFSXWIfUoPVn9JGJ+VAdLrp370=;\n b=WGTOLMfom/oxCMDJ44IuznQAjpCO0ZUo80ZBQ/yVGrexe+42rOtGjXil2irnQiJkjc4b30xNhttpc7tsupsl45/EkZ+gFZB5D+YuAZtFDDFDSaMNGDQcekfZEmpr2Q0w5ni2Bt7ubujkQQDviPmghotLdzkROCycWMclS5NeaLMk/NfcHyxfXIiu7yNmEGTIyBtpA5ACDgN3amIw3sRqiBndPhF/fynFDoCp4grQ5GzgFckV3tVOQV+ls7pNloEs2HG3JdXc6ys9T9Ea7/5+bsiVDGeGHFBgpiRxLPtwte0WM9OUqujE1mPm33HKFq4dFRURWfWChUBgzKGNtqS78Q==", "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=soRSspkOxg1Agie1+bFSXWIfUoPVn9JGJ+VAdLrp370=;\n b=Lh3DQibidh908lD2nJ2KQX3cSuhV78zSTqj/a317tPdjV5fH3S1nTvX3QIALZMKVcmjKLJi/Vf21ZZ4JPALPfGbpedmcVVHWyo5XASf2u2kiPEvjjsObBqnWjNhtq4ZmVBcmGHSsV16lDCevxkmc4c4nBpi4BxTA4bFVSSO+9NY=", "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 04/24] net/nfp: add the offload support of basic items", "Date": "Fri, 7 Oct 2022 10:18:26 +0800", "Message-Id": "<1665109126-16201-5-git-send-email-chaoyong.he@corigine.com>", "X-Mailer": "git-send-email 1.8.3.1", "In-Reply-To": "<1665109126-16201-1-git-send-email-chaoyong.he@corigine.com>", "References": "<1665109126-16201-1-git-send-email-chaoyong.he@corigine.com>", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "X-ClientProxiedBy": "SG2P153CA0043.APCP153.PROD.OUTLOOK.COM (2603:1096:4:c6::12)\n 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_|SA1PR13MB5465:EE_", "X-MS-Office365-Filtering-Correlation-Id": "fb162d50-7b53-4405-8b83-08daa80a559c", "X-MS-Exchange-SenderADCheck": "1", "X-MS-Exchange-AntiSpam-Relay": "0", "X-Microsoft-Antispam": "BCL:0;", "X-Microsoft-Antispam-Message-Info": "\n RGFSLwu5DgQFwAZgUggbzX+Jg3r0uZA+RXZ2SvaJDgUkKUMFCm+UA0V/7xytsA44DpD8+eIR3DNtY7k2UkX6vRIGgAlppo4gqckMTLIJOraeXwSsGf2ez2keUDuLcdT+W/+DlY953S/P/hONT+FgevKYLPQBDwhsof7d0bqFe80oHjrt1Z+0BWrU4DZd++NcWJ7e74hjYuEBxIzSgectlwBwP/r+UzMBu2JJe3B3NdDlDMQW9neZqKcLmZy0JCjxu7fyDdgDeX4lLmfciqw/BUeOZ2e9uHgG7/XlY+Clg2oOEdhg3PkSnYqRSIs1tWznlIlfHHnjpW5bqXYeKnjx+Kx/WLexiId/u460WnwmoV79tVU1Cc9AnUc9I0eaG8++ztdkNxP2g5UXjd7P18Sr0Nf1+qEOBsDCSUdEmReXzVWx5glJ9HsZDTEPBGsERbIhE7vz9Jwip9TAXwQeURQCTI1CYPH3kkLWOOo0Kj0z4m6Vq9sHypQMqdFfSbTE6eQT4dOW8+EFSg+QW3TEzdkHphpgRt8in4+hIwBLQ1tyUaShSuVPzaV66cFB3tbbTaXi+GVynInOY7A3buXn2EzlSJUFn0gJmoFhekjXH/2g6gt7gncmtORr4LAbJdEhT5rSUspuNxmk7diHKH7QqBQI/1ex+jEWTztXtesdDo4W6jKiKsgd/xQveUTTxiHa1d6r/6hG7vdQ51u5LISFJgogMOJBg9krdUnfWDC+J6PJEfYV8OJ+tTbj5d5AXIW4reL2j0xKc0lhB6y0qOy627mUcg==", "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)(39830400003)(366004)(136003)(346002)(376002)(396003)(451199015)(36756003)(86362001)(66946007)(30864003)(44832011)(38350700002)(38100700002)(8936002)(6486002)(6916009)(66574015)(83380400001)(52116002)(478600001)(41300700001)(6512007)(4326008)(107886003)(26005)(2616005)(6506007)(316002)(6666004)(5660300002)(66476007)(66556008)(2906002)(186003)(8676002);\n DIR:OUT; SFP:1102;", "X-MS-Exchange-AntiSpam-MessageData-ChunkCount": "1", "X-MS-Exchange-AntiSpam-MessageData-0": "=?utf-8?q?Jw9fftzJw1qFS++XnthBcTPtRytQ?=\n\t=?utf-8?q?yEjMphDtvo0Nu/+8RK5OYaA1RwC3/tEWfGGzbuNjPXsGr1VhtimOaeVwGZSnvDRf5?=\n\t=?utf-8?q?DJiDQ6C3/AWhfWONGZHVdw4b9LrbeF8VjUvziGo680AeTEbHaa9AM51Lnn8LwxcEM?=\n\t=?utf-8?q?261zi6YSrmJv65qcK8gNza44J8hifs7Z4qRkk62ZSdhm5DcUG0/YvNUShGZSL1RmD?=\n\t=?utf-8?q?z1j8VgchMK/C7+EP7dL+/9fHudWNGI7ZYMzqYaFt4Gr4ciDUlGmHKoI86cnHNLJmi?=\n\t=?utf-8?q?7vVJ7wE3AVPCSvLICb6Tf+o145y0/rqbtjUvrVbfbVpSjTjmpXCAMNb7pBxNfvEX2?=\n\t=?utf-8?q?zDBm3Qvk/SaiSCv9hFswk9FSNmnJDYcwAWERQJq3Rs8eEagjUFFz+f7jg/LVzP2+P?=\n\t=?utf-8?q?a2HcQYKO1qn0kZ0VwsgrnGR0nG06yngo8Pga96bDsSTvNi47iyhok5WcupqXsOy55?=\n\t=?utf-8?q?+axkGmyYI0agCsxfziFYZFaJofrR8GGVs4tAjDGZF3e9UZOV+82wGC7dUpnzX+OlE?=\n\t=?utf-8?q?Bc5X96Z0+7jy4+V4YzgSVs0yzmpz4c99Ktnd+dTeQoJJfsKq0BLfBeANh+BGNN5b8?=\n\t=?utf-8?q?aBBeUB34RS/ZPWlsLjbHWBsiiOF+wN9BbOBrJo3lQsAD+X8I74xpDbVO9YkR2m0zO?=\n\t=?utf-8?q?5xlIZVZ3YyO3pdf8wP4k2DpU4FTMP7zYtYV8uMCP+P7iJlzH8Dhy+MIR6+pj9Q6V/?=\n\t=?utf-8?q?7uhei/uw8rtlY2b9Fe+mBaAAADa62M3ix026+Rr367zlBb5onM2ML6h2flCcAkq4i?=\n\t=?utf-8?q?sFiupnVi2NKFTOl0ka2ve+ZhTvJrVjKcaHKtupgcQ+YBU83eaI5ARZbYrGvYDpi+T?=\n\t=?utf-8?q?dJFFBBQgkfF4aIBMvOUpqwuguCkIkiQw02nw7cZFnqgk55PJHUVjWIb4xQEAyYFcU?=\n\t=?utf-8?q?TNqQSsfarHn5GX7WawBAWnQPj4pBi80a+zqSEE/2qyN4k4BxWET4lNyu8N4Xanvqx?=\n\t=?utf-8?q?Pi0Zv0kzDWvAIu6ucCS3DZtbkGjH3m08AGH4w0PMLQuBNaJUecIEp24HhY9BLYJdf?=\n\t=?utf-8?q?QcqxqCMuaPVWgO73NukkgDWPPm0olqd0+MOqs7ZumEbC3o1sMVGlFWNrVfGLQGw/q?=\n\t=?utf-8?q?apt+iL2pqm4Pqc3MfGN6zoR8Ne7Ae14RBf/xI25edrnR0/lx9ISv5hW6zlUJ5fMUh?=\n\t=?utf-8?q?DQUQ3HcuSpPjlVa0cq+FZGWwALNQwz2br9fiz0cXoXfdN93gV7xyCeSlh1P5b8FrW?=\n\t=?utf-8?q?UuVQg+q4xHCtBhYJJClJADkNsf65WsbBXbVhpIFpDjD6cCKeSLQ1lZz0yDnk9KNJO?=\n\t=?utf-8?q?oWo+ukfTVUZhvyikKY6v+j2xJzq96LnavLDdMIP0k7yihFaIrRlcYxRFtnHAlekGm?=\n\t=?utf-8?q?BR/qEC3QGG9KFCoyKu9kOMKxNoWG9lBquq6mb9pI7PFhSuWfcxSVp95ehaWr6UBvA?=\n\t=?utf-8?q?Pwh6sYOIrueF6mC6Plkv5XKT1IX/voG8yuQ28mZSiIbykSy7CCxbMQDxXs5dkqzqV?=\n\t=?utf-8?q?liPQheJsxpzkeMAwm46g0odw91qUNqVfmzm/J4F+ya/zp8w4dgpgSgalihi/2e9wc?=\n\t=?utf-8?q?6678dqB+I/SvmEdubTS5clktClawPTlHhw=3D=3D?=", "X-OriginatorOrg": "corigine.com", "X-MS-Exchange-CrossTenant-Network-Message-Id": "\n fb162d50-7b53-4405-8b83-08daa80a559c", "X-MS-Exchange-CrossTenant-AuthSource": "SJ0PR13MB5545.namprd13.prod.outlook.com", "X-MS-Exchange-CrossTenant-AuthAs": "Internal", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "07 Oct 2022 02:19:16.8151 (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 8/G/7NJwqwNfEwuRRVAwmKlilHVDdC9BryEBzVnTwejWFvDujCNKTZjRmuvuU9ayBXrlwP1NsPkuhPVzgQp9Z6iqsm9JaKVgHeoTdXfYec4=", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "SA1PR13MB5465", "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 6794a9c..17ac784 100644\n--- a/doc/guides/rel_notes/release_22_11.rst\n+++ b/doc/guides/rel_notes/release_22_11.rst\n@@ -105,6 +105,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": [ "04/24" ] }