[resend,v5,6/6] test/crypto: add cryptodev_uadk_autotest

Message ID 20221024134409.1896776-7-zhangfei.gao@linaro.org (mailing list archive)
State Changes Requested, archived
Delegated to: akhil goyal
Headers
Series crypto/uadk: introduce uadk crypto driver |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/github-robot: build success github build: passed
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS

Commit Message

Zhangfei Gao Oct. 24, 2022, 1:44 p.m. UTC
  Example:
sudo dpdk-test --vdev=crypto_uadk --log-level=6
RTE>>cryptodev_uadk_autotest
RTE>>quit

Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org>
---
 app/test/test_cryptodev.c | 7 +++++++
 app/test/test_cryptodev.h | 1 +
 2 files changed, 8 insertions(+)
  

Comments

Akhil Goyal Oct. 25, 2022, 2:54 p.m. UTC | #1
> Subject: [EXT] [PATCH resend v5 6/6] test/crypto: add cryptodev_uadk_autotest
Rewrite patch title as
test/crypto: support uadk PMD

> 
Add a line
Updated test application to run autotest for uadk crypto PMD.
> Example:
> sudo dpdk-test --vdev=crypto_uadk --log-level=6
> RTE>>cryptodev_uadk_autotest
> RTE>>quit

Add this information in uadk.rst as well.

> 
> Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org>
> ---
>  app/test/test_cryptodev.c | 7 +++++++
>  app/test/test_cryptodev.h | 1 +
>  2 files changed, 8 insertions(+)
> 
> diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
> index c2b33686ed..183e24f150 100644
> --- a/app/test/test_cryptodev.c
> +++ b/app/test/test_cryptodev.c
> @@ -16467,6 +16467,12 @@ test_cryptodev_qat(void)
>  	return
> run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_QAT_SYM_PMD));
>  }
> 
> +static int
> +test_cryptodev_uadk(void)
> +{
> +	return
> run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_UADK_PMD));
> +}
> +
>  static int
>  test_cryptodev_virtio(void)
>  {
> @@ -16810,6 +16816,7 @@
> REGISTER_TEST_COMMAND(cryptodev_sw_mvsam_autotest,
> test_cryptodev_mrvl);
>  REGISTER_TEST_COMMAND(cryptodev_dpaa2_sec_autotest,
> test_cryptodev_dpaa2_sec);
>  REGISTER_TEST_COMMAND(cryptodev_dpaa_sec_autotest,
> test_cryptodev_dpaa_sec);
>  REGISTER_TEST_COMMAND(cryptodev_ccp_autotest, test_cryptodev_ccp);
> +REGISTER_TEST_COMMAND(cryptodev_uadk_autotest, test_cryptodev_uadk);
>  REGISTER_TEST_COMMAND(cryptodev_virtio_autotest, test_cryptodev_virtio);
>  REGISTER_TEST_COMMAND(cryptodev_octeontx_autotest,
> test_cryptodev_octeontx);
>  REGISTER_TEST_COMMAND(cryptodev_caam_jr_autotest,
> test_cryptodev_caam_jr);
> diff --git a/app/test/test_cryptodev.h b/app/test/test_cryptodev.h
> index 29a7d4db2b..abd795f54a 100644
> --- a/app/test/test_cryptodev.h
> +++ b/app/test/test_cryptodev.h
> @@ -74,6 +74,7 @@
>  #define CRYPTODEV_NAME_CN9K_PMD		crypto_cn9k
>  #define CRYPTODEV_NAME_CN10K_PMD	crypto_cn10k
>  #define CRYPTODEV_NAME_MLX5_PMD		crypto_mlx5
> +#define CRYPTODEV_NAME_UADK_PMD		crypto_uadk
> 
> 
Add the newly added autotest in "app/test/meson.build" in driver_test_names[]
  
Zhangfei Gao Oct. 26, 2022, 9:57 a.m. UTC | #2
On Tue, 25 Oct 2022 at 22:57, Akhil Goyal <gakhil@marvell.com> wrote:
>
> > Subject: [EXT] [PATCH resend v5 6/6] test/crypto: add cryptodev_uadk_autotest
> Rewrite patch title as
> test/crypto: support uadk PMD
>
> >
> Add a line
> Updated test application to run autotest for uadk crypto PMD.
> > Example:
> > sudo dpdk-test --vdev=crypto_uadk --log-level=6
> > RTE>>cryptodev_uadk_autotest
> > RTE>>quit
>
> Add this information in uadk.rst as well.

Is it OK to add uadk.rst change in this patch?
  

Patch

diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index c2b33686ed..183e24f150 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -16467,6 +16467,12 @@  test_cryptodev_qat(void)
 	return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_QAT_SYM_PMD));
 }
 
+static int
+test_cryptodev_uadk(void)
+{
+	return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_UADK_PMD));
+}
+
 static int
 test_cryptodev_virtio(void)
 {
@@ -16810,6 +16816,7 @@  REGISTER_TEST_COMMAND(cryptodev_sw_mvsam_autotest, test_cryptodev_mrvl);
 REGISTER_TEST_COMMAND(cryptodev_dpaa2_sec_autotest, test_cryptodev_dpaa2_sec);
 REGISTER_TEST_COMMAND(cryptodev_dpaa_sec_autotest, test_cryptodev_dpaa_sec);
 REGISTER_TEST_COMMAND(cryptodev_ccp_autotest, test_cryptodev_ccp);
+REGISTER_TEST_COMMAND(cryptodev_uadk_autotest, test_cryptodev_uadk);
 REGISTER_TEST_COMMAND(cryptodev_virtio_autotest, test_cryptodev_virtio);
 REGISTER_TEST_COMMAND(cryptodev_octeontx_autotest, test_cryptodev_octeontx);
 REGISTER_TEST_COMMAND(cryptodev_caam_jr_autotest, test_cryptodev_caam_jr);
diff --git a/app/test/test_cryptodev.h b/app/test/test_cryptodev.h
index 29a7d4db2b..abd795f54a 100644
--- a/app/test/test_cryptodev.h
+++ b/app/test/test_cryptodev.h
@@ -74,6 +74,7 @@ 
 #define CRYPTODEV_NAME_CN9K_PMD		crypto_cn9k
 #define CRYPTODEV_NAME_CN10K_PMD	crypto_cn10k
 #define CRYPTODEV_NAME_MLX5_PMD		crypto_mlx5
+#define CRYPTODEV_NAME_UADK_PMD		crypto_uadk
 
 
 enum cryptodev_api_test_type {