From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 15F0748AEF for ; Wed, 12 Nov 2025 17:53:47 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0FB09402A0; Wed, 12 Nov 2025 17:53:47 +0100 (CET) Received: from mail-wm1-f49.google.com (mail-wm1-f49.google.com [209.85.128.49]) by mails.dpdk.org (Postfix) with ESMTP id D8632402A0 for ; Wed, 12 Nov 2025 17:53:45 +0100 (CET) Received: by mail-wm1-f49.google.com with SMTP id 5b1f17b1804b1-47778704516so5697715e9.3 for ; Wed, 12 Nov 2025 08:53:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1762966425; x=1763571225; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=HONDQaFnaAwYbwuGPbU8NouKyFWPhaXaiyZluHSwyog=; b=EamaTTsYcIFhE5/GA51Qh9KUq3doJGDO04dcCfMuMEPH2qWaapriuw6/E9WsMXMeya jKEehrle0wDqofwbYdstcXuFdsJ3zBNCBOwq/v9magWLWuKHHeaZFZujv4cIquq4PHb8 xlIUeViHZwMYqoOm6Ey+JbvW2g4gPAgOp36lGScSXzw4EgttoVhzsAaxq08wIONhyK9+ Ekf+a4VtEG6GkSAB84x6UrmFeMdcTBqfFRmcN6F71hCghz/9kIbf0HqjMEKX4yyDHzXT qNPgNQNl3jBsl9jMbOULPxavTtmy4RxHb+vPFN0SkRKYXXU9UgHqp2r8OtquMhEadPIj g4vA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762966425; x=1763571225; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=HONDQaFnaAwYbwuGPbU8NouKyFWPhaXaiyZluHSwyog=; b=AbcUCXkU4w1V5Bo9DXyUBs2wf7EDPyHENUicjJa0lD+QUn0ar4RKOrDIAcUlDr0Uer jqEE7jGw3U8n8CBBMAuHHVZF26DemVtY3NxwlyRcKxkNFFzXCkE2ARI4vk6xIRo/coZe 3IY6HdU/ccfLqVv1fqzPvUhP2ypaegQOg7k31eYn/umah8TYqjd4B+infPd77ysaaMgz 9/1nv+gxDfU4QGmZy5fE8d5nqJlMu4qL7okEX1lOzrQXWbqWZYJz7hM/cUdvNr87we5h 4aThAIaCgwKZgoVgfqdb1KVkiLfbt4tZ5tggbLXcHB+o+LtRFbaOcS3LtKo8ncFAp0Qb Og1A== X-Gm-Message-State: AOJu0Yx6TxPCftK8wgbBBPWszjDwE6JJQwBfXQNhrTbpC+dunSj4RWwn JkBT1G54pbT+CjFHUcOtQAAlZzr07mwdH77hlzj5uxOtaUWlVbKgF5oAnOiEkg== X-Gm-Gg: ASbGnctUZqS+f28lK2yPj3YljyncUmMMU4mJH0cC1h3C7/vBNiuj8gc+CET6lAtvIj+ B0rRkZKgo9rTGhVCsF8K3zoLc97LfA1oI+BbuD6RBBaiXUzETtqHpj4VfgWAP8AiQxQANI1G5Te POrjZjS6GpNberngBWQVdEe4iQkBd+SULYX9T/9MjtMXJJ9MrgrgKxngmIpx5bre3bDrpTUwtXQ vmAYUEWAfmx4UlHk6usXOJJIcAdmm9MquxvPnUqeZ/RUtibabaKgfliBtTW7WPppZTQBrWSDBb5 SITsDa3rc7WJXioC0Mpq34HB1+N13G7Wj2k5hVQrgtK/W9Az+kc+LaEYE9KyBT8mjM/Z/CZ3/mW nuUMdh3GmEU5YWvmMU6Km7Uw4tbaBuhmz6KYM6TqzOk81xQ3iM10VspSBXtMYqD20Ba4GPoL2CV 3GN869WA== X-Google-Smtp-Source: AGHT+IGJtGbQKqjQQraafnEPYsifVEhCCTSQB6405/NVyNep2ZbFUIZS5bLvgZevnFtTokkmDpwugw== X-Received: by 2002:a05:600c:1c8a:b0:477:54f9:6ac2 with SMTP id 5b1f17b1804b1-47786ff784dmr37142965e9.0.1762966425256; Wed, 12 Nov 2025 08:53:45 -0800 (PST) Received: from localhost ([2a01:4b00:d036:ae00:a397:14bc:5982:5745]) by smtp.gmail.com with UTF8SMTPSA id ffacd0b85a97d-42abe62bf40sm33736161f8f.9.2025.11.12.08.53.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 12 Nov 2025 08:53:44 -0800 (PST) From: luca.boccassi@gmail.com To: Thomas Monjalon Cc: dpdk stable Subject: patch 'net/mlx5: remove useless constants' has been queued to stable release 22.11.11 Date: Wed, 12 Nov 2025 16:52:25 +0000 Message-ID: <20251112165308.1618107-11-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20251112165308.1618107-1-luca.boccassi@gmail.com> References: <20251027162001.3710450-79-luca.boccassi@gmail.com> <20251112165308.1618107-1-luca.boccassi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Hi, FYI, your patch has been queued to stable release 22.11.11 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 11/14/25. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/bluca/dpdk-stable This queued commit can be viewed at: https://github.com/bluca/dpdk-stable/commit/c497e8ae3655414f0fd624a4eefeb8b9d46fedc9 Thanks. Luca Boccassi --- >From c497e8ae3655414f0fd624a4eefeb8b9d46fedc9 Mon Sep 17 00:00:00 2001 From: Thomas Monjalon Date: Fri, 31 Oct 2025 21:38:26 +0100 Subject: [PATCH] net/mlx5: remove useless constants [ upstream commit 9d3788534cba7cc3424e2a83db6b186211d4e94f ] - MLX5_VEC_TX_CKSUM_OFFLOAD_CAP became unused when Tx metadata moved to a dynamic field. - MLX5_ALARM_TIMEOUT_US became unused when reworking link status wait. - MLX5_FLOW_SFT_HLIST_ARRAY_SIZE was never used in hash list handling. - MLX5_FLOW_MIN_ID_POOL_SIZE and MLX5_ID_GENERATION_ARRAY_FACTOR became unused when the flow ID generator was replaced by ipool. - MLX5_RSSQ_DEFAULT_NUM was used in the old per-thread flow workspace. - MLX5_FLOW_TABLE_HWS_POLICY became unused because meter is not supported in HWS. - IPPROTO_MPLS became unused because MPLS over IP is not supported. - MLX5_L4_RSS_TYPES has never been used. - ETH_TYPE_IPV4/V6_VXLAN were never used in HWS. - PCI_DRV_FLAGS became unused when migrating to bus-agnostic probing. Fixes: 9bf26e1318e3 ("ethdev: move egress metadata to dynamic field") Fixes: cfee94752b8f ("net/mlx5: fix link status to use wait to complete") Fixes: f3020a331dca ("net/mlx5: optimize hash list table allocate on demand") Fixes: 4ae8825c5085 ("net/mlx5: use indexed pool as id generator") Fixes: dc7c5e0aa905 ("net/mlx5: fix flow workspace destruction") Fixes: 645f240d1cd5 ("net/mlx5: remove unsupported flow meter action in HWS") Fixes: 14ad99d78a46 ("net/mlx5: remove unsupported flow item MPLS over IP") Fixes: ae67e3c43dd5 ("net/mlx5: support RSS expansion in non-template HWS setup") Fixes: c55c2bf35333 ("net/mlx5/hws: add definer layer") Fixes: a7f34989e9ad ("net/mlx5: migrate to bus-agnostic common interface") Signed-off-by: Thomas Monjalon --- drivers/net/mlx5/hws/mlx5dr_definer.c | 2 -- drivers/net/mlx5/mlx5.c | 3 --- drivers/net/mlx5/mlx5.h | 1 - drivers/net/mlx5/mlx5_defs.h | 6 ------ drivers/net/mlx5/mlx5_flow.h | 7 ------- drivers/net/mlx5/mlx5_rxtx_vec.h | 7 ------- drivers/net/mlx5/windows/mlx5_os.h | 2 -- 7 files changed, 28 deletions(-) diff --git a/drivers/net/mlx5/hws/mlx5dr_definer.c b/drivers/net/mlx5/hws/mlx5dr_definer.c index 527217487e..8770492d00 100644 --- a/drivers/net/mlx5/hws/mlx5dr_definer.c +++ b/drivers/net/mlx5/hws/mlx5dr_definer.c @@ -6,8 +6,6 @@ #define GTP_PDU_SC 0x85 #define BAD_PORT 0xBAD -#define ETH_TYPE_IPV4_VXLAN 0x0800 -#define ETH_TYPE_IPV6_VXLAN 0x86DD #define UDP_VXLAN_PORT 4789 #define STE_NO_VLAN 0x0 diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c index 357f5eac11..674861d765 100644 --- a/drivers/net/mlx5/mlx5.c +++ b/drivers/net/mlx5/mlx5.c @@ -384,9 +384,6 @@ static const struct mlx5_indexed_pool_config mlx5_ipool_cfg[] = { }, }; -#define MLX5_FLOW_MIN_ID_POOL_SIZE 512 -#define MLX5_ID_GENERATION_ARRAY_FACTOR 16 - #define MLX5_FLOW_TABLE_HLIST_ARRAY_SIZE 1024 /** diff --git a/drivers/net/mlx5/mlx5.h b/drivers/net/mlx5/mlx5.h index ad90b090cf..73d6bb6ad9 100644 --- a/drivers/net/mlx5/mlx5.h +++ b/drivers/net/mlx5/mlx5.h @@ -1149,7 +1149,6 @@ struct mlx5_flow_tbl_resource { #define MLX5_FLOW_TABLE_LEVEL_METER (MLX5_MAX_TABLES - 3) #define MLX5_FLOW_TABLE_LEVEL_POLICY (MLX5_MAX_TABLES - 4) #define MLX5_MAX_TABLES_EXTERNAL MLX5_FLOW_TABLE_LEVEL_POLICY -#define MLX5_FLOW_TABLE_HWS_POLICY (MLX5_MAX_TABLES - 10) #define MLX5_MAX_TABLES_FDB UINT16_MAX #define MLX5_FLOW_TABLE_FACTOR 10 diff --git a/drivers/net/mlx5/mlx5_defs.h b/drivers/net/mlx5/mlx5_defs.h index 2af8c731ef..f101b3b438 100644 --- a/drivers/net/mlx5/mlx5_defs.h +++ b/drivers/net/mlx5/mlx5_defs.h @@ -46,9 +46,6 @@ /* Maximum number of DCS created per port. */ #define MLX5_HWS_CNT_DCS_NUM 4 -/* Alarm timeout. */ -#define MLX5_ALARM_TIMEOUT_US 100000 - /* Maximum number of extended statistics counters. */ #define MLX5_MAX_XSTATS 64 @@ -170,9 +167,6 @@ /* Size of the hash table for tag table. */ #define MLX5_TAGS_HLIST_ARRAY_SIZE (1 << 15) -/* Size fo the hash table for SFT table. */ -#define MLX5_FLOW_SFT_HLIST_ARRAY_SIZE 4096 - /* Hairpin TX/RX queue configuration parameters. */ #define MLX5_HAIRPIN_QUEUE_STRIDE 6 #define MLX5_HAIRPIN_JUMBO_LOG_SIZE (14 + 2) diff --git a/drivers/net/mlx5/mlx5_flow.h b/drivers/net/mlx5/mlx5_flow.h index fc0b569505..39d7864f93 100644 --- a/drivers/net/mlx5/mlx5_flow.h +++ b/drivers/net/mlx5/mlx5_flow.h @@ -144,9 +144,6 @@ enum mlx5_feature_name { MLX5_SAMPLE_ID, }; -/* Default queue number. */ -#define MLX5_RSSQ_DEFAULT_NUM 16 - #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) @@ -353,10 +350,6 @@ enum mlx5_feature_name { #define MLX5_FLOW_XCAP_ACTIONS (MLX5_FLOW_ACTION_ENCAP | MLX5_FLOW_ACTION_DECAP) -#ifndef IPPROTO_MPLS -#define IPPROTO_MPLS 137 -#endif - /* UDP port number for MPLS */ #define MLX5_UDP_PORT_MPLS 6635 diff --git a/drivers/net/mlx5/mlx5_rxtx_vec.h b/drivers/net/mlx5/mlx5_rxtx_vec.h index 77c3f4efa0..672e301f77 100644 --- a/drivers/net/mlx5/mlx5_rxtx_vec.h +++ b/drivers/net/mlx5/mlx5_rxtx_vec.h @@ -13,13 +13,6 @@ #include "mlx5_autoconf.h" -/* HW checksum offload capabilities of vectorized Tx. */ -#define MLX5_VEC_TX_CKSUM_OFFLOAD_CAP \ - (RTE_ETH_TX_OFFLOAD_IPV4_CKSUM | \ - RTE_ETH_TX_OFFLOAD_UDP_CKSUM | \ - RTE_ETH_TX_OFFLOAD_TCP_CKSUM | \ - RTE_ETH_TX_OFFLOAD_OUTER_IPV4_CKSUM) - /* * Compile time sanity check for vectorized functions. */ diff --git a/drivers/net/mlx5/windows/mlx5_os.h b/drivers/net/mlx5/windows/mlx5_os.h index 8b58265687..7085e9b258 100644 --- a/drivers/net/mlx5/windows/mlx5_os.h +++ b/drivers/net/mlx5/windows/mlx5_os.h @@ -12,8 +12,6 @@ enum { MLX5_FS_PATH_MAX = MLX5_DEVX_DEVICE_PNP_SIZE + 1 }; -#define PCI_DRV_FLAGS 0 - #define MLX5_NAMESIZE MLX5_FS_NAME_MAX enum mlx5_sw_parsing_offloads { -- 2.47.3 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2025-11-12 16:20:41.392310646 +0000 +++ 0011-net-mlx5-remove-useless-constants.patch 2025-11-12 16:20:40.891716059 +0000 @@ -1 +1 @@ -From 9d3788534cba7cc3424e2a83db6b186211d4e94f Mon Sep 17 00:00:00 2001 +From c497e8ae3655414f0fd624a4eefeb8b9d46fedc9 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 9d3788534cba7cc3424e2a83db6b186211d4e94f ] + @@ -30 +31,0 @@ -Cc: stable@dpdk.org @@ -34,8 +35,8 @@ - drivers/net/mlx5/hws/mlx5dr_definer.c | 2 -- - drivers/net/mlx5/mlx5.c | 3 --- - drivers/net/mlx5/mlx5.h | 1 - - drivers/net/mlx5/mlx5_defs.h | 6 ------ - drivers/net/mlx5/mlx5_flow.h | 10 ---------- - drivers/net/mlx5/mlx5_rxtx_vec.h | 7 ------- - drivers/net/mlx5/windows/mlx5_os.h | 2 -- - 7 files changed, 31 deletions(-) + drivers/net/mlx5/hws/mlx5dr_definer.c | 2 -- + drivers/net/mlx5/mlx5.c | 3 --- + drivers/net/mlx5/mlx5.h | 1 - + drivers/net/mlx5/mlx5_defs.h | 6 ------ + drivers/net/mlx5/mlx5_flow.h | 7 ------- + drivers/net/mlx5/mlx5_rxtx_vec.h | 7 ------- + drivers/net/mlx5/windows/mlx5_os.h | 2 -- + 7 files changed, 28 deletions(-) @@ -44 +45 @@ -index 1c6b3e38c4..afa70bf793 100644 +index 527217487e..8770492d00 100644 @@ -47 +48,2 @@ -@@ -9,8 +9,6 @@ +@@ -6,8 +6,6 @@ + @@ -50 +51,0 @@ - #define BAD_SQN 0xBAD @@ -54,2 +55,2 @@ - #define UDP_VXLAN_GPE_PORT 4790 - #define UDP_GTPU_PORT 2152 + + #define STE_NO_VLAN 0x0 @@ -57 +58 @@ -index b018a4f0e2..4705c30801 100644 +index 357f5eac11..674861d765 100644 @@ -60 +61 @@ -@@ -394,9 +394,6 @@ static const struct mlx5_indexed_pool_config mlx5_ipool_cfg[] = { +@@ -384,9 +384,6 @@ static const struct mlx5_indexed_pool_config mlx5_ipool_cfg[] = { @@ -69 +70 @@ - #define MLX5_RXQ_ENH_CQE_COMP_MASK 0x80 + /** @@ -71 +72 @@ -index 07418b0922..233cb416cb 100644 +index ad90b090cf..73d6bb6ad9 100644 @@ -74 +75 @@ -@@ -1259,7 +1259,6 @@ struct mlx5_flow_tbl_resource { +@@ -1149,7 +1149,6 @@ struct mlx5_flow_tbl_resource { @@ -80,2 +81,2 @@ - #define MLX5_FLOW_TABLE_PTYPE_RSS_NUM 1024 - #define MLX5_FLOW_TABLE_PTYPE_RSS_LAST (MLX5_MAX_TABLES - 11) + #define MLX5_FLOW_TABLE_FACTOR 10 + @@ -83 +84 @@ -index d326fec000..b8e5122323 100644 +index 2af8c731ef..f101b3b438 100644 @@ -107 +108 @@ -index ff61706054..1dca6e0532 100644 +index fc0b569505..39d7864f93 100644 @@ -110,2 +111,2 @@ -@@ -222,9 +222,6 @@ struct mlx5_mirror { - struct mlx5_mirror_clone clone[MLX5_MIRROR_MAX_CLONES_NUM]; +@@ -144,9 +144,6 @@ enum mlx5_feature_name { + MLX5_SAMPLE_ID, @@ -120 +121 @@ -@@ -469,10 +466,6 @@ struct mlx5_mirror { +@@ -353,10 +350,6 @@ enum mlx5_feature_name { @@ -128,12 +129,2 @@ - #define MLX5_IPV6_HDR_ECN_MASK 0x3 - #define MLX5_IPV6_HDR_DSCP_SHIFT 2 - -@@ -519,9 +512,6 @@ struct mlx5_mirror { - RTE_ETH_RSS_NONFRAG_IPV4_TCP | RTE_ETH_RSS_NONFRAG_IPV4_UDP | \ - RTE_ETH_RSS_NONFRAG_IPV4_OTHER) - --/* Valid L4 RSS types */ --#define MLX5_L4_RSS_TYPES (RTE_ETH_RSS_L4_SRC_ONLY | RTE_ETH_RSS_L4_DST_ONLY) -- - /* IBV hash source bits for IPV4. */ - #define MLX5_IPV4_IBV_RX_HASH (IBV_RX_HASH_SRC_IPV4 | IBV_RX_HASH_DST_IPV4) + /* UDP port number for MPLS */ + #define MLX5_UDP_PORT_MPLS 6635 @@ -160 +151 @@ -index fb7198c244..57d147bd8c 100644 +index 8b58265687..7085e9b258 100644 @@ -163,3 +154,3 @@ -@@ -7,8 +7,6 @@ - - #include "mlx5_win_ext.h" +@@ -12,8 +12,6 @@ enum { + MLX5_FS_PATH_MAX = MLX5_DEVX_DEVICE_PNP_SIZE + 1 + };