patches for DPDK stable branches
 help / color / mirror / Atom feed
* [PATCH] test/crypto: fix return value for GMAC testcase
@ 2023-08-28  6:51 Hemant Agrawal
  2023-09-19 19:22 ` [EXT] " Akhil Goyal
  0 siblings, 1 reply; 2+ messages in thread
From: Hemant Agrawal @ 2023-08-28  6:51 UTC (permalink / raw)
  To: dev; +Cc: Hemant Agrawal, gakhil, stable

create_gmac_session is already converting the return value
of NOTSUPP to TEST_SKIPPED

Fixes: bdce2564dbf7 ("cryptodev: rework session framework")
Cc: gakhil@marvell.com
Cc: stable@dpdk.org

Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
---
 app/test/test_cryptodev.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index 956268bfcd..068e47a972 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -13538,7 +13538,7 @@ test_AES_GMAC_authentication(const struct gmac_test_data *tdata)
 	retval = create_gmac_session(ts_params->valid_devs[0],
 			tdata, RTE_CRYPTO_AUTH_OP_GENERATE);
 
-	if (retval == -ENOTSUP)
+	if (retval == TEST_SKIPPED)
 		return TEST_SKIPPED;
 	if (retval < 0)
 		return retval;
@@ -13671,7 +13671,7 @@ test_AES_GMAC_authentication_verify(const struct gmac_test_data *tdata)
 	retval = create_gmac_session(ts_params->valid_devs[0],
 			tdata, RTE_CRYPTO_AUTH_OP_VERIFY);
 
-	if (retval == -ENOTSUP)
+	if (retval == TEST_SKIPPED)
 		return TEST_SKIPPED;
 	if (retval < 0)
 		return retval;
@@ -13802,7 +13802,7 @@ test_AES_GMAC_authentication_SGL(const struct gmac_test_data *tdata,
 	retval = create_gmac_session(ts_params->valid_devs[0],
 			tdata, RTE_CRYPTO_AUTH_OP_GENERATE);
 
-	if (retval == -ENOTSUP)
+	if (retval == TEST_SKIPPED)
 		return TEST_SKIPPED;
 	if (retval < 0)
 		return retval;
@@ -14419,7 +14419,7 @@ test_authentication_verify_fail_when_data_corruption(
 			reference,
 			RTE_CRYPTO_AUTH_OP_VERIFY);
 
-	if (retval == -ENOTSUP)
+	if (retval == TEST_SKIPPED)
 		return TEST_SKIPPED;
 	if (retval < 0)
 		return retval;
@@ -14508,6 +14508,8 @@ test_authentication_verify_GMAC_fail_when_corruption(
 			reference,
 			RTE_CRYPTO_AUTH_OP_VERIFY,
 			RTE_CRYPTO_CIPHER_OP_DECRYPT);
+	if (retval == TEST_SKIPPED)
+		return TEST_SKIPPED;
 	if (retval < 0)
 		return retval;
 
@@ -14600,8 +14602,7 @@ test_authenticated_decryption_fail_when_corruption(
 			reference,
 			RTE_CRYPTO_AUTH_OP_VERIFY,
 			RTE_CRYPTO_CIPHER_OP_DECRYPT);
-
-	if (retval == -ENOTSUP)
+	if (retval == TEST_SKIPPED)
 		return TEST_SKIPPED;
 	if (retval < 0)
 		return retval;
-- 
2.25.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [EXT] [PATCH] test/crypto: fix return value for GMAC testcase
  2023-08-28  6:51 [PATCH] test/crypto: fix return value for GMAC testcase Hemant Agrawal
@ 2023-09-19 19:22 ` Akhil Goyal
  0 siblings, 0 replies; 2+ messages in thread
From: Akhil Goyal @ 2023-09-19 19:22 UTC (permalink / raw)
  To: Hemant Agrawal, dev; +Cc: stable

> create_gmac_session is already converting the return value
> of NOTSUPP to TEST_SKIPPED
> 
> Fixes: bdce2564dbf7 ("cryptodev: rework session framework")
> Cc: gakhil@marvell.com
> Cc: stable@dpdk.org
> 
> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
Acked-by: Akhil Goyal <gakhil@marvell.com>

Applied to dpdk-next-crypto
Thanks.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-09-19 19:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-28  6:51 [PATCH] test/crypto: fix return value for GMAC testcase Hemant Agrawal
2023-09-19 19:22 ` [EXT] " Akhil Goyal

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).