Re: [dpdk-dev] [PATCH v5 8/8] crypto/bcmfs: add crypto pmd into cryptodev test

[Akhil Goyal <akhil.goyal@nxp.com>]

Hi Vikas,
> 
> Add global test suite for bcmfs crypto pmd
> 
> Signed-off-by: Vikas Gupta <vikas.gupta@broadcom.com>
> Signed-off-by: Raveendra Padasalagi <raveendra.padasalagi@broadcom.com>
> Reviewed-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
> ---
>  app/test/test_cryptodev.c       | 17 +++++++++++++++++
>  app/test/test_cryptodev.h       |  1 +
>  doc/guides/cryptodevs/bcmfs.rst | 11 +++++++++++
>  3 files changed, 29 insertions(+)
> 
> diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
> index 70bf6fe2c1..9157115ab3 100644
> --- a/app/test/test_cryptodev.c
> +++ b/app/test/test_cryptodev.c
> @@ -13041,6 +13041,22 @@ test_cryptodev_nitrox(void)
>  	return unit_test_suite_runner(&cryptodev_nitrox_testsuite);
>  }
> 
> +static int
> +test_cryptodev_bcmfs(void)
> +{
> +	gbl_driver_id =	rte_cryptodev_driver_id_get(
> +			RTE_STR(CRYPTODEV_NAME_BCMFS_PMD));
> +
> +	if (gbl_driver_id == -1) {
> +		RTE_LOG(ERR, USER1, "BCMFS PMD must be loaded. Check if "
> +				"CONFIG_RTE_LIBRTE_PMD_BCMFS is enabled
> "
> +				"in config file to run this testsuite.\n");

Modified this LOG print message to remove the config. Configs are no longer used.

> +		return TEST_FAILED;
> +	}
> +
> +	return unit_test_suite_runner(&cryptodev_testsuite);
> +}
> +
>  REGISTER_TEST_COMMAND(cryptodev_qat_autotest, test_cryptodev_qat);
>  REGISTER_TEST_COMMAND(cryptodev_aesni_mb_autotest,
> test_cryptodev_aesni_mb);
>  REGISTER_TEST_COMMAND(cryptodev_cpu_aesni_mb_autotest,
> @@ -13063,3 +13079,4 @@
> REGISTER_TEST_COMMAND(cryptodev_octeontx_autotest,
> test_cryptodev_octeontx);
>  REGISTER_TEST_COMMAND(cryptodev_octeontx2_autotest,
> test_cryptodev_octeontx2);
>  REGISTER_TEST_COMMAND(cryptodev_caam_jr_autotest,
> test_cryptodev_caam_jr);
>  REGISTER_TEST_COMMAND(cryptodev_nitrox_autotest,
> test_cryptodev_nitrox);
> +REGISTER_TEST_COMMAND(cryptodev_bcmfs_autotest,
> test_cryptodev_bcmfs);
> diff --git a/app/test/test_cryptodev.h b/app/test/test_cryptodev.h
> index 41542e0552..c58126368c 100644
> --- a/app/test/test_cryptodev.h
> +++ b/app/test/test_cryptodev.h
> @@ -70,6 +70,7 @@
>  #define CRYPTODEV_NAME_OCTEONTX2_PMD	crypto_octeontx2
>  #define CRYPTODEV_NAME_CAAM_JR_PMD	crypto_caam_jr
>  #define CRYPTODEV_NAME_NITROX_PMD	crypto_nitrox_sym
> +#define CRYPTODEV_NAME_BCMFS_PMD	crypto_bcmfs
> 
>  /**
>   * Write (spread) data from buffer to mbuf data
> diff --git a/doc/guides/cryptodevs/bcmfs.rst b/doc/guides/cryptodevs/bcmfs.rst
> index f7e15f4cfb..5a7eb23c0f 100644
> --- a/doc/guides/cryptodevs/bcmfs.rst
> +++ b/doc/guides/cryptodevs/bcmfs.rst
> @@ -96,3 +96,14 @@ Limitations
> 
>  * Only supports the session-oriented API implementation (session-less APIs are
> not supported).
>  * CCM is not supported on Broadcom`s SoCs having FlexSparc4 unit.
> +
> +Testing
> +-------
> +
> +The symmetric crypto operations on BCMFS crypto PMD may be verified by
> running the test
> +application:
> +
> +.. code-block:: console
> +
> +    ./test

This has been changed to ./dpdk-test


> +    RTE>>cryptodev_bcmfs_autotest
> --
> 2.17.1