patch 'test/crypto: remove unused variable' has been queued to stable release 23.11.4

Xueming Li xuemingl at nvidia.com
Tue Apr 8 10:01:51 CEST 2025


Hi,

FYI, your patch has been queued to stable release 23.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 04/10/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://git.dpdk.org/dpdk-stable/log/?h=23.11-staging

This queued commit can be viewed at:
https://git.dpdk.org/dpdk-stable/commit/?h=23.11-staging&id=e94b87d237b1157a64517d2024f9a5fac289f713

Thanks.

Xueming Li <xuemingl at nvidia.com>

---
>From e94b87d237b1157a64517d2024f9a5fac289f713 Mon Sep 17 00:00:00 2001
From: Shaokai Zhang <felix.zhang at jaguarmicro.com>
Date: Thu, 20 Feb 2025 21:28:07 +0800
Subject: [PATCH] test/crypto: remove unused variable
Cc: Xueming Li <xuemingl at nvidia.com>

[ upstream commit dafeb6ae229bc572fc809e62e4e8e242b08ca2f7 ]

In the new session framework, each crypto session no longer requires
the session_priv_mpool, and it is no longer referenced during testsuite
setup. However, the structure `struct crypto_testsuite_params` still
contains the session_priv_mpool member, which may lead to
the misunderstanding that session_priv_mpool still needs initialization.

Fixes: bdce2564dbf7 ("cryptodev: rework session framework")
Fixes: 3f3fc3308bd0 ("security: remove private mempool usage")

Signed-off-by: Shaokai Zhang <felix.zhang at jaguarmicro.com>
Reviewed-by: Joey Xing <joey.xing at jaguarmicro.com>
Acked-by: Akhil Goyal <gakhil at marvell.com>
---
 app/test/test_cryptodev.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/app/test/test_cryptodev.h b/app/test/test_cryptodev.h
index f2c417a267..feee8c3f42 100644
--- a/app/test/test_cryptodev.h
+++ b/app/test/test_cryptodev.h
@@ -90,7 +90,6 @@ struct crypto_testsuite_params {
 	struct rte_mempool *large_mbuf_pool;
 	struct rte_mempool *op_mpool;
 	struct rte_mempool *session_mpool;
-	struct rte_mempool *session_priv_mpool;
 	struct rte_cryptodev_config conf;
 	struct rte_cryptodev_qp_conf qp_conf;
 
-- 
2.34.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2025-04-08 15:39:06.767271009 +0800
+++ 0027-test-crypto-remove-unused-variable.patch	2025-04-08 15:39:05.946436575 +0800
@@ -1 +1 @@
-From dafeb6ae229bc572fc809e62e4e8e242b08ca2f7 Mon Sep 17 00:00:00 2001
+From e94b87d237b1157a64517d2024f9a5fac289f713 Mon Sep 17 00:00:00 2001
@@ -4,0 +5,3 @@
+Cc: Xueming Li <xuemingl at nvidia.com>
+
+[ upstream commit dafeb6ae229bc572fc809e62e4e8e242b08ca2f7 ]
@@ -14 +16,0 @@
-Cc: stable at dpdk.org
@@ -24 +26 @@
-index bb54a33d62..190a220c73 100644
+index f2c417a267..feee8c3f42 100644
@@ -27 +29 @@
-@@ -87,7 +87,6 @@ struct crypto_testsuite_params {
+@@ -90,7 +90,6 @@ struct crypto_testsuite_params {


More information about the stable mailing list