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 09F4D48B69 for ; Fri, 21 Nov 2025 12:22:24 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0466E402E0; Fri, 21 Nov 2025 12:22:24 +0100 (CET) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mails.dpdk.org (Postfix) with ESMTP id 70215402D7 for ; Fri, 21 Nov 2025 12:22:23 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1763724143; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Ip862ENeC4pppSRgaykebreYSlLbWG9S+ChEDLiUObA=; b=OH2cgmmU4wwLoYKdW7/SRKspg0+dK6tRYO6VW70vMntNh2OF83QP/Hjmq8uGy467sHhgwq vxFaEJ/2Olf6Xzk0mL2dU8vZZoqWqXnHOuoh0B8EbCSsvKjpOBGpHgZYL9YbILGA3rNsJs sFnm2unmuyINSgl8eJR6YtFbyACqq+M= Received: from mx-prod-mc-06.mail-002.prod.us-west-2.aws.redhat.com (ec2-35-165-154-97.us-west-2.compute.amazonaws.com [35.165.154.97]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-384-7b36JJEZMxGNHkyh7g8BxQ-1; Fri, 21 Nov 2025 06:22:21 -0500 X-MC-Unique: 7b36JJEZMxGNHkyh7g8BxQ-1 X-Mimecast-MFC-AGG-ID: 7b36JJEZMxGNHkyh7g8BxQ_1763724140 Received: from mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-06.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 2FD9E180057A; Fri, 21 Nov 2025 11:22:20 +0000 (UTC) Received: from rh.redhat.com (unknown [10.42.28.165]) by mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 10FB230044DB; Fri, 21 Nov 2025 11:22:18 +0000 (UTC) From: Kevin Traynor To: Thomas Monjalon Cc: dpdk stable Subject: patch 'regex/mlx5: remove useless constants' has been queued to stable release 24.11.4 Date: Fri, 21 Nov 2025 11:19:57 +0000 Message-ID: <20251121112128.485623-13-ktraynor@redhat.com> In-Reply-To: <20251121112128.485623-1-ktraynor@redhat.com> References: <20251121112128.485623-1-ktraynor@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.30.177.4 X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: PQHo9i0iFAJrqYc_N4AKMf4NvWVLPrKKZln90xkRU04_1763724140 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit content-type: text/plain; charset="US-ASCII"; x-default=true 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 24.11.4 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/26/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/kevintraynor/dpdk-stable This queued commit can be viewed at: https://github.com/kevintraynor/dpdk-stable/commit/3dc350e47dcb3f900eada6b8e93d6754ad1222a6 Thanks. Kevin --- >From 3dc350e47dcb3f900eada6b8e93d6754ad1222a6 Mon Sep 17 00:00:00 2001 From: Thomas Monjalon Date: Fri, 31 Oct 2025 21:37:54 +0100 Subject: [PATCH] regex/mlx5: remove useless constants [ upstream commit 9f6da93ec89a60df71f8e4d4748dff108a395595 ] All these constants were unused since their introduction: - MLX5_REGEX_WQE_CTRL_OFFSET - MLX5_REGEX_RXP_ROF2_LINE_LEN - MLX5_RXP_BF4_ROF_VERSION_STRING - MLX5_RXP_CTRL_TYPE_* - MLX5_RXP_CTRL_JOB_DESC_FLAGS - MLX5_RXP_CTRL_VALID - MLX5_RXP_INITIALIZATION_TIMEOUT This constant became unused after supporting combined rule file: - MLX5_RXP_POLL_CSR_FOR_VALUE_TIMEOUT All these constants became unused after removing DB and register R/W: - MLX5_RXP_MAX_ENGINES - MLX5_RXP_EM_COUNT - MLX5_RXP_DB_NOT_ASSIGNED - MLX5_RXP_CSR_NUM_ENTRIES - MLX5_RXP_CSR_CTRL_DISABLE_L2C Fixes: 4d4e245ad637 ("regex/mlx5: support enqueue") Fixes: f324162e8e77 ("regex/mlx5: support combined rule file") Fixes: ab74680160ba ("regex/mlx5: support combined ROF file") Fixes: b34d816363b5 ("regex/mlx5: support rules import") Fixes: ab2e0b0d3531 ("regex/mlx5: remove register read/write") Fixes: 9fa82d287f65 ("regex/mlx5: move RXP to CrSpace") Signed-off-by: Thomas Monjalon --- drivers/regex/mlx5/mlx5_regex_fastpath.c | 1 - drivers/regex/mlx5/mlx5_rxp.c | 2 -- drivers/regex/mlx5/mlx5_rxp.h | 20 -------------------- 3 files changed, 23 deletions(-) diff --git a/drivers/regex/mlx5/mlx5_regex_fastpath.c b/drivers/regex/mlx5/mlx5_regex_fastpath.c index 8e5f8c9c95..6e21574d29 100644 --- a/drivers/regex/mlx5/mlx5_regex_fastpath.c +++ b/drivers/regex/mlx5/mlx5_regex_fastpath.c @@ -28,5 +28,4 @@ #define MLX5_REGEX_METADATA_SIZE ((size_t)64) #define MLX5_REGEX_MAX_OUTPUT (((size_t)1) << 11) -#define MLX5_REGEX_WQE_CTRL_OFFSET 12 #define MLX5_REGEX_WQE_METADATA_OFFSET 16 #define MLX5_REGEX_WQE_GATHER_OFFSET 32 diff --git a/drivers/regex/mlx5/mlx5_rxp.c b/drivers/regex/mlx5/mlx5_rxp.c index 7730e11fb1..dda4a7fdb0 100644 --- a/drivers/regex/mlx5/mlx5_rxp.c +++ b/drivers/regex/mlx5/mlx5_rxp.c @@ -25,6 +25,4 @@ #define MLX5_REGEX_MAX_GROUPS MLX5_RXP_MAX_SUBSETS -#define MLX5_REGEX_RXP_ROF2_LINE_LEN 34 - const uint64_t combined_rof_tag = 0xff52544424a52475; diff --git a/drivers/regex/mlx5/mlx5_rxp.h b/drivers/regex/mlx5/mlx5_rxp.h index b38b53cc14..1f982f3b61 100644 --- a/drivers/regex/mlx5/mlx5_rxp.h +++ b/drivers/regex/mlx5/mlx5_rxp.h @@ -10,14 +10,6 @@ #define MLX5_RXP_MAX_JOB_LENGTH 16384 #define MLX5_RXP_MAX_SUBSETS 4095 -#define MLX5_RXP_CSR_NUM_ENTRIES 31 #define MLX5_RXP_BF2_ROF_VERSION_STRING 0x07055254 #define MLX5_RXP_BF3_ROF_VERSION_STRING 0x00065254 -#define MLX5_RXP_BF4_ROF_VERSION_STRING 0x00075254 - -#define MLX5_RXP_CTRL_TYPE_MASK 7 -#define MLX5_RXP_CTRL_TYPE_JOB_DESCRIPTOR 0 -#define MLX5_RXP_CTRL_TYPE_RESPONSE_DESCRIPTOR 1 -#define MLX5_RXP_CTRL_TYPE_MEMORY_WRITE 4 -#define MLX5_RXP_CSR_CTRL_DISABLE_L2C (1 << 7) #define MLX5_RXP_CTRL_JOB_DESC_SOF 0x0010 @@ -25,10 +17,4 @@ #define MLX5_RXP_CTRL_JOB_DESC_HPM_ENABLE 0x0100 #define MLX5_RXP_CTRL_JOB_DESC_ANYMATCH_ENABLE 0x0200 -#define MLX5_RXP_CTRL_JOB_DESC_FLAGS (MLX5_RXP_CTRL_JOB_DESC_SOF | \ - MLX5_RXP_CTRL_JOB_DESC_EOF | \ - MLX5_RXP_CTRL_JOB_DESC_HPM_ENABLE | \ - MLX5_RXP_CTRL_JOB_DESC_ANYMATCH_ENABLE) - -#define MLX5_RXP_CTRL_VALID 0x8000 #define MLX5_RXP_RESP_STATUS_MAX_PRI_THREADS (1 << 3) @@ -129,10 +115,4 @@ enum mlx5_rxp_program_mode { }; -#define MLX5_RXP_POLL_CSR_FOR_VALUE_TIMEOUT 3000 /* Poll timeout in ms. */ -#define MLX5_RXP_INITIALIZATION_TIMEOUT 60000 /* Initialize timeout in ms. */ -#define MLX5_RXP_MAX_ENGINES 2u /* Number of RXP engines. */ -#define MLX5_RXP_EM_COUNT 1u /* Extra External Memories to use. */ -#define MLX5_RXP_DB_NOT_ASSIGNED 0xFF - struct mlx5_regex_mkey { struct mlx5dv_devx_umem *umem; -- 2.51.0 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2025-11-21 11:05:10.037293872 +0000 +++ 0013-regex-mlx5-remove-useless-constants.patch 2025-11-21 11:05:09.372200781 +0000 @@ -1 +1 @@ -From 9f6da93ec89a60df71f8e4d4748dff108a395595 Mon Sep 17 00:00:00 2001 +From 3dc350e47dcb3f900eada6b8e93d6754ad1222a6 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 9f6da93ec89a60df71f8e4d4748dff108a395595 ] + @@ -34 +35,0 @@ -Cc: stable@dpdk.org @@ -65 +66 @@ -index 08b9a2680b..8ef1541d1a 100644 +index b38b53cc14..1f982f3b61 100644