[PATCH v1] test/event_crypto_adapter: fix runtime parameter test

Shijith Thotton sthotton at marvell.com
Mon Apr 17 19:52:14 CEST 2023


The runtime parameters for the crypto adapter are applicable only for
devices without an internal port. To accommodate this, the UT has been
updated to skip the test for checking the runtime parameters if the API
to set runtime parameters is not supported.

Fixes: 04ed18cd41f1 ("eventdev/crypto: support runtime set/get parameters")

Signed-off-by: Shijith Thotton <sthotton at marvell.com>
---
 app/test/test_event_crypto_adapter.c | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/app/test/test_event_crypto_adapter.c b/app/test/test_event_crypto_adapter.c
index 264d6f731e..24f66be69f 100644
--- a/app/test/test_event_crypto_adapter.c
+++ b/app/test/test_event_crypto_adapter.c
@@ -275,7 +275,7 @@ test_crypto_adapter_stats(void)
 static int
 test_crypto_adapter_params(void)
 {
-	int err;
+	int err, rc;
 	struct rte_event_crypto_adapter_runtime_params in_params;
 	struct rte_event_crypto_adapter_runtime_params out_params;
 	uint32_t cap;
@@ -303,6 +303,10 @@ test_crypto_adapter_params(void)
 	/* Case 1: Get the default value of mbufs processed by adapter */
 	err = rte_event_crypto_adapter_runtime_params_get(TEST_ADAPTER_ID,
 							  &out_params);
+	if (err == -ENOTSUP) {
+		rc = TEST_SKIPPED;
+		goto queue_pair_del;
+	}
 	TEST_ASSERT(err == 0, "Expected 0 got %d", err);
 
 	/* Case 2: Set max_nb = 32 (=BATCH_SEIZE) */
@@ -370,11 +374,13 @@ test_crypto_adapter_params(void)
 	TEST_ASSERT(in_params.max_nb == out_params.max_nb, "Expected %u got %u",
 		    in_params.max_nb, out_params.max_nb);
 
+	rc = TEST_SUCCESS;
+queue_pair_del:
 	err = rte_event_crypto_adapter_queue_pair_del(TEST_ADAPTER_ID,
 					TEST_CDEV_ID, TEST_CDEV_QP_ID);
 	TEST_ASSERT_SUCCESS(err, "Failed to delete add queue pair\n");
 
-	return TEST_SUCCESS;
+	return rc;
 }
 
 static int
-- 
2.25.1



More information about the dev mailing list