From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR02-HE1-obe.outbound.protection.outlook.com (mail-eopbgr10043.outbound.protection.outlook.com [40.107.1.43]) by dpdk.org (Postfix) with ESMTP id CE5D41B125 for ; Tue, 25 Sep 2018 01:17:45 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=rqHYjmmojJ+5/nIxxfo7W19coWJKDlsyP1Rd3GVn6MA=; b=nGE+ihV4KSWvMBlb54wPGVsDJb/ShJ4AVxresi5Arp9ilhjRcMOWJYxRMMLQ9kbnP7oCtYQgh+iHD7kuPFKay2LC0+XulOKjst0L0gmmRAc+o3xRJDy8EO0MGjgaI8u0bHGPsFuN6M5aZA4pV04dc5DeTBj3YzSbiIFUq0HtWVQ= Received: from DB3PR0502MB3980.eurprd05.prod.outlook.com (52.134.72.27) by DB3PR0502MB3996.eurprd05.prod.outlook.com (52.134.65.142) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1164.25; Mon, 24 Sep 2018 23:17:42 +0000 Received: from DB3PR0502MB3980.eurprd05.prod.outlook.com ([fe80::1cb0:661b:ecab:6045]) by DB3PR0502MB3980.eurprd05.prod.outlook.com ([fe80::1cb0:661b:ecab:6045%2]) with mapi id 15.20.1164.017; Mon, 24 Sep 2018 23:17:42 +0000 From: Yongseok Koh To: Thomas Monjalon , Shahaf Shuler CC: "dev@dpdk.org" , Ori Kam Thread-Topic: [PATCH v3 04/11] net/mlx5: add support for multiple flow drivers Thread-Index: AQHUVFzKJgiuVuFmXUq6v/YvQGZ9CA== Date: Mon, 24 Sep 2018 23:17:42 +0000 Message-ID: <20180924231721.15799-5-yskoh@mellanox.com> References: <20180919072143.23211-1-yskoh@mellanox.com> <20180924231721.15799-1-yskoh@mellanox.com> In-Reply-To: <20180924231721.15799-1-yskoh@mellanox.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: BN6PR03CA0021.namprd03.prod.outlook.com (2603:10b6:404:23::31) To DB3PR0502MB3980.eurprd05.prod.outlook.com (2603:10a6:8:10::27) authentication-results: spf=none (sender IP is ) smtp.mailfrom=yskoh@mellanox.com; x-ms-exchange-messagesentrepresentingtype: 1 x-originating-ip: [209.116.155.178] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; DB3PR0502MB3996; 6:wYhM/RvXvRIBEELjZRRLAx+aGaroqh+n1gC4FucbAWLWMeHgEAiDauVgFnOmNzZHIqSLeYFKb50RCxwnlURozzX3eXInSns4VVXQGIjC6MkVgftbGbsoL5XTBxZ0cVSKRHmo39EbbPILjqsOz6wvXHgVo85Ds9EztKQki7WQAkZov/mGqxENwvSh3j6yBAN1JTQofAKsAQZYuo8ZiwmlIHzXpNaR75Zf+lKuzWWG28wuKyMJ4ihk8NatGBvkWriOxDZfkvgMNoLk3ENb7jidbLt3HNsLma/7SYJgkldf+2bv8mHPkmZyRABxEjUW56zlHF2QYuYGNV5byoH6gP9RXvUxRssAyU/dyP9aAUsmd884FZ9X8t7dZtfsjWqDJm+OZzytNOjWO6o1flXntZtiC7ky0tdfAya1QPf3yIfLLM81b1bzCiTbBqo2QanVUvN58BSbSVeyn1MrFsT1ySnLlQ==; 5:s3NymtKLVR145TnnE6PuOt0jM+1l4lJ4Xqz7S3dVDppXddZ8fTzhVxe29Z+ByJ0soZyscV+Sox7Ahwo7sd+Cun5DN8RMsCl+Rgg4D6D2bk1gdYqyFj8UUBZDnJzUcrotBqXdQL4xGLMFUfDJJnqLEI99Jen5CFgDTrMozjjMrfQ=; 7:1A0LQW+3nSF6WUoqdfIs/Yx7NLjgVjiS4VccmIWI3nocMkzP207Xkk4D8fxsaOKbI2xWKdoKHz7tXX90pxqtxb1Bft6mH4c7f1bjX7U652RPnyZhHvXrjZHVDwFU6AeODAoipFgVxNx4pT2GEiuon44c9jeD7E9F1M+rvqcFPfxxFcrlwosPLiqNwHb3cR/kRkNcM9FGojz57u2Rtby4y8zvUbhkAI0jcMAXllLpa9Nvn+kZ7zm5D99ibr/gZPfk x-ms-office365-filtering-correlation-id: 5698c8a2-3577-49e2-751b-08d62273ecda x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989299)(5600074)(711020)(4618075)(4534165)(4627221)(201703031133081)(201702281549075)(8990200)(2017052603328)(7153060)(7193020); SRVR:DB3PR0502MB3996; x-ms-traffictypediagnostic: DB3PR0502MB3996: x-ld-processed: a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtAddr x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(788757137089); x-ms-exchange-senderadcheck: 1 x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(8121501046)(5005006)(10201501046)(3002001)(93006095)(93001095)(3231355)(944501410)(52105095)(6055026)(149066)(150027)(6041310)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123558120)(20161123560045)(20161123562045)(20161123564045)(201708071742011)(7699051); SRVR:DB3PR0502MB3996; BCL:0; PCL:0; RULEID:; SRVR:DB3PR0502MB3996; x-forefront-prvs: 0805EC9467 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(396003)(376002)(346002)(366004)(39860400002)(136003)(199004)(189003)(51234002)(66066001)(2616005)(476003)(446003)(486006)(11346002)(16200700003)(6436002)(305945005)(7736002)(53946003)(386003)(6506007)(6486002)(6512007)(186003)(102836004)(5250100002)(76176011)(25786009)(26005)(68736007)(6116002)(54906003)(110136005)(99286004)(1076002)(3846002)(71200400001)(52116002)(86362001)(53936002)(71190400001)(97736004)(8936002)(107886003)(14444005)(256004)(4326008)(316002)(2900100001)(81156014)(8676002)(36756003)(81166006)(105586002)(2906002)(106356001)(6636002)(478600001)(5660300001)(14454004)(579004)(559001)(569006); DIR:OUT; SFP:1101; SCL:1; SRVR:DB3PR0502MB3996; H:DB3PR0502MB3980.eurprd05.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; received-spf: None (protection.outlook.com: mellanox.com does not designate permitted sender hosts) x-microsoft-antispam-message-info: dQRtDcTIHdqEkLz5ZB/fzyJ2QAuZ5vwRb/8xd8G1reqay4aEWx3MGMzNOKNb0Pj2pGIU0Ohwy2GpJEtjWU3HhZIIXhcjcfzyDMQb9z1eN38Cps1rnFl1H7Hw7TuSiqPVdTXqfjwXM8k7JzBHbktUo7oiGswySPRPqAAcoqNkaB8sknEsTP+gf4s6+wGVItChVpw3H27MBzZzcOTn5/b8fVFOnXGjvpiTQ92D+0LADbZXuu3cBUrKTx2Gfan6BRxKekIdvVtvAvlL4/cgAVYDK9H17AMUr+pspz/RPjhPCCBQtWsZwmRPq9nKXA89ikJE/fsFfI9S1AhcA2vVHdyP2ASRALlp7pcVwQ4liWw/UP4= spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-Network-Message-Id: 5698c8a2-3577-49e2-751b-08d62273ecda X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Sep 2018 23:17:42.1233 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB3PR0502MB3996 Subject: [dpdk-dev] [PATCH v3 04/11] net/mlx5: add support for multiple flow drivers X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 24 Sep 2018 23:17:46 -0000 From: Ori Kam In the current PMD there is only support for Verbs driver API, to configure NIC rules and TC driver API, to configure eswitch rules. In order to support new drivers that will enable the use of new features for example the Direct Verbs driver API. There is a need to split each driver to a dedicated file and use function pointer to access the driver. This commit moves the Verbs API to a detected file and introduce the use of function pointers in the flow handling. The functions pointers that are in use: * validate - handle the validation of the flow. It can use both specific functions or shared functions that will be located in the mlx5_flow.c. * prepare - allocate a the device flow. There can be number of device flows that are connected to a single requested flow. * translate - converts the requested device flow into the driver flow. * apply - insert the flow into the NIC. * remove - remove the flow from the NIC but keeps it in memory. * destroy - remove the flow from memory. Signed-off-by: Ori Kam Acked-by: Yongseok Koh --- drivers/net/mlx5/Makefile | 1 + drivers/net/mlx5/meson.build | 1 + drivers/net/mlx5/mlx5.c | 2 + drivers/net/mlx5/mlx5_flow.c | 1910 ++------------------------------= ---- drivers/net/mlx5/mlx5_flow.h | 257 +++++ drivers/net/mlx5/mlx5_flow_verbs.c | 1692 ++++++++++++++++++++++++++++++++ 6 files changed, 2026 insertions(+), 1837 deletions(-) create mode 100644 drivers/net/mlx5/mlx5_flow.h create mode 100644 drivers/net/mlx5/mlx5_flow_verbs.c diff --git a/drivers/net/mlx5/Makefile b/drivers/net/mlx5/Makefile index 2e70dec5b..9bd6bfb82 100644 --- a/drivers/net/mlx5/Makefile +++ b/drivers/net/mlx5/Makefile @@ -31,6 +31,7 @@ SRCS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) +=3D mlx5_stats.c SRCS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) +=3D mlx5_rss.c SRCS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) +=3D mlx5_mr.c SRCS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) +=3D mlx5_flow.c +SRCS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) +=3D mlx5_flow_verbs.c SRCS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) +=3D mlx5_socket.c SRCS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) +=3D mlx5_nl.c SRCS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) +=3D mlx5_nl_flow.c diff --git a/drivers/net/mlx5/meson.build b/drivers/net/mlx5/meson.build index 289c7a4c0..40cc95038 100644 --- a/drivers/net/mlx5/meson.build +++ b/drivers/net/mlx5/meson.build @@ -31,6 +31,7 @@ if build 'mlx5.c', 'mlx5_ethdev.c', 'mlx5_flow.c', + 'mlx5_flow_verbs.c', 'mlx5_mac.c', 'mlx5_mr.c', 'mlx5_nl.c', diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c index fd89e2af3..ab44864e9 100644 --- a/drivers/net/mlx5/mlx5.c +++ b/drivers/net/mlx5/mlx5.c @@ -46,6 +46,7 @@ #include "mlx5_defs.h" #include "mlx5_glue.h" #include "mlx5_mr.h" +#include "mlx5_flow.h" =20 /* Device parameter to enable RX completion queue compression. */ #define MLX5_RXQ_CQE_COMP_EN "rxq_cqe_comp_en" @@ -1185,6 +1186,7 @@ mlx5_dev_spawn(struct rte_device *dpdk_dev, if (err < 0) goto error; priv->config.flow_prio =3D err; + mlx5_flow_init_driver_ops(eth_dev); /* * Once the device is added to the list of memory event * callback, its global MR cache table cannot be expanded diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c index 2d55f08b8..1c177b9c8 100644 --- a/drivers/net/mlx5/mlx5_flow.c +++ b/drivers/net/mlx5/mlx5_flow.c @@ -31,83 +31,12 @@ #include "mlx5_defs.h" #include "mlx5_prm.h" #include "mlx5_glue.h" +#include "mlx5_flow.h" =20 /* Dev ops structure defined in mlx5.c */ extern const struct eth_dev_ops mlx5_dev_ops; extern const struct eth_dev_ops mlx5_dev_ops_isolate; =20 -/* Pattern outer Layer bits. */ -#define MLX5_FLOW_LAYER_OUTER_L2 (1u << 0) -#define MLX5_FLOW_LAYER_OUTER_L3_IPV4 (1u << 1) -#define MLX5_FLOW_LAYER_OUTER_L3_IPV6 (1u << 2) -#define MLX5_FLOW_LAYER_OUTER_L4_UDP (1u << 3) -#define MLX5_FLOW_LAYER_OUTER_L4_TCP (1u << 4) -#define MLX5_FLOW_LAYER_OUTER_VLAN (1u << 5) - -/* Pattern inner Layer bits. */ -#define MLX5_FLOW_LAYER_INNER_L2 (1u << 6) -#define MLX5_FLOW_LAYER_INNER_L3_IPV4 (1u << 7) -#define MLX5_FLOW_LAYER_INNER_L3_IPV6 (1u << 8) -#define MLX5_FLOW_LAYER_INNER_L4_UDP (1u << 9) -#define MLX5_FLOW_LAYER_INNER_L4_TCP (1u << 10) -#define MLX5_FLOW_LAYER_INNER_VLAN (1u << 11) - -/* Pattern tunnel Layer bits. */ -#define MLX5_FLOW_LAYER_VXLAN (1u << 12) -#define MLX5_FLOW_LAYER_VXLAN_GPE (1u << 13) -#define MLX5_FLOW_LAYER_GRE (1u << 14) -#define MLX5_FLOW_LAYER_MPLS (1u << 15) - -/* Outer Masks. */ -#define MLX5_FLOW_LAYER_OUTER_L3 \ - (MLX5_FLOW_LAYER_OUTER_L3_IPV4 | MLX5_FLOW_LAYER_OUTER_L3_IPV6) -#define MLX5_FLOW_LAYER_OUTER_L4 \ - (MLX5_FLOW_LAYER_OUTER_L4_UDP | MLX5_FLOW_LAYER_OUTER_L4_TCP) -#define MLX5_FLOW_LAYER_OUTER \ - (MLX5_FLOW_LAYER_OUTER_L2 | MLX5_FLOW_LAYER_OUTER_L3 | \ - MLX5_FLOW_LAYER_OUTER_L4) - -/* Tunnel Masks. */ -#define MLX5_FLOW_LAYER_TUNNEL \ - (MLX5_FLOW_LAYER_VXLAN | MLX5_FLOW_LAYER_VXLAN_GPE | \ - MLX5_FLOW_LAYER_GRE | MLX5_FLOW_LAYER_MPLS) - -/* Inner Masks. */ -#define MLX5_FLOW_LAYER_INNER_L3 \ - (MLX5_FLOW_LAYER_INNER_L3_IPV4 | MLX5_FLOW_LAYER_INNER_L3_IPV6) -#define MLX5_FLOW_LAYER_INNER_L4 \ - (MLX5_FLOW_LAYER_INNER_L4_UDP | MLX5_FLOW_LAYER_INNER_L4_TCP) -#define MLX5_FLOW_LAYER_INNER \ - (MLX5_FLOW_LAYER_INNER_L2 | MLX5_FLOW_LAYER_INNER_L3 | \ - MLX5_FLOW_LAYER_INNER_L4) - -/* Actions that modify the fate of matching traffic. */ -#define MLX5_FLOW_FATE_DROP (1u << 0) -#define MLX5_FLOW_FATE_QUEUE (1u << 1) -#define MLX5_FLOW_FATE_RSS (1u << 2) - -/* Modify a packet. */ -#define MLX5_FLOW_MOD_FLAG (1u << 0) -#define MLX5_FLOW_MOD_MARK (1u << 1) -#define MLX5_FLOW_MOD_COUNT (1u << 2) - -/* Actions */ -#define MLX5_ACTION_DROP (1u << 0) -#define MLX5_ACTION_QUEUE (1u << 1) -#define MLX5_ACTION_RSS (1u << 2) -#define MLX5_ACTION_FLAG (1u << 3) -#define MLX5_ACTION_MARK (1u << 4) -#define MLX5_ACTION_COUNT (1u << 5) - -/* possible L3 layers protocols filtering. */ -#define MLX5_IP_PROTOCOL_TCP 6 -#define MLX5_IP_PROTOCOL_UDP 17 -#define MLX5_IP_PROTOCOL_GRE 47 -#define MLX5_IP_PROTOCOL_MPLS 147 - -/* Priority reserved for default flows. */ -#define MLX5_FLOW_PRIO_RSVD ((uint32_t)-1) - enum mlx5_expansion { MLX5_EXPANSION_ROOT, MLX5_EXPANSION_ROOT_OUTER, @@ -278,57 +207,6 @@ static const struct rte_flow_expand_node mlx5_support_= expansion[] =3D { }, }; =20 -/** Handles information leading to a drop fate. */ -struct mlx5_flow_verbs { - LIST_ENTRY(mlx5_flow_verbs) next; - unsigned int size; /**< Size of the attribute. */ - struct { - struct ibv_flow_attr *attr; - /**< Pointer to the Specification buffer. */ - uint8_t *specs; /**< Pointer to the specifications. */ - }; - struct ibv_flow *flow; /**< Verbs flow pointer. */ - struct mlx5_hrxq *hrxq; /**< Hash Rx queue object. */ - uint64_t hash_fields; /**< Verbs hash Rx queue hash fields. */ -}; - -/** Device flow structure. */ -struct mlx5_flow { - LIST_ENTRY(mlx5_flow) next; - struct rte_flow *flow; /**< Pointer to the main flow. */ - uint32_t layers; /**< Bit-fields that holds the detected layers. */ - union { - struct mlx5_flow_verbs verbs; /**< Holds the verbs dev-flow. */ - }; -}; - -/* Counters information. */ -struct mlx5_flow_counter { - LIST_ENTRY(mlx5_flow_counter) next; /**< Pointer to the next counter. */ - uint32_t shared:1; /**< Share counter ID with other flow rules. */ - uint32_t ref_cnt:31; /**< Reference counter. */ - uint32_t id; /**< Counter ID. */ - struct ibv_counter_set *cs; /**< Holds the counters for the rule. */ - uint64_t hits; /**< Number of packets matched by the rule. */ - uint64_t bytes; /**< Number of bytes matched by the rule. */ -}; - -/* Flow structure. */ -struct rte_flow { - TAILQ_ENTRY(rte_flow) next; /**< Pointer to the next flow structure. */ - struct rte_flow_attr attributes; /**< User flow attribute. */ - uint32_t layers; /**< Bit-fields that holds the detected layers. */ - /**< Bit-fields of present layers see MLX5_FLOW_LAYER_*. */ - struct mlx5_flow_counter *counter; /**< Holds Verbs flow counter. */ - struct rte_flow_action_rss rss;/**< RSS context. */ - uint8_t key[MLX5_RSS_HASH_KEY_LEN]; /**< RSS hash key. */ - uint16_t (*queue)[]; /**< Destination queues to redirect traffic to. */ - void *nl_flow; /**< Netlink flow buffer if relevant. */ - LIST_HEAD(dev_flows, mlx5_flow) dev_flows; - /**< Device flows that are part of the flow. */ - uint32_t actions; /**< Bit-fields which mark all detected actions. */ -}; - static const struct rte_flow_ops mlx5_flow_ops =3D { .validate =3D mlx5_flow_validate, .create =3D mlx5_flow_create, @@ -364,23 +242,6 @@ struct mlx5_fdir { struct rte_flow_action_queue queue; }; =20 -/* Verbs specification header. */ -struct ibv_spec_header { - enum ibv_flow_spec_type type; - uint16_t size; -}; - -/* - * Number of sub priorities. - * For each kind of pattern matching i.e. L2, L3, L4 to have a correct - * matching on the NIC (firmware dependent) L4 most have the higher priori= ty - * followed by L3 and ending with L2. - */ -#define MLX5_PRIORITY_MAP_L2 2 -#define MLX5_PRIORITY_MAP_L3 1 -#define MLX5_PRIORITY_MAP_L4 0 -#define MLX5_PRIORITY_MAP_MAX 3 - /* Map of Verbs to Flow priority with 8 Verbs priorities. */ static const uint32_t priority_map_3[][MLX5_PRIORITY_MAP_MAX] =3D { { 0, 1, 2 }, { 2, 3, 4 }, { 5, 6, 7 }, @@ -421,6 +282,9 @@ static struct mlx5_flow_tunnel_info tunnels_info[] =3D = { }, }; =20 +/* Holds the nic operations that should be used. */ +struct mlx5_flow_driver_ops nic_ops; + /** * Discover the maximum number of priority available. * @@ -490,7 +354,7 @@ mlx5_flow_discover_priorities(struct rte_eth_dev *dev) } =20 /** - * Adjust flow priority. + * Adjust flow priority based on the highest layer and the request priorit= y. * * @param[in] dev * Pointer to the Ethernet device structure. @@ -502,10 +366,8 @@ mlx5_flow_discover_priorities(struct rte_eth_dev *dev) * @return * The new priority. */ -static uint32_t -mlx5_flow_adjust_priority(struct rte_eth_dev *dev, - int32_t priority, - uint32_t subpriority) +uint32_t mlx5_flow_adjust_priority(struct rte_eth_dev *dev, int32_t priori= ty, + uint32_t subpriority) { uint32_t res =3D 0; struct priv *priv =3D dev->data->dev_private; @@ -522,80 +384,6 @@ mlx5_flow_adjust_priority(struct rte_eth_dev *dev, } =20 /** - * Get a flow counter. - * - * @param[in] dev - * Pointer to the Ethernet device structure. - * @param[in] shared - * Indicate if this counter is shared with other flows. - * @param[in] id - * Counter identifier. - * - * @return - * A pointer to the counter, NULL otherwise and rte_errno is set. - */ -static struct mlx5_flow_counter * -mlx5_flow_counter_new(struct rte_eth_dev *dev, uint32_t shared, uint32_t i= d) -{ - struct priv *priv =3D dev->data->dev_private; - struct mlx5_flow_counter *cnt; - - LIST_FOREACH(cnt, &priv->flow_counters, next) { - if (!cnt->shared || cnt->shared !=3D shared) - continue; - if (cnt->id !=3D id) - continue; - cnt->ref_cnt++; - return cnt; - } -#ifdef HAVE_IBV_DEVICE_COUNTERS_SET_SUPPORT - - struct mlx5_flow_counter tmpl =3D { - .shared =3D shared, - .id =3D id, - .cs =3D mlx5_glue->create_counter_set - (priv->ctx, - &(struct ibv_counter_set_init_attr){ - .counter_set_id =3D id, - }), - .hits =3D 0, - .bytes =3D 0, - }; - - if (!tmpl.cs) { - rte_errno =3D errno; - return NULL; - } - cnt =3D rte_calloc(__func__, 1, sizeof(*cnt), 0); - if (!cnt) { - rte_errno =3D ENOMEM; - return NULL; - } - *cnt =3D tmpl; - LIST_INSERT_HEAD(&priv->flow_counters, cnt, next); - return cnt; -#endif - rte_errno =3D ENOTSUP; - return NULL; -} - -/** - * Release a flow counter. - * - * @param[in] counter - * Pointer to the counter handler. - */ -static void -mlx5_flow_counter_release(struct mlx5_flow_counter *counter) -{ - if (--counter->ref_cnt =3D=3D 0) { - claim_zero(mlx5_glue->destroy_counter_set(counter->cs)); - LIST_REMOVE(counter, next); - rte_free(counter); - } -} - -/** * Verify the @p item specifications (spec, last, mask) are compatible wit= h the * NIC capabilities. * @@ -656,904 +444,6 @@ mlx5_flow_item_acceptable(const struct rte_flow_item = *item, } =20 /** - * Add a verbs item specification into @p flow. - * - * @param[in, out] flow - * Pointer to flow structure. - * @param[in] src - * Create specification. - * @param[in] size - * Size in bytes of the specification to copy. - */ -static void -mlx5_flow_spec_verbs_add(struct mlx5_flow *flow, void *src, unsigned int s= ize) -{ - struct mlx5_flow_verbs *verbs =3D &flow->verbs; - - if (verbs->specs) { - void *dst; - - dst =3D (void *)(verbs->specs + verbs->size); - memcpy(dst, src, size); - ++verbs->attr->num_of_specs; - } - verbs->size +=3D size; -} - -/** - * Adjust verbs hash fields according to the @p flow information. - * - * @param[in] dev_flow. - * Pointer to dev flow structure. - * @param[in] tunnel - * 1 when the hash field is for a tunnel item. - * @param[in] layer_types - * ETH_RSS_* types. - * @param[in] hash_fields - * Item hash fields. - */ -static void -mlx5_flow_verbs_hashfields_adjust(struct mlx5_flow *dev_flow, - int tunnel __rte_unused, - uint32_t layer_types, uint64_t hash_fields) -{ -#ifdef HAVE_IBV_DEVICE_TUNNEL_SUPPORT - int rss_request_inner =3D dev_flow->flow->rss.level >=3D 2; - - hash_fields |=3D (tunnel ? IBV_RX_HASH_INNER : 0); - if (rss_request_inner && !tunnel) - hash_fields =3D 0; - else if (!rss_request_inner && tunnel) - hash_fields =3D 0; -#endif - if (!(dev_flow->flow->rss.types & layer_types)) - hash_fields =3D 0; - dev_flow->verbs.hash_fields |=3D hash_fields; -} - -/** - * Convert the @p item into a Verbs specification. This function assumes t= hat - * the input is valid and that there is space to insert the requested item - * into the flow. - * - * @param[in] item - * Item specification. - * @param[in] item_flags - * Bit field with all detected items. - * @param[in, out] dev_flow - * Pointer to dev_flow structure. - */ -static void -flow_verbs_translate_item_eth(const struct rte_flow_item *item, - uint64_t *item_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_item_eth *spec =3D item->spec; - const struct rte_flow_item_eth *mask =3D item->mask; - const int tunnel =3D !!(*item_flags & MLX5_FLOW_LAYER_TUNNEL); - const unsigned int size =3D sizeof(struct ibv_flow_spec_eth); - struct ibv_flow_spec_eth eth =3D { - .type =3D IBV_FLOW_SPEC_ETH | (tunnel ? IBV_FLOW_SPEC_INNER : 0), - .size =3D size, - }; - - if (!mask) - mask =3D &rte_flow_item_eth_mask; - if (spec) { - unsigned int i; - - memcpy(ð.val.dst_mac, spec->dst.addr_bytes, ETHER_ADDR_LEN); - memcpy(ð.val.src_mac, spec->src.addr_bytes, ETHER_ADDR_LEN); - eth.val.ether_type =3D spec->type; - memcpy(ð.mask.dst_mac, mask->dst.addr_bytes, ETHER_ADDR_LEN); - memcpy(ð.mask.src_mac, mask->src.addr_bytes, ETHER_ADDR_LEN); - eth.mask.ether_type =3D mask->type; - /* Remove unwanted bits from values. */ - for (i =3D 0; i < ETHER_ADDR_LEN; ++i) { - eth.val.dst_mac[i] &=3D eth.mask.dst_mac[i]; - eth.val.src_mac[i] &=3D eth.mask.src_mac[i]; - } - eth.val.ether_type &=3D eth.mask.ether_type; - dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L2; - } - mlx5_flow_spec_verbs_add(dev_flow, ð, size); - *item_flags |=3D tunnel ? - MLX5_FLOW_LAYER_INNER_L2 : - MLX5_FLOW_LAYER_OUTER_L2; -} - -/** - * Update the VLAN tag in the Verbs Ethernet specification. - * This function assumes that the input is valid and there is space to add - * the requested item. - * - * @param[in, out] attr - * Pointer to Verbs attributes structure. - * @param[in] eth - * Verbs structure containing the VLAN information to copy. - */ -static void -mlx5_flow_item_vlan_update(struct ibv_flow_attr *attr, - struct ibv_flow_spec_eth *eth) -{ - unsigned int i; - const enum ibv_flow_spec_type search =3D eth->type; - struct ibv_spec_header *hdr =3D (struct ibv_spec_header *) - ((uint8_t *)attr + sizeof(struct ibv_flow_attr)); - - for (i =3D 0; i !=3D attr->num_of_specs; ++i) { - if (hdr->type =3D=3D search) { - struct ibv_flow_spec_eth *e =3D - (struct ibv_flow_spec_eth *)hdr; - - e->val.vlan_tag =3D eth->val.vlan_tag; - e->mask.vlan_tag =3D eth->mask.vlan_tag; - e->val.ether_type =3D eth->val.ether_type; - e->mask.ether_type =3D eth->mask.ether_type; - break; - } - hdr =3D (struct ibv_spec_header *)((uint8_t *)hdr + hdr->size); - } -} - -/** - * Convert the @p item into a Verbs specification. This function assumes t= hat - * the input is valid and that there is space to insert the requested item - * into the flow. - * - * @param[in] item - * Item specification. - * @param[in, out] item_flags - * Bit mask that holds all detected items. - * @param[in, out] dev_flow - * Pointer to dev_flow structure. - */ -static void -flow_verbs_translate_item_vlan(const struct rte_flow_item *item, - uint64_t *item_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_item_vlan *spec =3D item->spec; - const struct rte_flow_item_vlan *mask =3D item->mask; - unsigned int size =3D sizeof(struct ibv_flow_spec_eth); - const int tunnel =3D !!(*item_flags & MLX5_FLOW_LAYER_TUNNEL); - struct ibv_flow_spec_eth eth =3D { - .type =3D IBV_FLOW_SPEC_ETH | (tunnel ? IBV_FLOW_SPEC_INNER : 0), - .size =3D size, - }; - const uint32_t l2m =3D tunnel ? MLX5_FLOW_LAYER_INNER_L2 : - MLX5_FLOW_LAYER_OUTER_L2; - - if (!mask) - mask =3D &rte_flow_item_vlan_mask; - if (spec) { - eth.val.vlan_tag =3D spec->tci; - eth.mask.vlan_tag =3D mask->tci; - eth.val.vlan_tag &=3D eth.mask.vlan_tag; - eth.val.ether_type =3D spec->inner_type; - eth.mask.ether_type =3D mask->inner_type; - eth.val.ether_type &=3D eth.mask.ether_type; - } - if (!(*item_flags & l2m)) { - dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L2; - mlx5_flow_spec_verbs_add(dev_flow, ð, size); - } else { - mlx5_flow_item_vlan_update(dev_flow->verbs.attr, - ð); - size =3D 0; /* Only an update is done in eth specification. */ - } - *item_flags |=3D tunnel ? - (MLX5_FLOW_LAYER_INNER_L2 | - MLX5_FLOW_LAYER_INNER_VLAN) : - (MLX5_FLOW_LAYER_OUTER_L2 | MLX5_FLOW_LAYER_OUTER_VLAN); -} - -/** - * Convert the @p item into a Verbs specification. This function assumes t= hat - * the input is valid and that there is space to insert the requested item - * into the flow. - * - * @param[in] item - * Item specification. - * @param[in, out] item_flags - * Bit mask that marks all detected items. - * @param[in, out] dev_flow - * Pointer to sepacific flow structure. - */ -static void -flow_verbs_translate_item_ipv4(const struct rte_flow_item *item, - uint64_t *item_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_item_ipv4 *spec =3D item->spec; - const struct rte_flow_item_ipv4 *mask =3D item->mask; - const int tunnel =3D !!(*item_flags & MLX5_FLOW_LAYER_TUNNEL); - unsigned int size =3D sizeof(struct ibv_flow_spec_ipv4_ext); - struct ibv_flow_spec_ipv4_ext ipv4 =3D { - .type =3D IBV_FLOW_SPEC_IPV4_EXT | - (tunnel ? IBV_FLOW_SPEC_INNER : 0), - .size =3D size, - }; - - if (!mask) - mask =3D &rte_flow_item_ipv4_mask; - *item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L3_IPV4 : - MLX5_FLOW_LAYER_OUTER_L3_IPV4; - if (spec) { - ipv4.val =3D (struct ibv_flow_ipv4_ext_filter){ - .src_ip =3D spec->hdr.src_addr, - .dst_ip =3D spec->hdr.dst_addr, - .proto =3D spec->hdr.next_proto_id, - .tos =3D spec->hdr.type_of_service, - }; - ipv4.mask =3D (struct ibv_flow_ipv4_ext_filter){ - .src_ip =3D mask->hdr.src_addr, - .dst_ip =3D mask->hdr.dst_addr, - .proto =3D mask->hdr.next_proto_id, - .tos =3D mask->hdr.type_of_service, - }; - /* Remove unwanted bits from values. */ - ipv4.val.src_ip &=3D ipv4.mask.src_ip; - ipv4.val.dst_ip &=3D ipv4.mask.dst_ip; - ipv4.val.proto &=3D ipv4.mask.proto; - ipv4.val.tos &=3D ipv4.mask.tos; - } - mlx5_flow_verbs_hashfields_adjust(dev_flow, tunnel, - (ETH_RSS_IPV4 | ETH_RSS_FRAG_IPV4 | - ETH_RSS_NONFRAG_IPV4_TCP | - ETH_RSS_NONFRAG_IPV4_UDP | - ETH_RSS_NONFRAG_IPV4_OTHER), - (IBV_RX_HASH_SRC_IPV4 | - IBV_RX_HASH_DST_IPV4)); - dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L3; - mlx5_flow_spec_verbs_add(dev_flow, &ipv4, size); -} - -/** - * Convert the @p item into a Verbs specification. This function assumes t= hat - * the input is valid and that there is space to insert the requested item - * into the flow. - * - * @param[in] item - * Item specification. - * @param[in, out] item_flags - * Bit mask that marks all detected items. - * @param[in, out] dev_flow - * Pointer to sepacific flow structure. - */ -static void -flow_verbs_translate_item_ipv6(const struct rte_flow_item *item, - uint64_t *item_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_item_ipv6 *spec =3D item->spec; - const struct rte_flow_item_ipv6 *mask =3D item->mask; - const int tunnel =3D !!(dev_flow->flow->layers & MLX5_FLOW_LAYER_TUNNEL); - unsigned int size =3D sizeof(struct ibv_flow_spec_ipv6); - struct ibv_flow_spec_ipv6 ipv6 =3D { - .type =3D IBV_FLOW_SPEC_IPV6 | (tunnel ? IBV_FLOW_SPEC_INNER : 0), - .size =3D size, - }; - - if (!mask) - mask =3D &rte_flow_item_ipv6_mask; - *item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L3_IPV6 : - MLX5_FLOW_LAYER_OUTER_L3_IPV6; - if (spec) { - unsigned int i; - uint32_t vtc_flow_val; - uint32_t vtc_flow_mask; - - memcpy(&ipv6.val.src_ip, spec->hdr.src_addr, - RTE_DIM(ipv6.val.src_ip)); - memcpy(&ipv6.val.dst_ip, spec->hdr.dst_addr, - RTE_DIM(ipv6.val.dst_ip)); - memcpy(&ipv6.mask.src_ip, mask->hdr.src_addr, - RTE_DIM(ipv6.mask.src_ip)); - memcpy(&ipv6.mask.dst_ip, mask->hdr.dst_addr, - RTE_DIM(ipv6.mask.dst_ip)); - vtc_flow_val =3D rte_be_to_cpu_32(spec->hdr.vtc_flow); - vtc_flow_mask =3D rte_be_to_cpu_32(mask->hdr.vtc_flow); - ipv6.val.flow_label =3D - rte_cpu_to_be_32((vtc_flow_val & IPV6_HDR_FL_MASK) >> - IPV6_HDR_FL_SHIFT); - ipv6.val.traffic_class =3D (vtc_flow_val & IPV6_HDR_TC_MASK) >> - IPV6_HDR_TC_SHIFT; - ipv6.val.next_hdr =3D spec->hdr.proto; - ipv6.val.hop_limit =3D spec->hdr.hop_limits; - ipv6.mask.flow_label =3D - rte_cpu_to_be_32((vtc_flow_mask & IPV6_HDR_FL_MASK) >> - IPV6_HDR_FL_SHIFT); - ipv6.mask.traffic_class =3D (vtc_flow_mask & IPV6_HDR_TC_MASK) >> - IPV6_HDR_TC_SHIFT; - ipv6.mask.next_hdr =3D mask->hdr.proto; - ipv6.mask.hop_limit =3D mask->hdr.hop_limits; - /* Remove unwanted bits from values. */ - for (i =3D 0; i < RTE_DIM(ipv6.val.src_ip); ++i) { - ipv6.val.src_ip[i] &=3D ipv6.mask.src_ip[i]; - ipv6.val.dst_ip[i] &=3D ipv6.mask.dst_ip[i]; - } - ipv6.val.flow_label &=3D ipv6.mask.flow_label; - ipv6.val.traffic_class &=3D ipv6.mask.traffic_class; - ipv6.val.next_hdr &=3D ipv6.mask.next_hdr; - ipv6.val.hop_limit &=3D ipv6.mask.hop_limit; - } - mlx5_flow_verbs_hashfields_adjust(dev_flow, tunnel, - (ETH_RSS_IPV6 | ETH_RSS_FRAG_IPV6 | - ETH_RSS_NONFRAG_IPV6_TCP | - ETH_RSS_NONFRAG_IPV6_UDP | - ETH_RSS_IPV6_EX | - ETH_RSS_IPV6_TCP_EX | - ETH_RSS_IPV6_UDP_EX | - ETH_RSS_NONFRAG_IPV6_OTHER), - (IBV_RX_HASH_SRC_IPV6 | - IBV_RX_HASH_DST_IPV6)); - dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L3; - mlx5_flow_spec_verbs_add(dev_flow, &ipv6, size); -} - -/** - * Convert the @p item into a Verbs specification. This function assumes t= hat - * the input is valid and that there is space to insert the requested item - * into the flow. - * - * @param[in] item - * Item specification. - * @param[in, out] item_flags - * Bit mask that marks all detected items. - * @param[in, out] dev_flow - * Pointer to sepacific flow structure. - */ -static void -flow_verbs_translate_item_udp(const struct rte_flow_item *item, - uint64_t *item_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_item_udp *spec =3D item->spec; - const struct rte_flow_item_udp *mask =3D item->mask; - const int tunnel =3D !!(*item_flags & MLX5_FLOW_LAYER_TUNNEL); - unsigned int size =3D sizeof(struct ibv_flow_spec_tcp_udp); - struct ibv_flow_spec_tcp_udp udp =3D { - .type =3D IBV_FLOW_SPEC_UDP | (tunnel ? IBV_FLOW_SPEC_INNER : 0), - .size =3D size, - }; - - if (!mask) - mask =3D &rte_flow_item_udp_mask; - *item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L4_UDP : - MLX5_FLOW_LAYER_OUTER_L4_UDP; - if (spec) { - udp.val.dst_port =3D spec->hdr.dst_port; - udp.val.src_port =3D spec->hdr.src_port; - udp.mask.dst_port =3D mask->hdr.dst_port; - udp.mask.src_port =3D mask->hdr.src_port; - /* Remove unwanted bits from values. */ - udp.val.src_port &=3D udp.mask.src_port; - udp.val.dst_port &=3D udp.mask.dst_port; - } - mlx5_flow_verbs_hashfields_adjust(dev_flow, - tunnel, ETH_RSS_UDP, - (IBV_RX_HASH_SRC_PORT_UDP | - IBV_RX_HASH_DST_PORT_UDP)); - dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L4; - mlx5_flow_spec_verbs_add(dev_flow, &udp, size); -} - -/** - * Convert the @p item into a Verbs specification. This function assumes t= hat - * the input is valid and that there is space to insert the requested item - * into the flow. - * - * @param[in] item - * Item specification. - * @param[in, out] item_flags - * Bit mask that marks all detected items. - * @param[in, out] dev_flow - * Pointer to sepacific flow structure. - */ -static void -flow_verbs_translate_item_tcp(const struct rte_flow_item *item, - uint64_t *item_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_item_tcp *spec =3D item->spec; - const struct rte_flow_item_tcp *mask =3D item->mask; - const int tunnel =3D !!(dev_flow->flow->layers & MLX5_FLOW_LAYER_TUNNEL); - unsigned int size =3D sizeof(struct ibv_flow_spec_tcp_udp); - struct ibv_flow_spec_tcp_udp tcp =3D { - .type =3D IBV_FLOW_SPEC_TCP | (tunnel ? IBV_FLOW_SPEC_INNER : 0), - .size =3D size, - }; - - if (!mask) - mask =3D &rte_flow_item_tcp_mask; - *item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L4_TCP : - MLX5_FLOW_LAYER_OUTER_L4_TCP; - if (spec) { - tcp.val.dst_port =3D spec->hdr.dst_port; - tcp.val.src_port =3D spec->hdr.src_port; - tcp.mask.dst_port =3D mask->hdr.dst_port; - tcp.mask.src_port =3D mask->hdr.src_port; - /* Remove unwanted bits from values. */ - tcp.val.src_port &=3D tcp.mask.src_port; - tcp.val.dst_port &=3D tcp.mask.dst_port; - } - mlx5_flow_verbs_hashfields_adjust(dev_flow, - tunnel, ETH_RSS_TCP, - (IBV_RX_HASH_SRC_PORT_TCP | - IBV_RX_HASH_DST_PORT_TCP)); - dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L4; - mlx5_flow_spec_verbs_add(dev_flow, &tcp, size); -} - -/** - * Convert the @p item into a Verbs specification. This function assumes t= hat - * the input is valid and that there is space to insert the requested item - * into the flow. - * - * @param[in] item - * Item specification. - * @param[in, out] item_flags - * Bit mask that marks all detected items. - * @param[in, out] dev_flow - * Pointer to sepacific flow structure. - */ -static void -flow_verbs_translate_item_vxlan(const struct rte_flow_item *item, - uint64_t *item_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_item_vxlan *spec =3D item->spec; - const struct rte_flow_item_vxlan *mask =3D item->mask; - unsigned int size =3D sizeof(struct ibv_flow_spec_tunnel); - struct ibv_flow_spec_tunnel vxlan =3D { - .type =3D IBV_FLOW_SPEC_VXLAN_TUNNEL, - .size =3D size, - }; - union vni { - uint32_t vlan_id; - uint8_t vni[4]; - } id =3D { .vlan_id =3D 0, }; - - if (!mask) - mask =3D &rte_flow_item_vxlan_mask; - if (spec) { - memcpy(&id.vni[1], spec->vni, 3); - vxlan.val.tunnel_id =3D id.vlan_id; - memcpy(&id.vni[1], mask->vni, 3); - vxlan.mask.tunnel_id =3D id.vlan_id; - /* Remove unwanted bits from values. */ - vxlan.val.tunnel_id &=3D vxlan.mask.tunnel_id; - } - mlx5_flow_spec_verbs_add(dev_flow, &vxlan, size); - dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L2; - *item_flags |=3D MLX5_FLOW_LAYER_VXLAN; -} - -/** - * Convert the @p item into a Verbs specification. This function assumes t= hat - * the input is valid and that there is space to insert the requested item - * into the flow. - * - * @param[in] item - * Item specification. - * @param[in, out] item_flags - * Bit mask that marks all detected items. - * @param[in, out] dev_flow - * Pointer to sepacific flow structure. - */ -static void -flow_verbs_translate_item_vxlan_gpe(const struct rte_flow_item *item, - uint64_t *item_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_item_vxlan_gpe *spec =3D item->spec; - const struct rte_flow_item_vxlan_gpe *mask =3D item->mask; - unsigned int size =3D sizeof(struct ibv_flow_spec_tunnel); - struct ibv_flow_spec_tunnel vxlan_gpe =3D { - .type =3D IBV_FLOW_SPEC_VXLAN_TUNNEL, - .size =3D size, - }; - union vni { - uint32_t vlan_id; - uint8_t vni[4]; - } id =3D { .vlan_id =3D 0, }; - - if (!mask) - mask =3D &rte_flow_item_vxlan_gpe_mask; - if (spec) { - memcpy(&id.vni[1], spec->vni, 3); - vxlan_gpe.val.tunnel_id =3D id.vlan_id; - memcpy(&id.vni[1], mask->vni, 3); - vxlan_gpe.mask.tunnel_id =3D id.vlan_id; - /* Remove unwanted bits from values. */ - vxlan_gpe.val.tunnel_id &=3D vxlan_gpe.mask.tunnel_id; - } - mlx5_flow_spec_verbs_add(dev_flow, &vxlan_gpe, size); - dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L2; - *item_flags |=3D MLX5_FLOW_LAYER_VXLAN_GPE; -} - -/** - * Update the protocol in Verbs IPv4/IPv6 spec. - * - * @param[in, out] attr - * Pointer to Verbs attributes structure. - * @param[in] search - * Specification type to search in order to update the IP protocol. - * @param[in] protocol - * Protocol value to set if none is present in the specification. - */ -static void -mlx5_flow_item_gre_ip_protocol_update(struct ibv_flow_attr *attr, - enum ibv_flow_spec_type search, - uint8_t protocol) -{ - unsigned int i; - struct ibv_spec_header *hdr =3D (struct ibv_spec_header *) - ((uint8_t *)attr + sizeof(struct ibv_flow_attr)); - - if (!attr) - return; - for (i =3D 0; i !=3D attr->num_of_specs; ++i) { - if (hdr->type =3D=3D search) { - union { - struct ibv_flow_spec_ipv4_ext *ipv4; - struct ibv_flow_spec_ipv6 *ipv6; - } ip; - - switch (search) { - case IBV_FLOW_SPEC_IPV4_EXT: - ip.ipv4 =3D (struct ibv_flow_spec_ipv4_ext *)hdr; - if (!ip.ipv4->val.proto) { - ip.ipv4->val.proto =3D protocol; - ip.ipv4->mask.proto =3D 0xff; - } - break; - case IBV_FLOW_SPEC_IPV6: - ip.ipv6 =3D (struct ibv_flow_spec_ipv6 *)hdr; - if (!ip.ipv6->val.next_hdr) { - ip.ipv6->val.next_hdr =3D protocol; - ip.ipv6->mask.next_hdr =3D 0xff; - } - break; - default: - break; - } - break; - } - hdr =3D (struct ibv_spec_header *)((uint8_t *)hdr + hdr->size); - } -} - -/** - * Convert the @p item into a Verbs specification. This function assumes t= hat - * the input is valid and that there is space to insert the requested item - * into the flow. - * - * @param[in] item - * Item specification. - * @param[in, out] item_flags - * Bit mask that marks all detected items. - * @param[in, out] dev_flow - * Pointer to sepacific flow structure. - */ -static void -flow_verbs_translate_item_gre(const struct rte_flow_item *item __rte_unuse= d, - uint64_t *item_flags, - struct mlx5_flow *dev_flow) -{ - struct mlx5_flow_verbs *verbs =3D &dev_flow->verbs; -#ifndef HAVE_IBV_DEVICE_MPLS_SUPPORT - unsigned int size =3D sizeof(struct ibv_flow_spec_tunnel); - struct ibv_flow_spec_tunnel tunnel =3D { - .type =3D IBV_FLOW_SPEC_VXLAN_TUNNEL, - .size =3D size, - }; -#else - const struct rte_flow_item_gre *spec =3D item->spec; - const struct rte_flow_item_gre *mask =3D item->mask; - unsigned int size =3D sizeof(struct ibv_flow_spec_gre); - struct ibv_flow_spec_gre tunnel =3D { - .type =3D IBV_FLOW_SPEC_GRE, - .size =3D size, - }; - - if (!mask) - mask =3D &rte_flow_item_gre_mask; - if (spec) { - tunnel.val.c_ks_res0_ver =3D spec->c_rsvd0_ver; - tunnel.val.protocol =3D spec->protocol; - tunnel.mask.c_ks_res0_ver =3D mask->c_rsvd0_ver; - tunnel.mask.protocol =3D mask->protocol; - /* Remove unwanted bits from values. */ - tunnel.val.c_ks_res0_ver &=3D tunnel.mask.c_ks_res0_ver; - tunnel.val.protocol &=3D tunnel.mask.protocol; - tunnel.val.key &=3D tunnel.mask.key; - } -#endif - if (*item_flags & MLX5_FLOW_LAYER_OUTER_L3_IPV4) - mlx5_flow_item_gre_ip_protocol_update - (verbs->attr, IBV_FLOW_SPEC_IPV4_EXT, - MLX5_IP_PROTOCOL_GRE); - else - mlx5_flow_item_gre_ip_protocol_update - (verbs->attr, IBV_FLOW_SPEC_IPV6, - MLX5_IP_PROTOCOL_GRE); - mlx5_flow_spec_verbs_add(dev_flow, &tunnel, size); - verbs->attr->priority =3D MLX5_PRIORITY_MAP_L2; - *item_flags |=3D MLX5_FLOW_LAYER_GRE; -} - -/** - * Convert the @p action into a Verbs specification. This function assumes= that - * the input is valid and that there is space to insert the requested acti= on - * into the flow. This function also return the action that was added. - * - * @param[in] item - * Item specification. - * @param[in, out] item_flags - * Bit mask that marks all detected items. - * @param[in, out] dev_flow - * Pointer to sepacific flow structure. - */ -static void -flow_verbs_translate_item_mpls(const struct rte_flow_item *item __rte_unus= ed, - uint64_t *action_flags __rte_unused, - struct mlx5_flow *dev_flow __rte_unused) -{ -#ifdef HAVE_IBV_DEVICE_MPLS_SUPPORT - const struct rte_flow_item_mpls *spec =3D item->spec; - const struct rte_flow_item_mpls *mask =3D item->mask; - unsigned int size =3D sizeof(struct ibv_flow_spec_mpls); - struct ibv_flow_spec_mpls mpls =3D { - .type =3D IBV_FLOW_SPEC_MPLS, - .size =3D size, - }; - - if (!mask) - mask =3D &rte_flow_item_mpls_mask; - if (spec) { - memcpy(&mpls.val.label, spec, sizeof(mpls.val.label)); - memcpy(&mpls.mask.label, mask, sizeof(mpls.mask.label)); - /* Remove unwanted bits from values. */ - mpls.val.label &=3D mpls.mask.label; - } - mlx5_flow_spec_verbs_add(dev_flow, &mpls, size); - dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L2; - *action_flags |=3D MLX5_FLOW_LAYER_MPLS; -#endif -} - -/** - * Convert the @p action into a Verbs specification. This function assumes= that - * the input is valid and that there is space to insert the requested acti= on - * into the flow. This function also return the action that was added. - * - * @param[in, out] action_flags - * Pointer to the detected actions. - * @param[in] dev_flow - * Pointer to mlx5_flow. - */ -static void -flow_verbs_translate_action_drop(uint64_t *action_flags, - struct mlx5_flow *dev_flow) -{ - unsigned int size =3D sizeof(struct ibv_flow_spec_action_drop); - struct ibv_flow_spec_action_drop drop =3D { - .type =3D IBV_FLOW_SPEC_ACTION_DROP, - .size =3D size, - }; - - mlx5_flow_spec_verbs_add(dev_flow, &drop, size); - *action_flags |=3D MLX5_ACTION_DROP; -} - -/** - * Convert the @p action into a Verbs specification. This function assumes= that - * the input is valid and that there is space to insert the requested acti= on - * into the flow. This function also return the action that was added. - * - * @param[in] action - * Action configuration. - * @param[in, out] action_flags - * Pointer to the detected actions. - * @param[in] dev_flow - * Pointer to mlx5_flow. - */ -static void -flow_verbs_translate_action_queue(const struct rte_flow_action *action, - uint64_t *action_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_action_queue *queue =3D action->conf; - struct rte_flow *flow =3D dev_flow->flow; - - if (flow->queue) - (*flow->queue)[0] =3D queue->index; - flow->rss.queue_num =3D 1; - *action_flags |=3D MLX5_ACTION_QUEUE; -} - -/** - * Convert the @p action into a Verbs specification. This function assumes= that - * the input is valid and that there is space to insert the requested acti= on - * into the flow. This function also return the action that was added. - * - * @param[in] action - * Action configuration. - * @param[in, out] action_flags - * Pointer to the detected actions. - * @param[in] dev_flow - * Pointer to mlx5_flow. - */ -static void -flow_verbs_translate_action_rss(const struct rte_flow_action *action, - uint64_t *action_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_action_rss *rss =3D action->conf; - struct rte_flow *flow =3D dev_flow->flow; - - if (flow->queue) - memcpy((*flow->queue), rss->queue, - rss->queue_num * sizeof(uint16_t)); - flow->rss.queue_num =3D rss->queue_num; - memcpy(flow->key, rss->key, MLX5_RSS_HASH_KEY_LEN); - flow->rss.types =3D rss->types; - flow->rss.level =3D rss->level; - *action_flags |=3D MLX5_ACTION_RSS; -} - -/** - * Convert the @p action into a Verbs specification. This function assumes= that - * the input is valid and that there is space to insert the requested acti= on - * into the flow. This function also return the action that was added. - * - * @param[in] action - * Action configuration. - * @param[in, out] action_flags - * Pointer to the detected actions. - * @param[in] dev_flow - * Pointer to mlx5_flow. - */ -static void -flow_verbs_translate_action_flag - (const struct rte_flow_action *action __rte_unused, - uint64_t *action_flags, - struct mlx5_flow *dev_flow) -{ - unsigned int size =3D sizeof(struct ibv_flow_spec_action_tag); - struct ibv_flow_spec_action_tag tag =3D { - .type =3D IBV_FLOW_SPEC_ACTION_TAG, - .size =3D size, - .tag_id =3D mlx5_flow_mark_set(MLX5_FLOW_MARK_DEFAULT), - }; - *action_flags |=3D MLX5_ACTION_MARK; - mlx5_flow_spec_verbs_add(dev_flow, &tag, size); -} - -/** - * Update verbs specification to modify the flag to mark. - * - * @param[in, out] verbs - * Pointer to the mlx5_flow_verbs structure. - * @param[in] mark_id - * Mark identifier to replace the flag. - */ -static void -flow_verbs_mark_update(struct mlx5_flow_verbs *verbs, uint32_t mark_id) -{ - struct ibv_spec_header *hdr; - int i; - - if (!verbs) - return; - /* Update Verbs specification. */ - hdr =3D (struct ibv_spec_header *)verbs->specs; - if (!hdr) - return; - for (i =3D 0; i !=3D verbs->attr->num_of_specs; ++i) { - if (hdr->type =3D=3D IBV_FLOW_SPEC_ACTION_TAG) { - struct ibv_flow_spec_action_tag *t =3D - (struct ibv_flow_spec_action_tag *)hdr; - - t->tag_id =3D mlx5_flow_mark_set(mark_id); - } - hdr =3D (struct ibv_spec_header *)((uintptr_t)hdr + hdr->size); - } -} - -/** - * Convert the @p action into a Verbs specification. This function assumes= that - * the input is valid and that there is space to insert the requested acti= on - * into the flow. This function also return the action that was added. - * - * @param[in] action - * Action configuration. - * @param[in, out] action_flags - * Pointer to the detected actions. - * @param[in] dev_flow - * Pointer to mlx5_flow. - */ -static void -flow_verbs_translate_action_mark(const struct rte_flow_action *action, - uint64_t *action_flags, - struct mlx5_flow *dev_flow) -{ - const struct rte_flow_action_mark *mark =3D action->conf; - unsigned int size =3D sizeof(struct ibv_flow_spec_action_tag); - struct ibv_flow_spec_action_tag tag =3D { - .type =3D IBV_FLOW_SPEC_ACTION_TAG, - .size =3D size, - }; - struct mlx5_flow_verbs *verbs =3D &dev_flow->verbs; - - if (*action_flags & MLX5_ACTION_FLAG) { - flow_verbs_mark_update(verbs, mark->id); - size =3D 0; - } else { - tag.tag_id =3D mlx5_flow_mark_set(mark->id); - mlx5_flow_spec_verbs_add(dev_flow, &tag, size); - } - *action_flags |=3D MLX5_ACTION_MARK; -} - -/** - * Convert the @p action into a Verbs specification. This function assumes= that - * the input is valid and that there is space to insert the requested acti= on - * into the flow. This function also return the action that was added. - * - * @param[in] dev - * Pointer to the Ethernet device structure. - * @param[in] action - * Action configuration. - * @param[in, out] action_flags - * Pointer to the detected actions. - * @param[in] dev_flow - * Pointer to mlx5_flow. - * @param[out] error - * Pointer to error structure. - * - * @return - * 0 On success else a negative errno value is returned and rte_errno is= set. - */ -static int -flow_verbs_translate_action_count(struct rte_eth_dev *dev, - const struct rte_flow_action *action, - uint64_t *action_flags, - struct mlx5_flow *dev_flow, - struct rte_flow_error *error) -{ - const struct rte_flow_action_count *count =3D action->conf; - struct rte_flow *flow =3D dev_flow->flow; -#ifdef HAVE_IBV_DEVICE_COUNTERS_SET_SUPPORT - unsigned int size =3D sizeof(struct ibv_flow_spec_counter_action); - struct ibv_flow_spec_counter_action counter =3D { - .type =3D IBV_FLOW_SPEC_ACTION_COUNT, - .size =3D size, - }; -#endif - - if (!flow->counter) { - flow->counter =3D mlx5_flow_counter_new(dev, count->shared, - count->id); - if (!flow->counter) - return rte_flow_error_set(error, ENOTSUP, - RTE_FLOW_ERROR_TYPE_ACTION, - action, - "cannot get counter" - " context."); - } - *action_flags |=3D MLX5_ACTION_COUNT; -#ifdef HAVE_IBV_DEVICE_COUNTERS_SET_SUPPORT - counter.counter_set_handle =3D flow->counter->cs->handle; - mlx5_flow_spec_verbs_add(dev_flow, &counter, size); -#endif - return 0; -} - -/** * Lookup and set the ptype in the data Rx part. A single Ptype can be us= ed, * if several tunnel rules are used on this queue, the tunnel ptype will b= e * cleared. @@ -1708,7 +598,7 @@ mlx5_flow_rxq_flags_clear(struct rte_eth_dev *dev) * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_action_flag(uint64_t action_flags, struct rte_flow_error *error) { @@ -1742,7 +632,7 @@ mlx5_flow_validate_action_flag(uint64_t action_flags, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_action_mark(const struct rte_flow_action *action, uint64_t action_flags, struct rte_flow_error *error) @@ -1787,7 +677,7 @@ mlx5_flow_validate_action_mark(const struct rte_flow_a= ction *action, * @return * 0 on success, a negative errno value otherwise and rte_ernno is set. */ -static int +int mlx5_flow_validate_action_drop(uint64_t action_flags, struct rte_flow_error *error) { @@ -1823,7 +713,7 @@ mlx5_flow_validate_action_drop(uint64_t action_flags, * @return * 0 on success, a negative errno value otherwise and rte_ernno is set. */ -static int +int mlx5_flow_validate_action_queue(const struct rte_flow_action *action, uint64_t action_flags, struct rte_eth_dev *dev, @@ -1866,7 +756,7 @@ mlx5_flow_validate_action_queue(const struct rte_flow_= action *action, * @return * 0 on success, a negative errno value otherwise and rte_ernno is set. */ -static int +int mlx5_flow_validate_action_rss(const struct rte_flow_action *action, uint64_t action_flags, struct rte_eth_dev *dev, @@ -1938,7 +828,7 @@ mlx5_flow_validate_action_rss(const struct rte_flow_ac= tion *action, * @return * 0 on success, a negative errno value otherwise and rte_ernno is set. */ -static int +int mlx5_flow_validate_action_count(struct rte_eth_dev *dev, struct rte_flow_error *error) { @@ -1965,7 +855,7 @@ mlx5_flow_validate_action_count(struct rte_eth_dev *de= v, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_attributes(struct rte_eth_dev *dev, const struct rte_flow_attr *attributes, struct rte_flow_error *error) @@ -2011,7 +901,7 @@ mlx5_flow_validate_attributes(struct rte_eth_dev *dev, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_item_eth(const struct rte_flow_item *item, uint64_t item_flags, struct rte_flow_error *error) @@ -2055,7 +945,7 @@ mlx5_flow_validate_item_eth(const struct rte_flow_item= *item, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_item_vlan(const struct rte_flow_item *item, int64_t item_flags, struct rte_flow_error *error) @@ -2116,7 +1006,7 @@ mlx5_flow_validate_item_vlan(const struct rte_flow_it= em *item, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_item_ipv4(const struct rte_flow_item *item, int64_t item_flags, struct rte_flow_error *error) @@ -2167,7 +1057,7 @@ mlx5_flow_validate_item_ipv4(const struct rte_flow_it= em *item, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_item_ipv6(const struct rte_flow_item *item, uint64_t item_flags, struct rte_flow_error *error) @@ -2236,7 +1126,7 @@ mlx5_flow_validate_item_ipv6(const struct rte_flow_it= em *item, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_item_udp(const struct rte_flow_item *item, uint64_t item_flags, uint8_t target_protocol, @@ -2287,7 +1177,7 @@ mlx5_flow_validate_item_udp(const struct rte_flow_ite= m *item, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_item_tcp(const struct rte_flow_item *item, uint64_t item_flags, uint8_t target_protocol, @@ -2338,7 +1228,7 @@ mlx5_flow_validate_item_tcp(const struct rte_flow_ite= m *item, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_item_vxlan(const struct rte_flow_item *item, uint64_t item_flags, struct rte_flow_error *error) @@ -2417,7 +1307,7 @@ mlx5_flow_validate_item_vxlan(const struct rte_flow_i= tem *item, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_item_vxlan_gpe(const struct rte_flow_item *item, uint64_t item_flags, struct rte_eth_dev *dev, @@ -2507,7 +1397,7 @@ mlx5_flow_validate_item_vxlan_gpe(const struct rte_fl= ow_item *item, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_item_gre(const struct rte_flow_item *item, uint64_t item_flags, uint8_t target_protocol, @@ -2564,7 +1454,7 @@ mlx5_flow_validate_item_gre(const struct rte_flow_ite= m *item, * @return * 0 on success, a negative errno value otherwise and rte_errno is set. */ -static int +int mlx5_flow_validate_item_mpls(const struct rte_flow_item *item __rte_unused= , uint64_t item_flags __rte_unused, uint8_t target_protocol __rte_unused, @@ -2582,225 +1472,22 @@ mlx5_flow_validate_item_mpls(const struct rte_flow= _item *item __rte_unused, if (item_flags & MLX5_FLOW_LAYER_TUNNEL) return rte_flow_error_set(error, ENOTSUP, RTE_FLOW_ERROR_TYPE_ITEM, item, - "a tunnel is already" - " present"); - if (!mask) - mask =3D &rte_flow_item_mpls_mask; - ret =3D mlx5_flow_item_acceptable - (item, (const uint8_t *)mask, - (const uint8_t *)&rte_flow_item_mpls_mask, - sizeof(struct rte_flow_item_mpls), error); - if (ret < 0) - return ret; - return 0; -#endif - return rte_flow_error_set(error, ENOTSUP, - RTE_FLOW_ERROR_TYPE_ITEM, item, - "MPLS is not supported by Verbs, please" - " update."); -} - -/** - * Internal validation function. - * - * @param[in] dev - * Pointer to the Ethernet device structure. - * @param[in] attr - * Pointer to the flow attributes. - * @param[in] items - * Pointer to the list of items. - * @param[in] actions - * Pointer to the list of actions. - * @param[out] error - * Pointer to the error structure. - * - * @return - * 0 on success, a negative errno value otherwise and rte_ernno is set. - */ -static int mlx5_flow_verbs_validate(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_error *error) -{ - int ret; - uint32_t action_flags =3D 0; - uint32_t item_flags =3D 0; - int tunnel =3D 0; - uint8_t next_protocol =3D 0xff; - - if (items =3D=3D NULL) - return -1; - ret =3D mlx5_flow_validate_attributes(dev, attr, error); + "a tunnel is already" + " present"); + if (!mask) + mask =3D &rte_flow_item_mpls_mask; + ret =3D mlx5_flow_item_acceptable + (item, (const uint8_t *)mask, + (const uint8_t *)&rte_flow_item_mpls_mask, + sizeof(struct rte_flow_item_mpls), error); if (ret < 0) return ret; - for (; items->type !=3D RTE_FLOW_ITEM_TYPE_END; items++) { - int ret =3D 0; - switch (items->type) { - case RTE_FLOW_ITEM_TYPE_VOID: - break; - case RTE_FLOW_ITEM_TYPE_ETH: - ret =3D mlx5_flow_validate_item_eth(items, item_flags, - error); - if (ret < 0) - return ret; - item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L2 : - MLX5_FLOW_LAYER_OUTER_L2; - break; - case RTE_FLOW_ITEM_TYPE_VLAN: - ret =3D mlx5_flow_validate_item_vlan(items, item_flags, - error); - if (ret < 0) - return ret; - item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_VLAN : - MLX5_FLOW_LAYER_OUTER_VLAN; - break; - case RTE_FLOW_ITEM_TYPE_IPV4: - ret =3D mlx5_flow_validate_item_ipv4(items, item_flags, - error); - if (ret < 0) - return ret; - item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L3_IPV4 : - MLX5_FLOW_LAYER_OUTER_L3_IPV4; - if (items->mask !=3D NULL && - ((const struct rte_flow_item_ipv4 *) - items->mask)->hdr.next_proto_id) - next_protocol =3D - ((const struct rte_flow_item_ipv4 *) - (items->spec))->hdr.next_proto_id; - break; - case RTE_FLOW_ITEM_TYPE_IPV6: - ret =3D mlx5_flow_validate_item_ipv6(items, item_flags, - error); - if (ret < 0) - return ret; - item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L3_IPV6 : - MLX5_FLOW_LAYER_OUTER_L3_IPV6; - if (items->mask !=3D NULL && - ((const struct rte_flow_item_ipv6 *) - items->mask)->hdr.proto) - next_protocol =3D - ((const struct rte_flow_item_ipv6 *) - items->spec)->hdr.proto; - break; - case RTE_FLOW_ITEM_TYPE_UDP: - ret =3D mlx5_flow_validate_item_udp(items, item_flags, - next_protocol, - error); - if (ret < 0) - return ret; - item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L4_UDP : - MLX5_FLOW_LAYER_OUTER_L4_UDP; - break; - case RTE_FLOW_ITEM_TYPE_TCP: - ret =3D mlx5_flow_validate_item_tcp(items, item_flags, - next_protocol, error); - if (ret < 0) - return ret; - item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L4_TCP : - MLX5_FLOW_LAYER_OUTER_L4_TCP; - break; - case RTE_FLOW_ITEM_TYPE_VXLAN: - ret =3D mlx5_flow_validate_item_vxlan(items, item_flags, - error); - if (ret < 0) - return ret; - item_flags |=3D MLX5_FLOW_LAYER_VXLAN; - break; - case RTE_FLOW_ITEM_TYPE_VXLAN_GPE: - ret =3D mlx5_flow_validate_item_vxlan_gpe(items, - item_flags, - dev, error); - if (ret < 0) - return ret; - item_flags |=3D MLX5_FLOW_LAYER_VXLAN_GPE; - break; - case RTE_FLOW_ITEM_TYPE_GRE: - ret =3D mlx5_flow_validate_item_gre(items, item_flags, - next_protocol, error); - if (ret < 0) - return ret; - item_flags |=3D MLX5_FLOW_LAYER_GRE; - break; - case RTE_FLOW_ITEM_TYPE_MPLS: - ret =3D mlx5_flow_validate_item_mpls(items, item_flags, - next_protocol, - error); - if (ret < 0) - return ret; - if (next_protocol !=3D 0xff && - next_protocol !=3D MLX5_IP_PROTOCOL_MPLS) - return rte_flow_error_set - (error, ENOTSUP, - RTE_FLOW_ERROR_TYPE_ITEM, items, - "protocol filtering not compatible" - " with MPLS layer"); - item_flags |=3D MLX5_FLOW_LAYER_MPLS; - break; - default: - return rte_flow_error_set(error, ENOTSUP, - RTE_FLOW_ERROR_TYPE_ITEM, - NULL, - "item not supported"); - } - } - for (; actions->type !=3D RTE_FLOW_ACTION_TYPE_END; actions++) { - tunnel =3D !!(item_flags & MLX5_FLOW_LAYER_TUNNEL); - switch (actions->type) { - case RTE_FLOW_ACTION_TYPE_VOID: - break; - case RTE_FLOW_ACTION_TYPE_FLAG: - ret =3D mlx5_flow_validate_action_flag(action_flags, - error); - if (ret < 0) - return ret; - action_flags |=3D MLX5_ACTION_FLAG; - break; - case RTE_FLOW_ACTION_TYPE_MARK: - ret =3D mlx5_flow_validate_action_mark(actions, - action_flags, - error); - if (ret < 0) - return ret; - action_flags |=3D MLX5_ACTION_MARK; - break; - case RTE_FLOW_ACTION_TYPE_DROP: - ret =3D mlx5_flow_validate_action_drop(action_flags, - error); - if (ret < 0) - return ret; - action_flags |=3D MLX5_ACTION_DROP; - break; - case RTE_FLOW_ACTION_TYPE_QUEUE: - ret =3D mlx5_flow_validate_action_queue(actions, - action_flags, dev, - error); - if (ret < 0) - return ret; - action_flags |=3D MLX5_ACTION_QUEUE; - break; - case RTE_FLOW_ACTION_TYPE_RSS: - ret =3D mlx5_flow_validate_action_rss(actions, - action_flags, dev, - error); - if (ret < 0) - return ret; - action_flags |=3D MLX5_ACTION_RSS; - break; - case RTE_FLOW_ACTION_TYPE_COUNT: - ret =3D mlx5_flow_validate_action_count(dev, error); - if (ret < 0) - return ret; - action_flags |=3D MLX5_ACTION_COUNT; - break; - default: - return rte_flow_error_set(error, ENOTSUP, - RTE_FLOW_ERROR_TYPE_ACTION, - actions, - "action not supported"); - } - } return 0; +#endif + return rte_flow_error_set(error, ENOTSUP, + RTE_FLOW_ERROR_TYPE_ITEM, item, + "MPLS is not supported by Verbs, please" + " update."); } =20 /** @@ -2818,159 +1505,13 @@ mlx5_flow_validate(struct rte_eth_dev *dev, { int ret; =20 - ret =3D mlx5_flow_verbs_validate(dev, attr, items, actions, error); + ret =3D nic_ops.validate(dev, attr, items, actions, error); if (ret < 0) return ret; return 0; } =20 /** - * Calculate the required bytes that are needed for the action part of the= verbs - * flow, in addtion returns bit-fields with all the detected action, in or= der to - * avoid another interation over the actions. - * - * @param[in] actions - * Pointer to the list of actions. - * @param[out] action_flags - * Pointer to the detected actions. - * - * @return - * The size of the memory needed for all actions. - */ -static int -mlx5_flow_verbs_get_actions_and_size(const struct rte_flow_action actions[= ], - uint64_t *action_flags) -{ - int size =3D 0; - uint64_t detected_actions =3D 0; - - for (; actions->type !=3D RTE_FLOW_ACTION_TYPE_END; actions++) { - switch (actions->type) { - case RTE_FLOW_ACTION_TYPE_VOID: - break; - case RTE_FLOW_ACTION_TYPE_FLAG: - size +=3D sizeof(struct ibv_flow_spec_action_tag); - detected_actions |=3D MLX5_ACTION_FLAG; - break; - case RTE_FLOW_ACTION_TYPE_MARK: - size +=3D sizeof(struct ibv_flow_spec_action_tag); - detected_actions |=3D MLX5_ACTION_MARK; - break; - case RTE_FLOW_ACTION_TYPE_DROP: - size +=3D sizeof(struct ibv_flow_spec_action_drop); - detected_actions |=3D MLX5_ACTION_DROP; - break; - case RTE_FLOW_ACTION_TYPE_QUEUE: - detected_actions |=3D MLX5_ACTION_QUEUE; - break; - case RTE_FLOW_ACTION_TYPE_RSS: - detected_actions |=3D MLX5_ACTION_RSS; - break; - case RTE_FLOW_ACTION_TYPE_COUNT: -#ifdef HAVE_IBV_DEVICE_COUNTERS_SET_SUPPORT - size +=3D sizeof(struct ibv_flow_spec_counter_action); -#endif - detected_actions |=3D MLX5_ACTION_COUNT; - break; - default: - break; - } - } - *action_flags =3D detected_actions; - return size; -} - -/** - * Calculate the required bytes that are needed for the item part of the v= erbs - * flow, in addtion returns bit-fields with all the detected action, in or= der to - * avoid another interation over the actions. - * - * @param[in] actions - * Pointer to the list of items. - * @param[in, out] item_flags - * Pointer to the detected items. - * - * @return - * The size of the memory needed for all items. - */ -static int -mlx5_flow_verbs_get_items_and_size(const struct rte_flow_item items[], - uint64_t *item_flags) -{ - int size =3D 0; - uint64_t detected_items =3D 0; - const int tunnel =3D !!(*item_flags & MLX5_FLOW_LAYER_TUNNEL); - - for (; items->type !=3D RTE_FLOW_ITEM_TYPE_END; items++) { - switch (items->type) { - case RTE_FLOW_ITEM_TYPE_VOID: - break; - case RTE_FLOW_ITEM_TYPE_ETH: - size +=3D sizeof(struct ibv_flow_spec_eth); - detected_items |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L2 : - MLX5_FLOW_LAYER_OUTER_L2; - break; - case RTE_FLOW_ITEM_TYPE_VLAN: - size +=3D sizeof(struct ibv_flow_spec_eth); - detected_items |=3D tunnel ? MLX5_FLOW_LAYER_INNER_VLAN : - MLX5_FLOW_LAYER_OUTER_VLAN; - break; - case RTE_FLOW_ITEM_TYPE_IPV4: - size +=3D sizeof(struct ibv_flow_spec_ipv4_ext); - detected_items |=3D tunnel ? - MLX5_FLOW_LAYER_INNER_L3_IPV4 : - MLX5_FLOW_LAYER_OUTER_L3_IPV4; - break; - case RTE_FLOW_ITEM_TYPE_IPV6: - size +=3D sizeof(struct ibv_flow_spec_ipv6); - detected_items |=3D tunnel ? - MLX5_FLOW_LAYER_INNER_L3_IPV6 : - MLX5_FLOW_LAYER_OUTER_L3_IPV6; - break; - case RTE_FLOW_ITEM_TYPE_UDP: - size +=3D sizeof(struct ibv_flow_spec_tcp_udp); - detected_items |=3D tunnel ? - MLX5_FLOW_LAYER_INNER_L4_UDP : - MLX5_FLOW_LAYER_OUTER_L4_UDP; - break; - case RTE_FLOW_ITEM_TYPE_TCP: - size +=3D sizeof(struct ibv_flow_spec_tcp_udp); - detected_items |=3D tunnel ? - MLX5_FLOW_LAYER_INNER_L4_TCP : - MLX5_FLOW_LAYER_OUTER_L4_TCP; - break; - case RTE_FLOW_ITEM_TYPE_VXLAN: - size +=3D sizeof(struct ibv_flow_spec_tunnel); - detected_items |=3D MLX5_FLOW_LAYER_VXLAN; - break; - case RTE_FLOW_ITEM_TYPE_VXLAN_GPE: - size +=3D sizeof(struct ibv_flow_spec_tunnel); - detected_items |=3D MLX5_FLOW_LAYER_VXLAN_GPE; - break; -#ifdef HAVE_IBV_DEVICE_MPLS_SUPPORT - case RTE_FLOW_ITEM_TYPE_GRE: - size +=3D sizeof(struct ibv_flow_spec_gre); - detected_items |=3D MLX5_FLOW_LAYER_GRE; - break; - case RTE_FLOW_ITEM_TYPE_MPLS: - size +=3D sizeof(struct ibv_flow_spec_mpls); - detected_items |=3D MLX5_FLOW_LAYER_MPLS; - break; -#else - case RTE_FLOW_ITEM_TYPE_GRE: - size +=3D sizeof(struct ibv_flow_spec_tunnel); - detected_items |=3D MLX5_FLOW_LAYER_TUNNEL; - break; -#endif - default: - break; - } - } - *item_flags =3D detected_items; - return size; -} - -/** * Get RSS action from the action list. * * @param[in] actions @@ -2994,320 +1535,6 @@ mlx5_flow_get_rss_action(const struct rte_flow_acti= on actions[]) return NULL; } =20 -/** - * Internal preparation function. Allocate mlx5_flow with the required siz= e. - * The required size is calculate based on the actions and items. This fun= ction - * also returns the detected actions and items for later use. - * - * @param[in] attr - * Pointer to the flow attributes. - * @param[in] items - * Pointer to the list of items. - * @param[in] actions - * Pointer to the list of actions. - * @param[out] item_flags - * Pointer to bit mask of all items detected. - * @param[out] action_flags - * Pointer to bit mask of all actions detected. - * @param[out] error - * Pointer to the error structure. - * - * @return - * Pointer to mlx5_flow object on success, otherwise NULL and rte_errno - * is set. - */ -static struct mlx5_flow * -mlx5_flow_verbs_prepare(const struct rte_flow_attr *attr __rte_unused, - const struct rte_flow_item items[], - const struct rte_flow_action actions[], - uint64_t *item_flags, - uint64_t *action_flags, - struct rte_flow_error *error) -{ - uint32_t size =3D sizeof(struct mlx5_flow) + sizeof(struct ibv_flow_attr)= ; - struct mlx5_flow *flow; - - size +=3D mlx5_flow_verbs_get_actions_and_size(actions, action_flags); - size +=3D mlx5_flow_verbs_get_items_and_size(items, item_flags); - flow =3D rte_calloc(__func__, 1, size, 0); - if (!flow) { - rte_flow_error_set(error, ENOMEM, - RTE_FLOW_ERROR_TYPE_UNSPECIFIED, - NULL, - "not enough memory to create flow"); - return NULL; - } - flow->verbs.attr =3D (void *)(flow + 1); - flow->verbs.specs =3D (uint8_t *)(flow + 1) + - sizeof(struct ibv_flow_attr); - return flow; -} - -/** - * - * Fill the flow with verb spec. - * - * @param[in] dev - * Pointer to Ethernet device. - * @param[in, out] dev_flow - * Pointer to the mlx5 flow. - * @param[in] attr - * Pointer to the flow attributes. - * @param[in] items - * Pointer to the list of items. - * @param[in] actions - * Pointer to the list of actions. - * @param[out] error - * Pointer to the error structure. - * - * @return - * 0 on success, else a negative errno value otherwise and rte_ernno is = set. - */ -static int mlx5_flow_verbs_translate(struct rte_eth_dev *dev, - struct mlx5_flow *dev_flow, - const struct rte_flow_attr *attr, - const struct rte_flow_item items[], - const struct rte_flow_action actions[], - struct rte_flow_error *error) -{ - uint64_t action_flags =3D 0; - uint64_t item_flags =3D 0; - uint64_t priority =3D attr->priority; - struct priv *priv =3D dev->data->dev_private; - - if (priority =3D=3D MLX5_FLOW_PRIO_RSVD) - priority =3D priv->config.flow_prio - 1; - for (; actions->type !=3D RTE_FLOW_ACTION_TYPE_END; actions++) { - int ret; - switch (actions->type) { - case RTE_FLOW_ACTION_TYPE_VOID: - break; - case RTE_FLOW_ACTION_TYPE_FLAG: - flow_verbs_translate_action_flag(actions, - &action_flags, - dev_flow); - break; - case RTE_FLOW_ACTION_TYPE_MARK: - flow_verbs_translate_action_mark(actions, - &action_flags, - dev_flow); - break; - case RTE_FLOW_ACTION_TYPE_DROP: - flow_verbs_translate_action_drop(&action_flags, - dev_flow); - break; - case RTE_FLOW_ACTION_TYPE_QUEUE: - flow_verbs_translate_action_queue(actions, - &action_flags, - dev_flow); - break; - case RTE_FLOW_ACTION_TYPE_RSS: - flow_verbs_translate_action_rss(actions, - &action_flags, - dev_flow); - break; - case RTE_FLOW_ACTION_TYPE_COUNT: - ret =3D flow_verbs_translate_action_count(dev, - actions, - &action_flags, - dev_flow, - error); - if (ret < 0) - return ret; - break; - default: - return rte_flow_error_set(error, ENOTSUP, - RTE_FLOW_ERROR_TYPE_ACTION, - actions, - "action not supported"); - } - } - dev_flow->flow->actions |=3D action_flags; - for (; items->type !=3D RTE_FLOW_ITEM_TYPE_END; items++) { - switch (items->type) { - case RTE_FLOW_ITEM_TYPE_VOID: - break; - case RTE_FLOW_ITEM_TYPE_ETH: - flow_verbs_translate_item_eth(items, &item_flags, - dev_flow); - break; - case RTE_FLOW_ITEM_TYPE_VLAN: - flow_verbs_translate_item_vlan(items, &item_flags, - dev_flow); - break; - case RTE_FLOW_ITEM_TYPE_IPV4: - flow_verbs_translate_item_ipv4(items, &item_flags, - dev_flow); - break; - case RTE_FLOW_ITEM_TYPE_IPV6: - flow_verbs_translate_item_ipv6(items, &item_flags, - dev_flow); - break; - case RTE_FLOW_ITEM_TYPE_UDP: - flow_verbs_translate_item_udp(items, &item_flags, - dev_flow); - break; - case RTE_FLOW_ITEM_TYPE_TCP: - flow_verbs_translate_item_tcp(items, &item_flags, - dev_flow); - break; - case RTE_FLOW_ITEM_TYPE_VXLAN: - flow_verbs_translate_item_vxlan(items, &item_flags, - dev_flow); - break; - case RTE_FLOW_ITEM_TYPE_VXLAN_GPE: - flow_verbs_translate_item_vxlan_gpe(items, &item_flags, - dev_flow); - break; - case RTE_FLOW_ITEM_TYPE_GRE: - flow_verbs_translate_item_gre(items, &item_flags, - dev_flow); - break; - case RTE_FLOW_ITEM_TYPE_MPLS: - flow_verbs_translate_item_mpls(items, &item_flags, - dev_flow); - break; - default: - return rte_flow_error_set(error, ENOTSUP, - RTE_FLOW_ERROR_TYPE_ITEM, - NULL, - "item not supported"); - } - } - dev_flow->verbs.attr->priority =3D mlx5_flow_adjust_priority(dev, - priority, - dev_flow->verbs.attr->priority); - return 0; -} - -/** - * Remove the flow. - * - * @param[in] dev - * Pointer to the Ethernet device structure. - * @param[in, out] flow - * Pointer to flow structure. - */ -static void -mlx5_flow_remove(struct rte_eth_dev *dev, struct rte_flow *flow) -{ - struct priv *priv =3D dev->data->dev_private; - struct mlx5_flow_verbs *verbs; - struct mlx5_flow *dev_flow; - - if (flow->nl_flow && priv->mnl_socket) - mlx5_nl_flow_destroy(priv->mnl_socket, flow->nl_flow, NULL); - LIST_FOREACH(dev_flow, &flow->dev_flows, next) { - verbs =3D &dev_flow->verbs; - if (verbs->flow) { - claim_zero(mlx5_glue->destroy_flow(verbs->flow)); - verbs->flow =3D NULL; - } - if (verbs->hrxq) { - if (flow->actions & MLX5_ACTION_DROP) - mlx5_hrxq_drop_release(dev); - else - mlx5_hrxq_release(dev, verbs->hrxq); - verbs->hrxq =3D NULL; - } - } - if (flow->counter) { - mlx5_flow_counter_release(flow->counter); - flow->counter =3D NULL; - } -} - -/** - * Apply the flow. - * - * @param[in] dev - * Pointer to the Ethernet device structure. - * @param[in, out] flow - * Pointer to flow structure. - * @param[out] error - * Pointer to error structure. - * - * @return - * 0 on success, a negative errno value otherwise and rte_errno is set. - */ -static int -mlx5_flow_apply(struct rte_eth_dev *dev, struct rte_flow *flow, - struct rte_flow_error *error) -{ - struct priv *priv =3D dev->data->dev_private; - struct mlx5_flow_verbs *verbs; - struct mlx5_flow *dev_flow; - int err; - - LIST_FOREACH(dev_flow, &flow->dev_flows, next) { - verbs =3D &dev_flow->verbs; - if (flow->actions & MLX5_ACTION_DROP) { - verbs->hrxq =3D mlx5_hrxq_drop_new(dev); - if (!verbs->hrxq) { - rte_flow_error_set - (error, errno, - RTE_FLOW_ERROR_TYPE_UNSPECIFIED, - NULL, - "cannot get drop hash queue"); - goto error; - } - } else { - struct mlx5_hrxq *hrxq; - - hrxq =3D mlx5_hrxq_get(dev, flow->key, - MLX5_RSS_HASH_KEY_LEN, - verbs->hash_fields, - (*flow->queue), - flow->rss.queue_num); - if (!hrxq) - hrxq =3D mlx5_hrxq_new(dev, flow->key, - MLX5_RSS_HASH_KEY_LEN, - verbs->hash_fields, - (*flow->queue), - flow->rss.queue_num, - !!(flow->layers & - MLX5_FLOW_LAYER_TUNNEL)); - if (!hrxq) { - rte_flow_error_set - (error, rte_errno, - RTE_FLOW_ERROR_TYPE_UNSPECIFIED, - NULL, - "cannot get hash queue"); - goto error; - } - verbs->hrxq =3D hrxq; - } - verbs->flow =3D mlx5_glue->create_flow(verbs->hrxq->qp, - verbs->attr); - if (!verbs->flow) { - rte_flow_error_set(error, errno, - RTE_FLOW_ERROR_TYPE_UNSPECIFIED, - NULL, - "hardware refuses to create flow"); - goto error; - } - } - if (flow->nl_flow && - priv->mnl_socket && - mlx5_nl_flow_create(priv->mnl_socket, flow->nl_flow, error)) - goto error; - return 0; -error: - err =3D rte_errno; /* Save rte_errno before cleanup. */ - LIST_FOREACH(dev_flow, &flow->dev_flows, next) { - verbs =3D &dev_flow->verbs; - if (verbs->hrxq) { - if (flow->actions & MLX5_ACTION_DROP) - mlx5_hrxq_drop_release(dev); - else - mlx5_hrxq_release(dev, verbs->hrxq); - verbs->hrxq =3D NULL; - } - } - rte_errno =3D err; /* Restore rte_errno. */ - return -rte_errno; -} - static unsigned int mlx5_find_graph_root(const struct rte_flow_item pattern[], uint32_t rss_le= vel) { @@ -3396,31 +1623,34 @@ mlx5_flow_list_create(struct rte_eth_dev *dev, buf->entry[0].pattern =3D (void *)(uintptr_t)items; } for (i =3D 0; i < buf->entries; ++i) { - dev_flow =3D mlx5_flow_verbs_prepare(attr, buf->entry[i].pattern, - actions, &item_flags, - &action_flags, error); + dev_flow =3D nic_ops.prepare(attr, buf->entry[i].pattern, + actions, &item_flags, + &action_flags, error); + if (!dev_flow) + goto error; dev_flow->flow =3D flow; LIST_INSERT_HEAD(&flow->dev_flows, dev_flow, next); - mlx5_flow_verbs_translate(dev, dev_flow, attr, - buf->entry[i].pattern, - actions, - error); + ret =3D nic_ops.translate(dev, dev_flow, attr, + buf->entry[i].pattern, + actions, error); + if (ret < 0) + goto error; } if (dev->data->dev_started) { - ret =3D mlx5_flow_apply(dev, flow, error); - if (ret < 0) { - ret =3D rte_errno; /* Save rte_errno before cleanup. */ - if (flow) { - mlx5_flow_remove(dev, flow); - rte_free(flow); - } - rte_errno =3D ret; /* Restore rte_errno. */ - return NULL; - } + ret =3D nic_ops.apply(dev, flow, error); + if (ret < 0) + goto error; } TAILQ_INSERT_TAIL(list, flow, next); mlx5_flow_rxq_flags_set(dev, flow); return flow; +error: + ret =3D rte_errno; /* Save rte_errno before cleanup. */ + assert(flow); + nic_ops.destroy(dev, flow); + rte_free(flow); + rte_errno =3D ret; /* Restore rte_errno. */ + return NULL; } =20 /** @@ -3455,7 +1685,7 @@ static void mlx5_flow_list_destroy(struct rte_eth_dev *dev, struct mlx5_flows *list, struct rte_flow *flow) { - mlx5_flow_remove(dev, flow); + nic_ops.destroy(dev, flow); TAILQ_REMOVE(list, flow, next); /* * Update RX queue flags only if port is started, otherwise it is @@ -3463,12 +1693,6 @@ mlx5_flow_list_destroy(struct rte_eth_dev *dev, stru= ct mlx5_flows *list, */ if (dev->data->dev_started) mlx5_flow_rxq_flags_trim(dev, flow); - while (!LIST_EMPTY(&flow->dev_flows)) { - struct mlx5_flow *dev_flow; - dev_flow =3D LIST_FIRST(&flow->dev_flows); - LIST_REMOVE(dev_flow, next); - rte_free(dev_flow); - } rte_free(flow); } =20 @@ -3505,7 +1729,7 @@ mlx5_flow_stop(struct rte_eth_dev *dev, struct mlx5_f= lows *list) struct rte_flow *flow; =20 TAILQ_FOREACH_REVERSE(flow, list, mlx5_flows, next) - mlx5_flow_remove(dev, flow); + nic_ops.remove(dev, flow); mlx5_flow_rxq_flags_clear(dev); } =20 @@ -3528,7 +1752,7 @@ mlx5_flow_start(struct rte_eth_dev *dev, struct mlx5_= flows *list) int ret =3D 0; =20 TAILQ_FOREACH(flow, list, next) { - ret =3D mlx5_flow_apply(dev, flow, &error); + ret =3D nic_ops.apply(dev, flow, &error); if (ret < 0) goto error; mlx5_flow_rxq_flags_set(dev, flow); @@ -4219,3 +2443,15 @@ mlx5_dev_filter_ctrl(struct rte_eth_dev *dev, } return 0; } + +/** + * Init the driver ops structure. + * + * @param dev + * Pointer to Ethernet device structure. + */ +void +mlx5_flow_init_driver_ops(struct rte_eth_dev *dev __rte_unused) +{ + mlx5_flow_verbs_get_driver_ops(&nic_ops); +} diff --git a/drivers/net/mlx5/mlx5_flow.h b/drivers/net/mlx5/mlx5_flow.h new file mode 100644 index 000000000..4df60db92 --- /dev/null +++ b/drivers/net/mlx5/mlx5_flow.h @@ -0,0 +1,257 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright 2018 Mellanox Technologies, Ltd + */ + +#ifndef RTE_PMD_MLX5_FLOW_H_ +#define RTE_PMD_MLX5_FLOW_H_ + +#include +#include +#include +#include + +/* Verbs header. */ +/* ISO C doesn't support unnamed structs/unions, disabling -pedantic. */ +#ifdef PEDANTIC +#pragma GCC diagnostic ignored "-Wpedantic" +#endif +#include +#ifdef PEDANTIC +#pragma GCC diagnostic error "-Wpedantic" +#endif + +/* Pattern outer Layer bits. */ +#define MLX5_FLOW_LAYER_OUTER_L2 (1u << 0) +#define MLX5_FLOW_LAYER_OUTER_L3_IPV4 (1u << 1) +#define MLX5_FLOW_LAYER_OUTER_L3_IPV6 (1u << 2) +#define MLX5_FLOW_LAYER_OUTER_L4_UDP (1u << 3) +#define MLX5_FLOW_LAYER_OUTER_L4_TCP (1u << 4) +#define MLX5_FLOW_LAYER_OUTER_VLAN (1u << 5) + +/* Pattern inner Layer bits. */ +#define MLX5_FLOW_LAYER_INNER_L2 (1u << 6) +#define MLX5_FLOW_LAYER_INNER_L3_IPV4 (1u << 7) +#define MLX5_FLOW_LAYER_INNER_L3_IPV6 (1u << 8) +#define MLX5_FLOW_LAYER_INNER_L4_UDP (1u << 9) +#define MLX5_FLOW_LAYER_INNER_L4_TCP (1u << 10) +#define MLX5_FLOW_LAYER_INNER_VLAN (1u << 11) + +/* Pattern tunnel Layer bits. */ +#define MLX5_FLOW_LAYER_VXLAN (1u << 12) +#define MLX5_FLOW_LAYER_VXLAN_GPE (1u << 13) +#define MLX5_FLOW_LAYER_GRE (1u << 14) +#define MLX5_FLOW_LAYER_MPLS (1u << 15) + +/* Outer Masks. */ +#define MLX5_FLOW_LAYER_OUTER_L3 \ + (MLX5_FLOW_LAYER_OUTER_L3_IPV4 | MLX5_FLOW_LAYER_OUTER_L3_IPV6) +#define MLX5_FLOW_LAYER_OUTER_L4 \ + (MLX5_FLOW_LAYER_OUTER_L4_UDP | MLX5_FLOW_LAYER_OUTER_L4_TCP) +#define MLX5_FLOW_LAYER_OUTER \ + (MLX5_FLOW_LAYER_OUTER_L2 | MLX5_FLOW_LAYER_OUTER_L3 | \ + MLX5_FLOW_LAYER_OUTER_L4) + +/* Tunnel Masks. */ +#define MLX5_FLOW_LAYER_TUNNEL \ + (MLX5_FLOW_LAYER_VXLAN | MLX5_FLOW_LAYER_VXLAN_GPE | \ + MLX5_FLOW_LAYER_GRE | MLX5_FLOW_LAYER_MPLS) + +/* Inner Masks. */ +#define MLX5_FLOW_LAYER_INNER_L3 \ + (MLX5_FLOW_LAYER_INNER_L3_IPV4 | MLX5_FLOW_LAYER_INNER_L3_IPV6) +#define MLX5_FLOW_LAYER_INNER_L4 \ + (MLX5_FLOW_LAYER_INNER_L4_UDP | MLX5_FLOW_LAYER_INNER_L4_TCP) +#define MLX5_FLOW_LAYER_INNER \ + (MLX5_FLOW_LAYER_INNER_L2 | MLX5_FLOW_LAYER_INNER_L3 | \ + MLX5_FLOW_LAYER_INNER_L4) + +/* Actions that modify the fate of matching traffic. */ +#define MLX5_FLOW_FATE_DROP (1u << 0) +#define MLX5_FLOW_FATE_QUEUE (1u << 1) +#define MLX5_FLOW_FATE_RSS (1u << 2) + +/* Modify a packet. */ +#define MLX5_FLOW_MOD_FLAG (1u << 0) +#define MLX5_FLOW_MOD_MARK (1u << 1) +#define MLX5_FLOW_MOD_COUNT (1u << 2) + +/* Actions */ +#define MLX5_ACTION_DROP (1u << 0) +#define MLX5_ACTION_QUEUE (1u << 1) +#define MLX5_ACTION_RSS (1u << 2) +#define MLX5_ACTION_FLAG (1u << 3) +#define MLX5_ACTION_MARK (1u << 4) +#define MLX5_ACTION_COUNT (1u << 5) + +/* possible L3 layers protocols filtering. */ +#define MLX5_IP_PROTOCOL_TCP 6 +#define MLX5_IP_PROTOCOL_UDP 17 +#define MLX5_IP_PROTOCOL_GRE 47 +#define MLX5_IP_PROTOCOL_MPLS 147 + +/* Priority reserved for default flows. */ +#define MLX5_FLOW_PRIO_RSVD ((uint32_t)-1) + +/* + * Number of sub priorities. + * For each kind of pattern matching i.e. L2, L3, L4 to have a correct + * matching on the NIC (firmware dependent) L4 most have the higher priori= ty + * followed by L3 and ending with L2. + */ +#define MLX5_PRIORITY_MAP_L2 2 +#define MLX5_PRIORITY_MAP_L3 1 +#define MLX5_PRIORITY_MAP_L4 0 +#define MLX5_PRIORITY_MAP_MAX 3 + +/* Verbs specification header. */ +struct ibv_spec_header { + enum ibv_flow_spec_type type; + uint16_t size; +}; + +/** Handles information leading to a drop fate. */ +struct mlx5_flow_verbs { + LIST_ENTRY(mlx5_flow_verbs) next; + unsigned int size; /**< Size of the attribute. */ + struct { + struct ibv_flow_attr *attr; + /**< Pointer to the Specification buffer. */ + uint8_t *specs; /**< Pointer to the specifications. */ + }; + struct ibv_flow *flow; /**< Verbs flow pointer. */ + struct mlx5_hrxq *hrxq; /**< Hash Rx queue object. */ + uint64_t hash_fields; /**< Verbs hash Rx queue hash fields. */ +}; + +/** Device flow structure. */ +struct mlx5_flow { + LIST_ENTRY(mlx5_flow) next; + struct rte_flow *flow; /**< Pointer to the main flow. */ + uint32_t layers; /**< Bit-fields that holds the detected layers. */ + union { + struct mlx5_flow_verbs verbs; /**< Holds the verbs dev-flow. */ + }; +}; + +/* Counters information. */ +struct mlx5_flow_counter { + LIST_ENTRY(mlx5_flow_counter) next; /**< Pointer to the next counter. */ + uint32_t shared:1; /**< Share counter ID with other flow rules. */ + uint32_t ref_cnt:31; /**< Reference counter. */ + uint32_t id; /**< Counter ID. */ + struct ibv_counter_set *cs; /**< Holds the counters for the rule. */ + uint64_t hits; /**< Number of packets matched by the rule. */ + uint64_t bytes; /**< Number of bytes matched by the rule. */ +}; + +/* Flow structure. */ +struct rte_flow { + TAILQ_ENTRY(rte_flow) next; /**< Pointer to the next flow structure. */ + struct rte_flow_attr attributes; /**< User flow attribute. */ + uint32_t layers; + /**< Bit-fields of present layers see MLX5_FLOW_LAYER_*. */ + struct mlx5_flow_counter *counter; /**< Holds flow counter. */ + struct rte_flow_action_rss rss;/**< RSS context. */ + uint8_t key[MLX5_RSS_HASH_KEY_LEN]; /**< RSS hash key. */ + uint16_t (*queue)[]; /**< Destination queues to redirect traffic to. */ + void *nl_flow; /**< Netlink flow buffer if relevant. */ + LIST_HEAD(dev_flows, mlx5_flow) dev_flows; + /**< Device flows that are part of the flow. */ + uint32_t actions; /**< Bit-fields which mark all detected actions. */ +}; +typedef int (*mlx5_flow_validate_t)(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_error *error); +typedef struct mlx5_flow *(*mlx5_flow_prepare_t) + (const struct rte_flow_attr *attr, const struct rte_flow_item items[], + const struct rte_flow_action actions[], uint64_t *item_flags, + uint64_t *action_flags, struct rte_flow_error *error); +typedef int (*mlx5_flow_translate_t)(struct rte_eth_dev *dev, + struct mlx5_flow *dev_flow, + const struct rte_flow_attr *attr, + const struct rte_flow_item items[], + const struct rte_flow_action actions[], + struct rte_flow_error *error); +typedef int (*mlx5_flow_apply_t)(struct rte_eth_dev *dev, struct rte_flow = *flow, + struct rte_flow_error *error); +typedef void (*mlx5_flow_remove_t)(struct rte_eth_dev *dev, + struct rte_flow *flow); +typedef void (*mlx5_flow_destroy_t)(struct rte_eth_dev *dev, + struct rte_flow *flow); +struct mlx5_flow_driver_ops { + mlx5_flow_validate_t validate; + mlx5_flow_prepare_t prepare; + mlx5_flow_translate_t translate; + mlx5_flow_apply_t apply; + mlx5_flow_remove_t remove; + mlx5_flow_destroy_t destroy; +}; + +/* mlx5_flow.c */ + +uint32_t mlx5_flow_adjust_priority(struct rte_eth_dev *dev, int32_t priori= ty, + uint32_t subpriority); +int mlx5_flow_validate_action_count(struct rte_eth_dev *dev, + struct rte_flow_error *error); +int mlx5_flow_validate_action_drop(uint64_t action_flags, + struct rte_flow_error *error); +int mlx5_flow_validate_action_flag(uint64_t action_flags, + struct rte_flow_error *error); +int mlx5_flow_validate_action_mark(const struct rte_flow_action *action, + uint64_t action_flags, + struct rte_flow_error *error); +int mlx5_flow_validate_action_queue(const struct rte_flow_action *action, + uint64_t action_flags, + struct rte_eth_dev *dev, + struct rte_flow_error *error); +int mlx5_flow_validate_action_rss(const struct rte_flow_action *action, + uint64_t action_flags, + struct rte_eth_dev *dev, + struct rte_flow_error *error); +int mlx5_flow_validate_attributes(struct rte_eth_dev *dev, + const struct rte_flow_attr *attributes, + struct rte_flow_error *error); +int mlx5_flow_validate_item_eth(const struct rte_flow_item *item, + uint64_t item_flags, + struct rte_flow_error *error); +int mlx5_flow_validate_item_gre(const struct rte_flow_item *item, + uint64_t item_flags, + uint8_t target_protocol, + struct rte_flow_error *error); +int mlx5_flow_validate_item_ipv4(const struct rte_flow_item *item, + int64_t item_flags, + struct rte_flow_error *error); +int mlx5_flow_validate_item_ipv6(const struct rte_flow_item *item, + uint64_t item_flags, + struct rte_flow_error *error); +int mlx5_flow_validate_item_mpls(const struct rte_flow_item *item, + uint64_t item_flags, + uint8_t target_protocol, + struct rte_flow_error *error); +int mlx5_flow_validate_item_tcp(const struct rte_flow_item *item, + uint64_t item_flags, + uint8_t target_protocol, + struct rte_flow_error *error); +int mlx5_flow_validate_item_udp(const struct rte_flow_item *item, + uint64_t item_flags, + uint8_t target_protocol, + struct rte_flow_error *error); +int mlx5_flow_validate_item_vlan(const struct rte_flow_item *item, + int64_t item_flags, + struct rte_flow_error *error); +int mlx5_flow_validate_item_vxlan(const struct rte_flow_item *item, + uint64_t item_flags, + struct rte_flow_error *error); +int mlx5_flow_validate_item_vxlan_gpe(const struct rte_flow_item *item, + uint64_t item_flags, + struct rte_eth_dev *dev, + struct rte_flow_error *error); +void mlx5_flow_init_driver_ops(struct rte_eth_dev *dev); + +/* mlx5_flow_verbs.c */ + +void mlx5_flow_verbs_get_driver_ops(struct mlx5_flow_driver_ops *flow_ops)= ; + +#endif /* RTE_PMD_MLX5_FLOW_H_ */ diff --git a/drivers/net/mlx5/mlx5_flow_verbs.c b/drivers/net/mlx5/mlx5_flo= w_verbs.c new file mode 100644 index 000000000..e8e16cc37 --- /dev/null +++ b/drivers/net/mlx5/mlx5_flow_verbs.c @@ -0,0 +1,1692 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright 2018 Mellanox Technologies, Ltd + */ + +#include +#include +#include +#include + +/* Verbs header. */ +/* ISO C doesn't support unnamed structs/unions, disabling -pedantic. */ +#ifdef PEDANTIC +#pragma GCC diagnostic ignored "-Wpedantic" +#endif +#include +#ifdef PEDANTIC +#pragma GCC diagnostic error "-Wpedantic" +#endif + +#include +#include +#include +#include +#include +#include +#include +#include + +#include "mlx5.h" +#include "mlx5_defs.h" +#include "mlx5_prm.h" +#include "mlx5_glue.h" +#include "mlx5_flow.h" + +/** + * Get a flow counter. + * + * @param[in] dev + * Pointer to the Ethernet device structure. + * @param[in] shared + * Indicate if this counter is shared with other flows. + * @param[in] id + * Counter identifier. + * + * @return + * A pointer to the counter, NULL otherwise and rte_errno is set. + */ +static struct mlx5_flow_counter * +flow_verbs_counter_new(struct rte_eth_dev *dev, uint32_t shared, uint32_t = id) +{ + struct priv *priv =3D dev->data->dev_private; + struct mlx5_flow_counter *cnt; + + LIST_FOREACH(cnt, &priv->flow_counters, next) { + if (!cnt->shared || cnt->shared !=3D shared) + continue; + if (cnt->id !=3D id) + continue; + cnt->ref_cnt++; + return cnt; + } +#ifdef HAVE_IBV_DEVICE_COUNTERS_SET_SUPPORT + + struct mlx5_flow_counter tmpl =3D { + .shared =3D shared, + .id =3D id, + .cs =3D mlx5_glue->create_counter_set + (priv->ctx, + &(struct ibv_counter_set_init_attr){ + .counter_set_id =3D id, + }), + .hits =3D 0, + .bytes =3D 0, + }; + + if (!tmpl.cs) { + rte_errno =3D errno; + return NULL; + } + cnt =3D rte_calloc(__func__, 1, sizeof(*cnt), 0); + if (!cnt) { + rte_errno =3D ENOMEM; + return NULL; + } + *cnt =3D tmpl; + LIST_INSERT_HEAD(&priv->flow_counters, cnt, next); + return cnt; +#endif + rte_errno =3D ENOTSUP; + return NULL; +} + +/** + * Release a flow counter. + * + * @param[in] counter + * Pointer to the counter handler. + */ +static void +flow_verbs_counter_release(struct mlx5_flow_counter *counter) +{ + if (--counter->ref_cnt =3D=3D 0) { + claim_zero(mlx5_glue->destroy_counter_set(counter->cs)); + LIST_REMOVE(counter, next); + rte_free(counter); + } +} + +/** + * Add a verbs item specification into @p flow. + * + * @param[in, out] flow + * Pointer to flow structure. + * @param[in] src + * Create specification. + * @param[in] size + * Size in bytes of the specification to copy. + */ +static void +flow_verbs_spec_add(struct mlx5_flow *flow, void *src, unsigned int size) +{ + struct mlx5_flow_verbs *verbs =3D &flow->verbs; + + if (verbs->specs) { + void *dst; + + dst =3D (void *)(verbs->specs + verbs->size); + memcpy(dst, src, size); + ++verbs->attr->num_of_specs; + } + verbs->size +=3D size; +} + +/** + * Adjust verbs hash fields according to the @p flow information. + * + * @param[in] dev_flow. + * Pointer to dev flow structure. + * @param[in] tunnel + * 1 when the hash field is for a tunnel item. + * @param[in] layer_types + * ETH_RSS_* types. + * @param[in] hash_fields + * Item hash fields. + */ +static void +flow_verbs_hashfields_adjust(struct mlx5_flow *dev_flow, + int tunnel __rte_unused, + uint32_t layer_types, uint64_t hash_fields) +{ +#ifdef HAVE_IBV_DEVICE_TUNNEL_SUPPORT + int rss_request_inner =3D dev_flow->flow->rss.level >=3D 2; + + hash_fields |=3D (tunnel ? IBV_RX_HASH_INNER : 0); + if (rss_request_inner && !tunnel) + hash_fields =3D 0; + else if (rss_request_inner < 2 && tunnel) + hash_fields =3D 0; +#endif + if (!(dev_flow->flow->rss.types & layer_types)) + hash_fields =3D 0; + dev_flow->verbs.hash_fields |=3D hash_fields; +} + +/** + * Convert the @p item into a Verbs specification. This function assumes t= hat + * the input is valid and that there is space to insert the requested item + * into the flow. + * + * @param[in] item + * Item specification. + * @param[in] item_flags + * Bit field with all detected items. + * @param[in, out] dev_flow + * Pointer to dev_flow structure. + */ +static void +flow_verbs_translate_item_eth(const struct rte_flow_item *item, + uint64_t *item_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_item_eth *spec =3D item->spec; + const struct rte_flow_item_eth *mask =3D item->mask; + const int tunnel =3D !!(*item_flags & MLX5_FLOW_LAYER_TUNNEL); + const unsigned int size =3D sizeof(struct ibv_flow_spec_eth); + struct ibv_flow_spec_eth eth =3D { + .type =3D IBV_FLOW_SPEC_ETH | (tunnel ? IBV_FLOW_SPEC_INNER : 0), + .size =3D size, + }; + + if (!mask) + mask =3D &rte_flow_item_eth_mask; + if (spec) { + unsigned int i; + + memcpy(ð.val.dst_mac, spec->dst.addr_bytes, ETHER_ADDR_LEN); + memcpy(ð.val.src_mac, spec->src.addr_bytes, ETHER_ADDR_LEN); + eth.val.ether_type =3D spec->type; + memcpy(ð.mask.dst_mac, mask->dst.addr_bytes, ETHER_ADDR_LEN); + memcpy(ð.mask.src_mac, mask->src.addr_bytes, ETHER_ADDR_LEN); + eth.mask.ether_type =3D mask->type; + /* Remove unwanted bits from values. */ + for (i =3D 0; i < ETHER_ADDR_LEN; ++i) { + eth.val.dst_mac[i] &=3D eth.mask.dst_mac[i]; + eth.val.src_mac[i] &=3D eth.mask.src_mac[i]; + } + eth.val.ether_type &=3D eth.mask.ether_type; + dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L2; + } + flow_verbs_spec_add(dev_flow, ð, size); + *item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L2 : + MLX5_FLOW_LAYER_OUTER_L2; +} + +/** + * Update the VLAN tag in the Verbs Ethernet specification. + * This function assumes that the input is valid and there is space to add + * the requested item. + * + * @param[in, out] attr + * Pointer to Verbs attributes structure. + * @param[in] eth + * Verbs structure containing the VLAN information to copy. + */ +static void +flow_verbs_item_vlan_update(struct ibv_flow_attr *attr, + struct ibv_flow_spec_eth *eth) +{ + unsigned int i; + const enum ibv_flow_spec_type search =3D eth->type; + struct ibv_spec_header *hdr =3D (struct ibv_spec_header *) + ((uint8_t *)attr + sizeof(struct ibv_flow_attr)); + + for (i =3D 0; i !=3D attr->num_of_specs; ++i) { + if (hdr->type =3D=3D search) { + struct ibv_flow_spec_eth *e =3D + (struct ibv_flow_spec_eth *)hdr; + + e->val.vlan_tag =3D eth->val.vlan_tag; + e->mask.vlan_tag =3D eth->mask.vlan_tag; + e->val.ether_type =3D eth->val.ether_type; + e->mask.ether_type =3D eth->mask.ether_type; + break; + } + hdr =3D (struct ibv_spec_header *)((uint8_t *)hdr + hdr->size); + } +} + +/** + * Convert the @p item into a Verbs specification. This function assumes t= hat + * the input is valid and that there is space to insert the requested item + * into the flow. + * + * @param[in] item + * Item specification. + * @param[in, out] item_flags + * Bit mask that holds all detected items. + * @param[in, out] dev_flow + * Pointer to dev_flow structure. + */ +static void +flow_verbs_translate_item_vlan(const struct rte_flow_item *item, + uint64_t *item_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_item_vlan *spec =3D item->spec; + const struct rte_flow_item_vlan *mask =3D item->mask; + unsigned int size =3D sizeof(struct ibv_flow_spec_eth); + const int tunnel =3D !!(*item_flags & MLX5_FLOW_LAYER_TUNNEL); + struct ibv_flow_spec_eth eth =3D { + .type =3D IBV_FLOW_SPEC_ETH | (tunnel ? IBV_FLOW_SPEC_INNER : 0), + .size =3D size, + }; + const uint32_t l2m =3D tunnel ? MLX5_FLOW_LAYER_INNER_L2 : + MLX5_FLOW_LAYER_OUTER_L2; + + if (!mask) + mask =3D &rte_flow_item_vlan_mask; + if (spec) { + eth.val.vlan_tag =3D spec->tci; + eth.mask.vlan_tag =3D mask->tci; + eth.val.vlan_tag &=3D eth.mask.vlan_tag; + eth.val.ether_type =3D spec->inner_type; + eth.mask.ether_type =3D mask->inner_type; + eth.val.ether_type &=3D eth.mask.ether_type; + } + if (!(*item_flags & l2m)) { + dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L2; + flow_verbs_spec_add(dev_flow, ð, size); + } else { + flow_verbs_item_vlan_update(dev_flow->verbs.attr, ð); + size =3D 0; /* Only an update is done in eth specification. */ + } + *item_flags |=3D tunnel ? + (MLX5_FLOW_LAYER_INNER_L2 | MLX5_FLOW_LAYER_INNER_VLAN) : + (MLX5_FLOW_LAYER_OUTER_L2 | MLX5_FLOW_LAYER_OUTER_VLAN); +} + +/** + * Convert the @p item into a Verbs specification. This function assumes t= hat + * the input is valid and that there is space to insert the requested item + * into the flow. + * + * @param[in] item + * Item specification. + * @param[in, out] item_flags + * Bit mask that marks all detected items. + * @param[in, out] dev_flow + * Pointer to sepacific flow structure. + */ +static void +flow_verbs_translate_item_ipv4(const struct rte_flow_item *item, + uint64_t *item_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_item_ipv4 *spec =3D item->spec; + const struct rte_flow_item_ipv4 *mask =3D item->mask; + const int tunnel =3D !!(*item_flags & MLX5_FLOW_LAYER_TUNNEL); + unsigned int size =3D sizeof(struct ibv_flow_spec_ipv4_ext); + struct ibv_flow_spec_ipv4_ext ipv4 =3D { + .type =3D IBV_FLOW_SPEC_IPV4_EXT | + (tunnel ? IBV_FLOW_SPEC_INNER : 0), + .size =3D size, + }; + + if (!mask) + mask =3D &rte_flow_item_ipv4_mask; + *item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L3_IPV4 : + MLX5_FLOW_LAYER_OUTER_L3_IPV4; + if (spec) { + ipv4.val =3D (struct ibv_flow_ipv4_ext_filter){ + .src_ip =3D spec->hdr.src_addr, + .dst_ip =3D spec->hdr.dst_addr, + .proto =3D spec->hdr.next_proto_id, + .tos =3D spec->hdr.type_of_service, + }; + ipv4.mask =3D (struct ibv_flow_ipv4_ext_filter){ + .src_ip =3D mask->hdr.src_addr, + .dst_ip =3D mask->hdr.dst_addr, + .proto =3D mask->hdr.next_proto_id, + .tos =3D mask->hdr.type_of_service, + }; + /* Remove unwanted bits from values. */ + ipv4.val.src_ip &=3D ipv4.mask.src_ip; + ipv4.val.dst_ip &=3D ipv4.mask.dst_ip; + ipv4.val.proto &=3D ipv4.mask.proto; + ipv4.val.tos &=3D ipv4.mask.tos; + } + flow_verbs_hashfields_adjust(dev_flow, tunnel, + (ETH_RSS_IPV4 | ETH_RSS_FRAG_IPV4 | + ETH_RSS_NONFRAG_IPV4_TCP | + ETH_RSS_NONFRAG_IPV4_UDP | + ETH_RSS_NONFRAG_IPV4_OTHER), + (IBV_RX_HASH_SRC_IPV4 | + IBV_RX_HASH_DST_IPV4)); + dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L3; + flow_verbs_spec_add(dev_flow, &ipv4, size); +} + +/** + * Convert the @p item into a Verbs specification. This function assumes t= hat + * the input is valid and that there is space to insert the requested item + * into the flow. + * + * @param[in] item + * Item specification. + * @param[in, out] item_flags + * Bit mask that marks all detected items. + * @param[in, out] dev_flow + * Pointer to sepacific flow structure. + */ +static void +flow_verbs_translate_item_ipv6(const struct rte_flow_item *item, + uint64_t *item_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_item_ipv6 *spec =3D item->spec; + const struct rte_flow_item_ipv6 *mask =3D item->mask; + const int tunnel =3D !!(dev_flow->flow->layers & MLX5_FLOW_LAYER_TUNNEL); + unsigned int size =3D sizeof(struct ibv_flow_spec_ipv6); + struct ibv_flow_spec_ipv6 ipv6 =3D { + .type =3D IBV_FLOW_SPEC_IPV6 | (tunnel ? IBV_FLOW_SPEC_INNER : 0), + .size =3D size, + }; + + if (!mask) + mask =3D &rte_flow_item_ipv6_mask; + *item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L3_IPV6 : + MLX5_FLOW_LAYER_OUTER_L3_IPV6; + if (spec) { + unsigned int i; + uint32_t vtc_flow_val; + uint32_t vtc_flow_mask; + + memcpy(&ipv6.val.src_ip, spec->hdr.src_addr, + RTE_DIM(ipv6.val.src_ip)); + memcpy(&ipv6.val.dst_ip, spec->hdr.dst_addr, + RTE_DIM(ipv6.val.dst_ip)); + memcpy(&ipv6.mask.src_ip, mask->hdr.src_addr, + RTE_DIM(ipv6.mask.src_ip)); + memcpy(&ipv6.mask.dst_ip, mask->hdr.dst_addr, + RTE_DIM(ipv6.mask.dst_ip)); + vtc_flow_val =3D rte_be_to_cpu_32(spec->hdr.vtc_flow); + vtc_flow_mask =3D rte_be_to_cpu_32(mask->hdr.vtc_flow); + ipv6.val.flow_label =3D + rte_cpu_to_be_32((vtc_flow_val & IPV6_HDR_FL_MASK) >> + IPV6_HDR_FL_SHIFT); + ipv6.val.traffic_class =3D (vtc_flow_val & IPV6_HDR_TC_MASK) >> + IPV6_HDR_TC_SHIFT; + ipv6.val.next_hdr =3D spec->hdr.proto; + ipv6.val.hop_limit =3D spec->hdr.hop_limits; + ipv6.mask.flow_label =3D + rte_cpu_to_be_32((vtc_flow_mask & IPV6_HDR_FL_MASK) >> + IPV6_HDR_FL_SHIFT); + ipv6.mask.traffic_class =3D (vtc_flow_mask & IPV6_HDR_TC_MASK) >> + IPV6_HDR_TC_SHIFT; + ipv6.mask.next_hdr =3D mask->hdr.proto; + ipv6.mask.hop_limit =3D mask->hdr.hop_limits; + /* Remove unwanted bits from values. */ + for (i =3D 0; i < RTE_DIM(ipv6.val.src_ip); ++i) { + ipv6.val.src_ip[i] &=3D ipv6.mask.src_ip[i]; + ipv6.val.dst_ip[i] &=3D ipv6.mask.dst_ip[i]; + } + ipv6.val.flow_label &=3D ipv6.mask.flow_label; + ipv6.val.traffic_class &=3D ipv6.mask.traffic_class; + ipv6.val.next_hdr &=3D ipv6.mask.next_hdr; + ipv6.val.hop_limit &=3D ipv6.mask.hop_limit; + } + flow_verbs_hashfields_adjust(dev_flow, tunnel, + (ETH_RSS_IPV6 | ETH_RSS_FRAG_IPV6 | + ETH_RSS_NONFRAG_IPV6_TCP | + ETH_RSS_NONFRAG_IPV6_UDP | + ETH_RSS_IPV6_EX | + ETH_RSS_IPV6_TCP_EX | + ETH_RSS_IPV6_UDP_EX | + ETH_RSS_NONFRAG_IPV6_OTHER), + (IBV_RX_HASH_SRC_IPV6 | + IBV_RX_HASH_DST_IPV6)); + dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L3; + flow_verbs_spec_add(dev_flow, &ipv6, size); +} + +/** + * Convert the @p item into a Verbs specification. This function assumes t= hat + * the input is valid and that there is space to insert the requested item + * into the flow. + * + * @param[in] item + * Item specification. + * @param[in, out] item_flags + * Bit mask that marks all detected items. + * @param[in, out] dev_flow + * Pointer to sepacific flow structure. + */ +static void +flow_verbs_translate_item_udp(const struct rte_flow_item *item, + uint64_t *item_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_item_udp *spec =3D item->spec; + const struct rte_flow_item_udp *mask =3D item->mask; + const int tunnel =3D !!(*item_flags & MLX5_FLOW_LAYER_TUNNEL); + unsigned int size =3D sizeof(struct ibv_flow_spec_tcp_udp); + struct ibv_flow_spec_tcp_udp udp =3D { + .type =3D IBV_FLOW_SPEC_UDP | (tunnel ? IBV_FLOW_SPEC_INNER : 0), + .size =3D size, + }; + + if (!mask) + mask =3D &rte_flow_item_udp_mask; + *item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L4_UDP : + MLX5_FLOW_LAYER_OUTER_L4_UDP; + if (spec) { + udp.val.dst_port =3D spec->hdr.dst_port; + udp.val.src_port =3D spec->hdr.src_port; + udp.mask.dst_port =3D mask->hdr.dst_port; + udp.mask.src_port =3D mask->hdr.src_port; + /* Remove unwanted bits from values. */ + udp.val.src_port &=3D udp.mask.src_port; + udp.val.dst_port &=3D udp.mask.dst_port; + } + flow_verbs_hashfields_adjust(dev_flow, + tunnel, ETH_RSS_UDP, + (IBV_RX_HASH_SRC_PORT_UDP | + IBV_RX_HASH_DST_PORT_UDP)); + dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L4; + flow_verbs_spec_add(dev_flow, &udp, size); +} + +/** + * Convert the @p item into a Verbs specification. This function assumes t= hat + * the input is valid and that there is space to insert the requested item + * into the flow. + * + * @param[in] item + * Item specification. + * @param[in, out] item_flags + * Bit mask that marks all detected items. + * @param[in, out] dev_flow + * Pointer to sepacific flow structure. + */ +static void +flow_verbs_translate_item_tcp(const struct rte_flow_item *item, + uint64_t *item_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_item_tcp *spec =3D item->spec; + const struct rte_flow_item_tcp *mask =3D item->mask; + const int tunnel =3D !!(dev_flow->flow->layers & MLX5_FLOW_LAYER_TUNNEL); + unsigned int size =3D sizeof(struct ibv_flow_spec_tcp_udp); + struct ibv_flow_spec_tcp_udp tcp =3D { + .type =3D IBV_FLOW_SPEC_TCP | (tunnel ? IBV_FLOW_SPEC_INNER : 0), + .size =3D size, + }; + + if (!mask) + mask =3D &rte_flow_item_tcp_mask; + *item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L4_TCP : + MLX5_FLOW_LAYER_OUTER_L4_TCP; + if (spec) { + tcp.val.dst_port =3D spec->hdr.dst_port; + tcp.val.src_port =3D spec->hdr.src_port; + tcp.mask.dst_port =3D mask->hdr.dst_port; + tcp.mask.src_port =3D mask->hdr.src_port; + /* Remove unwanted bits from values. */ + tcp.val.src_port &=3D tcp.mask.src_port; + tcp.val.dst_port &=3D tcp.mask.dst_port; + } + flow_verbs_hashfields_adjust(dev_flow, + tunnel, ETH_RSS_TCP, + (IBV_RX_HASH_SRC_PORT_TCP | + IBV_RX_HASH_DST_PORT_TCP)); + dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L4; + flow_verbs_spec_add(dev_flow, &tcp, size); +} + +/** + * Convert the @p item into a Verbs specification. This function assumes t= hat + * the input is valid and that there is space to insert the requested item + * into the flow. + * + * @param[in] item + * Item specification. + * @param[in, out] item_flags + * Bit mask that marks all detected items. + * @param[in, out] dev_flow + * Pointer to sepacific flow structure. + */ +static void +flow_verbs_translate_item_vxlan(const struct rte_flow_item *item, + uint64_t *item_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_item_vxlan *spec =3D item->spec; + const struct rte_flow_item_vxlan *mask =3D item->mask; + unsigned int size =3D sizeof(struct ibv_flow_spec_tunnel); + struct ibv_flow_spec_tunnel vxlan =3D { + .type =3D IBV_FLOW_SPEC_VXLAN_TUNNEL, + .size =3D size, + }; + union vni { + uint32_t vlan_id; + uint8_t vni[4]; + } id =3D { .vlan_id =3D 0, }; + + if (!mask) + mask =3D &rte_flow_item_vxlan_mask; + if (spec) { + memcpy(&id.vni[1], spec->vni, 3); + vxlan.val.tunnel_id =3D id.vlan_id; + memcpy(&id.vni[1], mask->vni, 3); + vxlan.mask.tunnel_id =3D id.vlan_id; + /* Remove unwanted bits from values. */ + vxlan.val.tunnel_id &=3D vxlan.mask.tunnel_id; + } + flow_verbs_spec_add(dev_flow, &vxlan, size); + dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L2; + *item_flags |=3D MLX5_FLOW_LAYER_VXLAN; +} + +/** + * Convert the @p item into a Verbs specification. This function assumes t= hat + * the input is valid and that there is space to insert the requested item + * into the flow. + * + * @param[in] item + * Item specification. + * @param[in, out] item_flags + * Bit mask that marks all detected items. + * @param[in, out] dev_flow + * Pointer to sepacific flow structure. + */ +static void +flow_verbs_translate_item_vxlan_gpe(const struct rte_flow_item *item, + uint64_t *item_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_item_vxlan_gpe *spec =3D item->spec; + const struct rte_flow_item_vxlan_gpe *mask =3D item->mask; + unsigned int size =3D sizeof(struct ibv_flow_spec_tunnel); + struct ibv_flow_spec_tunnel vxlan_gpe =3D { + .type =3D IBV_FLOW_SPEC_VXLAN_TUNNEL, + .size =3D size, + }; + union vni { + uint32_t vlan_id; + uint8_t vni[4]; + } id =3D { .vlan_id =3D 0, }; + + if (!mask) + mask =3D &rte_flow_item_vxlan_gpe_mask; + if (spec) { + memcpy(&id.vni[1], spec->vni, 3); + vxlan_gpe.val.tunnel_id =3D id.vlan_id; + memcpy(&id.vni[1], mask->vni, 3); + vxlan_gpe.mask.tunnel_id =3D id.vlan_id; + /* Remove unwanted bits from values. */ + vxlan_gpe.val.tunnel_id &=3D vxlan_gpe.mask.tunnel_id; + } + flow_verbs_spec_add(dev_flow, &vxlan_gpe, size); + dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L2; + *item_flags |=3D MLX5_FLOW_LAYER_VXLAN_GPE; +} + +/** + * Update the protocol in Verbs IPv4/IPv6 spec. + * + * @param[in, out] attr + * Pointer to Verbs attributes structure. + * @param[in] search + * Specification type to search in order to update the IP protocol. + * @param[in] protocol + * Protocol value to set if none is present in the specification. + */ +static void +flow_verbs_item_gre_ip_protocol_update(struct ibv_flow_attr *attr, + enum ibv_flow_spec_type search, + uint8_t protocol) +{ + unsigned int i; + struct ibv_spec_header *hdr =3D (struct ibv_spec_header *) + ((uint8_t *)attr + sizeof(struct ibv_flow_attr)); + + if (!attr) + return; + for (i =3D 0; i !=3D attr->num_of_specs; ++i) { + if (hdr->type =3D=3D search) { + union { + struct ibv_flow_spec_ipv4_ext *ipv4; + struct ibv_flow_spec_ipv6 *ipv6; + } ip; + + switch (search) { + case IBV_FLOW_SPEC_IPV4_EXT: + ip.ipv4 =3D (struct ibv_flow_spec_ipv4_ext *)hdr; + if (!ip.ipv4->val.proto) { + ip.ipv4->val.proto =3D protocol; + ip.ipv4->mask.proto =3D 0xff; + } + break; + case IBV_FLOW_SPEC_IPV6: + ip.ipv6 =3D (struct ibv_flow_spec_ipv6 *)hdr; + if (!ip.ipv6->val.next_hdr) { + ip.ipv6->val.next_hdr =3D protocol; + ip.ipv6->mask.next_hdr =3D 0xff; + } + break; + default: + break; + } + break; + } + hdr =3D (struct ibv_spec_header *)((uint8_t *)hdr + hdr->size); + } +} + +/** + * Convert the @p item into a Verbs specification. This function assumes t= hat + * the input is valid and that there is space to insert the requested item + * into the flow. + * + * @param[in] item + * Item specification. + * @param[in, out] item_flags + * Bit mask that marks all detected items. + * @param[in, out] dev_flow + * Pointer to sepacific flow structure. + */ +static void +flow_verbs_translate_item_gre(const struct rte_flow_item *item __rte_unuse= d, + uint64_t *item_flags, + struct mlx5_flow *dev_flow) +{ + struct mlx5_flow_verbs *verbs =3D &dev_flow->verbs; +#ifndef HAVE_IBV_DEVICE_MPLS_SUPPORT + unsigned int size =3D sizeof(struct ibv_flow_spec_tunnel); + struct ibv_flow_spec_tunnel tunnel =3D { + .type =3D IBV_FLOW_SPEC_VXLAN_TUNNEL, + .size =3D size, + }; +#else + const struct rte_flow_item_gre *spec =3D item->spec; + const struct rte_flow_item_gre *mask =3D item->mask; + unsigned int size =3D sizeof(struct ibv_flow_spec_gre); + struct ibv_flow_spec_gre tunnel =3D { + .type =3D IBV_FLOW_SPEC_GRE, + .size =3D size, + }; + + if (!mask) + mask =3D &rte_flow_item_gre_mask; + if (spec) { + tunnel.val.c_ks_res0_ver =3D spec->c_rsvd0_ver; + tunnel.val.protocol =3D spec->protocol; + tunnel.mask.c_ks_res0_ver =3D mask->c_rsvd0_ver; + tunnel.mask.protocol =3D mask->protocol; + /* Remove unwanted bits from values. */ + tunnel.val.c_ks_res0_ver &=3D tunnel.mask.c_ks_res0_ver; + tunnel.val.protocol &=3D tunnel.mask.protocol; + tunnel.val.key &=3D tunnel.mask.key; + } +#endif + if (*item_flags & MLX5_FLOW_LAYER_OUTER_L3_IPV4) + flow_verbs_item_gre_ip_protocol_update(verbs->attr, + IBV_FLOW_SPEC_IPV4_EXT, + MLX5_IP_PROTOCOL_GRE); + else + flow_verbs_item_gre_ip_protocol_update(verbs->attr, + IBV_FLOW_SPEC_IPV6, + MLX5_IP_PROTOCOL_GRE); + flow_verbs_spec_add(dev_flow, &tunnel, size); + verbs->attr->priority =3D MLX5_PRIORITY_MAP_L2; + *item_flags |=3D MLX5_FLOW_LAYER_GRE; +} + +/** + * Convert the @p action into a Verbs specification. This function assumes= that + * the input is valid and that there is space to insert the requested acti= on + * into the flow. This function also return the action that was added. + * + * @param[in] item + * Item specification. + * @param[in, out] item_flags + * Bit mask that marks all detected items. + * @param[in, out] dev_flow + * Pointer to sepacific flow structure. + */ +static void +flow_verbs_translate_item_mpls(const struct rte_flow_item *item __rte_unus= ed, + uint64_t *action_flags __rte_unused, + struct mlx5_flow *dev_flow __rte_unused) +{ +#ifdef HAVE_IBV_DEVICE_MPLS_SUPPORT + const struct rte_flow_item_mpls *spec =3D item->spec; + const struct rte_flow_item_mpls *mask =3D item->mask; + unsigned int size =3D sizeof(struct ibv_flow_spec_mpls); + struct ibv_flow_spec_mpls mpls =3D { + .type =3D IBV_FLOW_SPEC_MPLS, + .size =3D size, + }; + + if (!mask) + mask =3D &rte_flow_item_mpls_mask; + if (spec) { + memcpy(&mpls.val.label, spec, sizeof(mpls.val.label)); + memcpy(&mpls.mask.label, mask, sizeof(mpls.mask.label)); + /* Remove unwanted bits from values. */ + mpls.val.label &=3D mpls.mask.label; + } + flow_verbs_spec_add(dev_flow, &mpls, size); + dev_flow->verbs.attr->priority =3D MLX5_PRIORITY_MAP_L2; + *action_flags |=3D MLX5_FLOW_LAYER_MPLS; +#endif +} + +/** + * Convert the @p action into a Verbs specification. This function assumes= that + * the input is valid and that there is space to insert the requested acti= on + * into the flow. This function also return the action that was added. + * + * @param[in, out] action_flags + * Pointer to the detected actions. + * @param[in] dev_flow + * Pointer to mlx5_flow. + */ +static void +flow_verbs_translate_action_drop(uint64_t *action_flags, + struct mlx5_flow *dev_flow) +{ + unsigned int size =3D sizeof(struct ibv_flow_spec_action_drop); + struct ibv_flow_spec_action_drop drop =3D { + .type =3D IBV_FLOW_SPEC_ACTION_DROP, + .size =3D size, + }; + + flow_verbs_spec_add(dev_flow, &drop, size); + *action_flags |=3D MLX5_ACTION_DROP; +} + +/** + * Convert the @p action into a Verbs specification. This function assumes= that + * the input is valid and that there is space to insert the requested acti= on + * into the flow. This function also return the action that was added. + * + * @param[in] action + * Action configuration. + * @param[in, out] action_flags + * Pointer to the detected actions. + * @param[in] dev_flow + * Pointer to mlx5_flow. + */ +static void +flow_verbs_translate_action_queue(const struct rte_flow_action *action, + uint64_t *action_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_action_queue *queue =3D action->conf; + struct rte_flow *flow =3D dev_flow->flow; + + if (flow->queue) + (*flow->queue)[0] =3D queue->index; + flow->rss.queue_num =3D 1; + *action_flags |=3D MLX5_ACTION_QUEUE; +} + +/** + * Convert the @p action into a Verbs specification. This function assumes= that + * the input is valid and that there is space to insert the requested acti= on + * into the flow. This function also return the action that was added. + * + * @param[in] action + * Action configuration. + * @param[in, out] action_flags + * Pointer to the detected actions. + * @param[in] dev_flow + * Pointer to mlx5_flow. + */ +static void +flow_verbs_translate_action_rss(const struct rte_flow_action *action, + uint64_t *action_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_action_rss *rss =3D action->conf; + struct rte_flow *flow =3D dev_flow->flow; + + if (flow->queue) + memcpy((*flow->queue), rss->queue, + rss->queue_num * sizeof(uint16_t)); + flow->rss.queue_num =3D rss->queue_num; + memcpy(flow->key, rss->key, MLX5_RSS_HASH_KEY_LEN); + flow->rss.types =3D rss->types; + flow->rss.level =3D rss->level; + *action_flags |=3D MLX5_ACTION_RSS; +} + +/** + * Convert the @p action into a Verbs specification. This function assumes= that + * the input is valid and that there is space to insert the requested acti= on + * into the flow. This function also return the action that was added. + * + * @param[in] action + * Action configuration. + * @param[in, out] action_flags + * Pointer to the detected actions. + * @param[in] dev_flow + * Pointer to mlx5_flow. + */ +static void +flow_verbs_translate_action_flag + (const struct rte_flow_action *action __rte_unused, + uint64_t *action_flags, + struct mlx5_flow *dev_flow) +{ + unsigned int size =3D sizeof(struct ibv_flow_spec_action_tag); + struct ibv_flow_spec_action_tag tag =3D { + .type =3D IBV_FLOW_SPEC_ACTION_TAG, + .size =3D size, + .tag_id =3D mlx5_flow_mark_set(MLX5_FLOW_MARK_DEFAULT), + }; + *action_flags |=3D MLX5_ACTION_MARK; + flow_verbs_spec_add(dev_flow, &tag, size); +} + +/** + * Update verbs specification to modify the flag to mark. + * + * @param[in, out] verbs + * Pointer to the mlx5_flow_verbs structure. + * @param[in] mark_id + * Mark identifier to replace the flag. + */ +static void +flow_verbs_mark_update(struct mlx5_flow_verbs *verbs, uint32_t mark_id) +{ + struct ibv_spec_header *hdr; + int i; + + if (!verbs) + return; + /* Update Verbs specification. */ + hdr =3D (struct ibv_spec_header *)verbs->specs; + if (!hdr) + return; + for (i =3D 0; i !=3D verbs->attr->num_of_specs; ++i) { + if (hdr->type =3D=3D IBV_FLOW_SPEC_ACTION_TAG) { + struct ibv_flow_spec_action_tag *t =3D + (struct ibv_flow_spec_action_tag *)hdr; + + t->tag_id =3D mlx5_flow_mark_set(mark_id); + } + hdr =3D (struct ibv_spec_header *)((uintptr_t)hdr + hdr->size); + } +} + +/** + * Convert the @p action into a Verbs specification. This function assumes= that + * the input is valid and that there is space to insert the requested acti= on + * into the flow. This function also return the action that was added. + * + * @param[in] action + * Action configuration. + * @param[in, out] action_flags + * Pointer to the detected actions. + * @param[in] dev_flow + * Pointer to mlx5_flow. + */ +static void +flow_verbs_translate_action_mark(const struct rte_flow_action *action, + uint64_t *action_flags, + struct mlx5_flow *dev_flow) +{ + const struct rte_flow_action_mark *mark =3D action->conf; + unsigned int size =3D sizeof(struct ibv_flow_spec_action_tag); + struct ibv_flow_spec_action_tag tag =3D { + .type =3D IBV_FLOW_SPEC_ACTION_TAG, + .size =3D size, + }; + struct mlx5_flow_verbs *verbs =3D &dev_flow->verbs; + + if (*action_flags & MLX5_ACTION_FLAG) { + flow_verbs_mark_update(verbs, mark->id); + size =3D 0; + } else { + tag.tag_id =3D mlx5_flow_mark_set(mark->id); + flow_verbs_spec_add(dev_flow, &tag, size); + } + *action_flags |=3D MLX5_ACTION_MARK; +} + +/** + * Convert the @p action into a Verbs specification. This function assumes= that + * the input is valid and that there is space to insert the requested acti= on + * into the flow. This function also return the action that was added. + * + * @param[in] dev + * Pointer to the Ethernet device structure. + * @param[in] action + * Action configuration. + * @param[in, out] action_flags + * Pointer to the detected actions. + * @param[in] dev_flow + * Pointer to mlx5_flow. + * @param[out] error + * Pointer to error structure. + * + * @return + * 0 On success else a negative errno value is returned and rte_errno is= set. + */ +static int +flow_verbs_translate_action_count(struct rte_eth_dev *dev, + const struct rte_flow_action *action, + uint64_t *action_flags, + struct mlx5_flow *dev_flow, + struct rte_flow_error *error) +{ + const struct rte_flow_action_count *count =3D action->conf; + struct rte_flow *flow =3D dev_flow->flow; +#ifdef HAVE_IBV_DEVICE_COUNTERS_SET_SUPPORT + unsigned int size =3D sizeof(struct ibv_flow_spec_counter_action); + struct ibv_flow_spec_counter_action counter =3D { + .type =3D IBV_FLOW_SPEC_ACTION_COUNT, + .size =3D size, + }; +#endif + + if (!flow->counter) { + flow->counter =3D flow_verbs_counter_new(dev, count->shared, + count->id); + if (!flow->counter) + return rte_flow_error_set(error, ENOTSUP, + RTE_FLOW_ERROR_TYPE_ACTION, + action, + "cannot get counter" + " context."); + } + *action_flags |=3D MLX5_ACTION_COUNT; +#ifdef HAVE_IBV_DEVICE_COUNTERS_SET_SUPPORT + counter.counter_set_handle =3D flow->counter->cs->handle; + flow_verbs_spec_add(dev_flow, &counter, size); +#endif + return 0; +} + +/** + * Internal validation function. For validating both actions and items. + * + * @param[in] dev + * Pointer to the Ethernet device structure. + * @param[in] attr + * Pointer to the flow attributes. + * @param[in] items + * Pointer to the list of items. + * @param[in] actions + * Pointer to the list of actions. + * @param[out] error + * Pointer to the error structure. + * + * @return + * 0 on success, a negative errno value otherwise and rte_errno is set. + */ +static int +flow_verbs_validate(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_error *error) +{ + int ret; + uint32_t action_flags =3D 0; + uint32_t item_flags =3D 0; + int tunnel =3D 0; + uint8_t next_protocol =3D 0xff; + + if (items =3D=3D NULL) + return -1; + ret =3D mlx5_flow_validate_attributes(dev, attr, error); + if (ret < 0) + return ret; + for (; items->type !=3D RTE_FLOW_ITEM_TYPE_END; items++) { + int ret =3D 0; + switch (items->type) { + case RTE_FLOW_ITEM_TYPE_VOID: + break; + case RTE_FLOW_ITEM_TYPE_ETH: + ret =3D mlx5_flow_validate_item_eth(items, item_flags, + error); + if (ret < 0) + return ret; + item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L2 : + MLX5_FLOW_LAYER_OUTER_L2; + break; + case RTE_FLOW_ITEM_TYPE_VLAN: + ret =3D mlx5_flow_validate_item_vlan(items, item_flags, + error); + if (ret < 0) + return ret; + item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_VLAN : + MLX5_FLOW_LAYER_OUTER_VLAN; + break; + case RTE_FLOW_ITEM_TYPE_IPV4: + ret =3D mlx5_flow_validate_item_ipv4(items, item_flags, + error); + if (ret < 0) + return ret; + item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L3_IPV4 : + MLX5_FLOW_LAYER_OUTER_L3_IPV4; + if (items->mask !=3D NULL && + ((const struct rte_flow_item_ipv4 *) + items->mask)->hdr.next_proto_id) + next_protocol =3D + ((const struct rte_flow_item_ipv4 *) + (items->spec))->hdr.next_proto_id; + break; + case RTE_FLOW_ITEM_TYPE_IPV6: + ret =3D mlx5_flow_validate_item_ipv6(items, item_flags, + error); + if (ret < 0) + return ret; + item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L3_IPV6 : + MLX5_FLOW_LAYER_OUTER_L3_IPV6; + if (items->mask !=3D NULL && + ((const struct rte_flow_item_ipv6 *) + items->mask)->hdr.proto) + next_protocol =3D + ((const struct rte_flow_item_ipv6 *) + items->spec)->hdr.proto; + break; + case RTE_FLOW_ITEM_TYPE_UDP: + ret =3D mlx5_flow_validate_item_udp(items, item_flags, + next_protocol, + error); + if (ret < 0) + return ret; + item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L4_UDP : + MLX5_FLOW_LAYER_OUTER_L4_UDP; + break; + case RTE_FLOW_ITEM_TYPE_TCP: + ret =3D mlx5_flow_validate_item_tcp(items, item_flags, + next_protocol, error); + if (ret < 0) + return ret; + item_flags |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L4_TCP : + MLX5_FLOW_LAYER_OUTER_L4_TCP; + break; + case RTE_FLOW_ITEM_TYPE_VXLAN: + ret =3D mlx5_flow_validate_item_vxlan(items, item_flags, + error); + if (ret < 0) + return ret; + item_flags |=3D MLX5_FLOW_LAYER_VXLAN; + break; + case RTE_FLOW_ITEM_TYPE_VXLAN_GPE: + ret =3D mlx5_flow_validate_item_vxlan_gpe(items, + item_flags, + dev, error); + if (ret < 0) + return ret; + item_flags |=3D MLX5_FLOW_LAYER_VXLAN_GPE; + break; + case RTE_FLOW_ITEM_TYPE_GRE: + ret =3D mlx5_flow_validate_item_gre(items, item_flags, + next_protocol, error); + if (ret < 0) + return ret; + item_flags |=3D MLX5_FLOW_LAYER_GRE; + break; + case RTE_FLOW_ITEM_TYPE_MPLS: + ret =3D mlx5_flow_validate_item_mpls(items, item_flags, + next_protocol, + error); + if (ret < 0) + return ret; + if (next_protocol !=3D 0xff && + next_protocol !=3D MLX5_IP_PROTOCOL_MPLS) + return rte_flow_error_set + (error, ENOTSUP, + RTE_FLOW_ERROR_TYPE_ITEM, items, + "protocol filtering not compatible" + " with MPLS layer"); + item_flags |=3D MLX5_FLOW_LAYER_MPLS; + break; + default: + return rte_flow_error_set(error, ENOTSUP, + RTE_FLOW_ERROR_TYPE_ITEM, + NULL, "item not supported"); + } + } + for (; actions->type !=3D RTE_FLOW_ACTION_TYPE_END; actions++) { + tunnel =3D !!(item_flags & MLX5_FLOW_LAYER_TUNNEL); + switch (actions->type) { + case RTE_FLOW_ACTION_TYPE_VOID: + break; + case RTE_FLOW_ACTION_TYPE_FLAG: + ret =3D mlx5_flow_validate_action_flag(action_flags, + error); + if (ret < 0) + return ret; + action_flags |=3D MLX5_ACTION_FLAG; + break; + case RTE_FLOW_ACTION_TYPE_MARK: + ret =3D mlx5_flow_validate_action_mark(actions, + action_flags, + error); + if (ret < 0) + return ret; + action_flags |=3D MLX5_ACTION_MARK; + break; + case RTE_FLOW_ACTION_TYPE_DROP: + ret =3D mlx5_flow_validate_action_drop(action_flags, + error); + if (ret < 0) + return ret; + action_flags |=3D MLX5_ACTION_DROP; + break; + case RTE_FLOW_ACTION_TYPE_QUEUE: + ret =3D mlx5_flow_validate_action_queue(actions, + action_flags, dev, + error); + if (ret < 0) + return ret; + action_flags |=3D MLX5_ACTION_QUEUE; + break; + case RTE_FLOW_ACTION_TYPE_RSS: + ret =3D mlx5_flow_validate_action_rss(actions, + action_flags, dev, + error); + if (ret < 0) + return ret; + action_flags |=3D MLX5_ACTION_RSS; + break; + case RTE_FLOW_ACTION_TYPE_COUNT: + ret =3D mlx5_flow_validate_action_count(dev, error); + if (ret < 0) + return ret; + action_flags |=3D MLX5_ACTION_COUNT; + break; + default: + return rte_flow_error_set(error, ENOTSUP, + RTE_FLOW_ERROR_TYPE_ACTION, + actions, + "action not supported"); + } + } + return 0; +} + +/** + * Calculate the required bytes that are needed for the action part of the= verbs + * flow, in addtion returns bit-fields with all the detected action, in or= der to + * avoid another interation over the actions. + * + * @param[in] actions + * Pointer to the list of actions. + * @param[out] action_flags + * Pointer to the detected actions. + * + * @return + * The size of the memory needed for all actions. + */ +static int +flow_verbs_get_actions_and_size(const struct rte_flow_action actions[], + uint64_t *action_flags) +{ + int size =3D 0; + uint64_t detected_actions =3D 0; + + for (; actions->type !=3D RTE_FLOW_ACTION_TYPE_END; actions++) { + switch (actions->type) { + case RTE_FLOW_ACTION_TYPE_VOID: + break; + case RTE_FLOW_ACTION_TYPE_FLAG: + size +=3D sizeof(struct ibv_flow_spec_action_tag); + detected_actions |=3D MLX5_ACTION_FLAG; + break; + case RTE_FLOW_ACTION_TYPE_MARK: + size +=3D sizeof(struct ibv_flow_spec_action_tag); + detected_actions |=3D MLX5_ACTION_MARK; + break; + case RTE_FLOW_ACTION_TYPE_DROP: + size +=3D sizeof(struct ibv_flow_spec_action_drop); + detected_actions |=3D MLX5_ACTION_DROP; + break; + case RTE_FLOW_ACTION_TYPE_QUEUE: + detected_actions |=3D MLX5_ACTION_QUEUE; + break; + case RTE_FLOW_ACTION_TYPE_RSS: + detected_actions |=3D MLX5_ACTION_RSS; + break; + case RTE_FLOW_ACTION_TYPE_COUNT: +#ifdef HAVE_IBV_DEVICE_COUNTERS_SET_SUPPORT + size +=3D sizeof(struct ibv_flow_spec_counter_action); +#endif + detected_actions |=3D MLX5_ACTION_COUNT; + break; + default: + break; + } + } + *action_flags =3D detected_actions; + return size; +} + +/** + * Calculate the required bytes that are needed for the item part of the v= erbs + * flow, in addtion returns bit-fields with all the detected action, in or= der to + * avoid another interation over the actions. + * + * @param[in] actions + * Pointer to the list of items. + * @param[in, out] item_flags + * Pointer to the detected items. + * + * @return + * The size of the memory needed for all items. + */ +static int +flow_verbs_get_items_and_size(const struct rte_flow_item items[], + uint64_t *item_flags) +{ + int size =3D 0; + uint64_t detected_items =3D 0; + const int tunnel =3D !!(*item_flags & MLX5_FLOW_LAYER_TUNNEL); + + for (; items->type !=3D RTE_FLOW_ITEM_TYPE_END; items++) { + switch (items->type) { + case RTE_FLOW_ITEM_TYPE_VOID: + break; + case RTE_FLOW_ITEM_TYPE_ETH: + size +=3D sizeof(struct ibv_flow_spec_eth); + detected_items |=3D tunnel ? MLX5_FLOW_LAYER_INNER_L2 : + MLX5_FLOW_LAYER_OUTER_L2; + break; + case RTE_FLOW_ITEM_TYPE_VLAN: + size +=3D sizeof(struct ibv_flow_spec_eth); + detected_items |=3D tunnel ? MLX5_FLOW_LAYER_INNER_VLAN : + MLX5_FLOW_LAYER_OUTER_VLAN; + break; + case RTE_FLOW_ITEM_TYPE_IPV4: + size +=3D sizeof(struct ibv_flow_spec_ipv4_ext); + detected_items |=3D tunnel ? + MLX5_FLOW_LAYER_INNER_L3_IPV4 : + MLX5_FLOW_LAYER_OUTER_L3_IPV4; + break; + case RTE_FLOW_ITEM_TYPE_IPV6: + size +=3D sizeof(struct ibv_flow_spec_ipv6); + detected_items |=3D tunnel ? + MLX5_FLOW_LAYER_INNER_L3_IPV6 : + MLX5_FLOW_LAYER_OUTER_L3_IPV6; + break; + case RTE_FLOW_ITEM_TYPE_UDP: + size +=3D sizeof(struct ibv_flow_spec_tcp_udp); + detected_items |=3D tunnel ? + MLX5_FLOW_LAYER_INNER_L4_UDP : + MLX5_FLOW_LAYER_OUTER_L4_UDP; + break; + case RTE_FLOW_ITEM_TYPE_TCP: + size +=3D sizeof(struct ibv_flow_spec_tcp_udp); + detected_items |=3D tunnel ? + MLX5_FLOW_LAYER_INNER_L4_TCP : + MLX5_FLOW_LAYER_OUTER_L4_TCP; + break; + case RTE_FLOW_ITEM_TYPE_VXLAN: + size +=3D sizeof(struct ibv_flow_spec_tunnel); + detected_items |=3D MLX5_FLOW_LAYER_VXLAN; + break; + case RTE_FLOW_ITEM_TYPE_VXLAN_GPE: + size +=3D sizeof(struct ibv_flow_spec_tunnel); + detected_items |=3D MLX5_FLOW_LAYER_VXLAN_GPE; + break; +#ifdef HAVE_IBV_DEVICE_MPLS_SUPPORT + case RTE_FLOW_ITEM_TYPE_GRE: + size +=3D sizeof(struct ibv_flow_spec_gre); + detected_items |=3D MLX5_FLOW_LAYER_GRE; + break; + case RTE_FLOW_ITEM_TYPE_MPLS: + size +=3D sizeof(struct ibv_flow_spec_mpls); + detected_items |=3D MLX5_FLOW_LAYER_MPLS; + break; +#else + case RTE_FLOW_ITEM_TYPE_GRE: + size +=3D sizeof(struct ibv_flow_spec_tunnel); + detected_items |=3D MLX5_FLOW_LAYER_TUNNEL; + break; +#endif + default: + break; + } + } + *item_flags =3D detected_items; + return size; +} + +/** + * Internal preparation function. Allocate mlx5_flow with the required siz= e. + * The required size is calculate based on the actions and items. This fun= ction + * also returns the detected actions and items for later use. + * + * @param[in] attr + * Pointer to the flow attributes. + * @param[in] items + * Pointer to the list of items. + * @param[in] actions + * Pointer to the list of actions. + * @param[out] item_flags + * Pointer to bit mask of all items detected. + * @param[out] action_flags + * Pointer to bit mask of all actions detected. + * @param[out] error + * Pointer to the error structure. + * + * @return + * Pointer to mlx5_flow object on success, otherwise NULL and rte_errno + * is set. + */ +static struct mlx5_flow * +flow_verbs_prepare(const struct rte_flow_attr *attr __rte_unused, + const struct rte_flow_item items[], + const struct rte_flow_action actions[], + uint64_t *item_flags, + uint64_t *action_flags, + struct rte_flow_error *error) +{ + uint32_t size =3D sizeof(struct mlx5_flow) + sizeof(struct ibv_flow_attr)= ; + struct mlx5_flow *flow; + + size +=3D flow_verbs_get_actions_and_size(actions, action_flags); + size +=3D flow_verbs_get_items_and_size(items, item_flags); + flow =3D rte_calloc(__func__, 1, size, 0); + if (!flow) { + rte_flow_error_set(error, ENOMEM, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL, + "not enough memory to create flow"); + return NULL; + } + flow->verbs.attr =3D (void *)(flow + 1); + flow->verbs.specs =3D + (uint8_t *)(flow + 1) + sizeof(struct ibv_flow_attr); + return flow; +} + +/** + * Fill the flow with verb spec. + * + * @param[in] dev + * Pointer to Ethernet device. + * @param[in, out] dev_flow + * Pointer to the mlx5 flow. + * @param[in] attr + * Pointer to the flow attributes. + * @param[in] items + * Pointer to the list of items. + * @param[in] actions + * Pointer to the list of actions. + * @param[out] error + * Pointer to the error structure. + * + * @return + * 0 on success, else a negative errno value otherwise and rte_ernno is = set. + */ +static int +flow_verbs_translate(struct rte_eth_dev *dev, + struct mlx5_flow *dev_flow, + const struct rte_flow_attr *attr, + const struct rte_flow_item items[], + const struct rte_flow_action actions[], + struct rte_flow_error *error) +{ + uint64_t action_flags =3D 0; + uint64_t item_flags =3D 0; + uint64_t priority =3D attr->priority; + struct priv *priv =3D dev->data->dev_private; + + if (priority =3D=3D MLX5_FLOW_PRIO_RSVD) + priority =3D priv->config.flow_prio - 1; + for (; actions->type !=3D RTE_FLOW_ACTION_TYPE_END; actions++) { + int ret; + switch (actions->type) { + case RTE_FLOW_ACTION_TYPE_VOID: + break; + case RTE_FLOW_ACTION_TYPE_FLAG: + flow_verbs_translate_action_flag(actions, + &action_flags, + dev_flow); + break; + case RTE_FLOW_ACTION_TYPE_MARK: + flow_verbs_translate_action_mark(actions, + &action_flags, + dev_flow); + break; + case RTE_FLOW_ACTION_TYPE_DROP: + flow_verbs_translate_action_drop(&action_flags, + dev_flow); + break; + case RTE_FLOW_ACTION_TYPE_QUEUE: + flow_verbs_translate_action_queue(actions, + &action_flags, + dev_flow); + break; + case RTE_FLOW_ACTION_TYPE_RSS: + flow_verbs_translate_action_rss(actions, + &action_flags, + dev_flow); + break; + case RTE_FLOW_ACTION_TYPE_COUNT: + ret =3D flow_verbs_translate_action_count(dev, + actions, + &action_flags, + dev_flow, + error); + if (ret < 0) + return ret; + break; + default: + return rte_flow_error_set(error, ENOTSUP, + RTE_FLOW_ERROR_TYPE_ACTION, + actions, + "action not supported"); + } + } + dev_flow->flow->actions |=3D action_flags; + for (; items->type !=3D RTE_FLOW_ITEM_TYPE_END; items++) { + switch (items->type) { + case RTE_FLOW_ITEM_TYPE_VOID: + break; + case RTE_FLOW_ITEM_TYPE_ETH: + flow_verbs_translate_item_eth(items, &item_flags, + dev_flow); + break; + case RTE_FLOW_ITEM_TYPE_VLAN: + flow_verbs_translate_item_vlan(items, &item_flags, + dev_flow); + break; + case RTE_FLOW_ITEM_TYPE_IPV4: + flow_verbs_translate_item_ipv4(items, &item_flags, + dev_flow); + break; + case RTE_FLOW_ITEM_TYPE_IPV6: + flow_verbs_translate_item_ipv6(items, &item_flags, + dev_flow); + break; + case RTE_FLOW_ITEM_TYPE_UDP: + flow_verbs_translate_item_udp(items, &item_flags, + dev_flow); + break; + case RTE_FLOW_ITEM_TYPE_TCP: + flow_verbs_translate_item_tcp(items, &item_flags, + dev_flow); + break; + case RTE_FLOW_ITEM_TYPE_VXLAN: + flow_verbs_translate_item_vxlan(items, &item_flags, + dev_flow); + break; + case RTE_FLOW_ITEM_TYPE_VXLAN_GPE: + flow_verbs_translate_item_vxlan_gpe(items, &item_flags, + dev_flow); + break; + case RTE_FLOW_ITEM_TYPE_GRE: + flow_verbs_translate_item_gre(items, &item_flags, + dev_flow); + break; + case RTE_FLOW_ITEM_TYPE_MPLS: + flow_verbs_translate_item_mpls(items, &item_flags, + dev_flow); + break; + default: + return rte_flow_error_set(error, ENOTSUP, + RTE_FLOW_ERROR_TYPE_ITEM, + NULL, + "item not supported"); + } + } + dev_flow->verbs.attr->priority =3D + mlx5_flow_adjust_priority(dev, priority, + dev_flow->verbs.attr->priority); + return 0; +} + +/** + * Remove the flow from the NIC but keeps it in memory. + * + * @param[in] dev + * Pointer to the Ethernet device structure. + * @param[in, out] flow + * Pointer to flow structure. + */ +static void +flow_verbs_remove(struct rte_eth_dev *dev, struct rte_flow *flow) +{ + struct mlx5_flow_verbs *verbs; + struct mlx5_flow *dev_flow; + + if (!flow) + return; + LIST_FOREACH(dev_flow, &flow->dev_flows, next) { + verbs =3D &dev_flow->verbs; + if (verbs->flow) { + claim_zero(mlx5_glue->destroy_flow(verbs->flow)); + verbs->flow =3D NULL; + } + if (verbs->hrxq) { + if (flow->actions & MLX5_ACTION_DROP) + mlx5_hrxq_drop_release(dev); + else + mlx5_hrxq_release(dev, verbs->hrxq); + verbs->hrxq =3D NULL; + } + } + if (flow->counter) { + flow_verbs_counter_release(flow->counter); + flow->counter =3D NULL; + } +} + +/** + * Remove the flow from the NIC and the memory. + * + * @param[in] dev + * Pointer to the Ethernet device structure. + * @param[in, out] flow + * Pointer to flow structure. + */ +static void +flow_verbs_destroy(struct rte_eth_dev *dev, struct rte_flow *flow) +{ + struct mlx5_flow *dev_flow; + + if (!flow) + return; + flow_verbs_remove(dev, flow); + while (!LIST_EMPTY(&flow->dev_flows)) { + dev_flow =3D LIST_FIRST(&flow->dev_flows); + LIST_REMOVE(dev_flow, next); + rte_free(dev_flow); + } +} + +/** + * Apply the flow to the NIC. + * + * @param[in] dev + * Pointer to the Ethernet device structure. + * @param[in, out] flow + * Pointer to flow structure. + * @param[out] error + * Pointer to error structure. + * + * @return + * 0 on success, a negative errno value otherwise and rte_errno is set. + */ +static int +flow_verbs_apply(struct rte_eth_dev *dev, struct rte_flow *flow, + struct rte_flow_error *error) +{ + struct mlx5_flow_verbs *verbs; + struct mlx5_flow *dev_flow; + int err; + + LIST_FOREACH(dev_flow, &flow->dev_flows, next) { + verbs =3D &dev_flow->verbs; + if (flow->actions & MLX5_ACTION_DROP) { + verbs->hrxq =3D mlx5_hrxq_drop_new(dev); + if (!verbs->hrxq) { + rte_flow_error_set + (error, errno, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL, + "cannot get drop hash queue"); + goto error; + } + } else { + struct mlx5_hrxq *hrxq; + + hrxq =3D mlx5_hrxq_get(dev, flow->key, + MLX5_RSS_HASH_KEY_LEN, + verbs->hash_fields, + (*flow->queue), + flow->rss.queue_num); + if (!hrxq) + hrxq =3D mlx5_hrxq_new(dev, flow->key, + MLX5_RSS_HASH_KEY_LEN, + verbs->hash_fields, + (*flow->queue), + flow->rss.queue_num, + !!(flow->layers & + MLX5_FLOW_LAYER_TUNNEL)); + if (!hrxq) { + rte_flow_error_set + (error, rte_errno, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL, + "cannot get hash queue"); + goto error; + } + verbs->hrxq =3D hrxq; + } + verbs->flow =3D mlx5_glue->create_flow(verbs->hrxq->qp, + verbs->attr); + if (!verbs->flow) { + rte_flow_error_set(error, errno, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, + NULL, + "hardware refuses to create flow"); + goto error; + } + } + return 0; +error: + err =3D rte_errno; /* Save rte_errno before cleanup. */ + LIST_FOREACH(dev_flow, &flow->dev_flows, next) { + verbs =3D &dev_flow->verbs; + if (verbs->hrxq) { + if (flow->actions & MLX5_ACTION_DROP) + mlx5_hrxq_drop_release(dev); + else + mlx5_hrxq_release(dev, verbs->hrxq); + verbs->hrxq =3D NULL; + } + } + rte_errno =3D err; /* Restore rte_errno. */ + return -rte_errno; +} + +void +mlx5_flow_verbs_get_driver_ops(struct mlx5_flow_driver_ops *flow_ops) +{ + *flow_ops =3D (struct mlx5_flow_driver_ops) { + .validate =3D flow_verbs_validate, + .prepare =3D flow_verbs_prepare, + .translate =3D flow_verbs_translate, + .apply =3D flow_verbs_apply, + .remove =3D flow_verbs_remove, + .destroy =3D flow_verbs_destroy, + }; +} --=20 2.11.0