[dpdk-dev] [PATCH v2 3/3] test/event_crypto_adapter: free resources during exit

[Gujjar, Abhinandan S]

Hi Ankur,

Please see comment below.

> -----Original Message-----
> From: Ankur Dwivedi <adwivedi at marvell.com>
> Sent: Friday, September 25, 2020 12:59 PM
> To: dev at dpdk.org
> Cc: Gujjar, Abhinandan S <abhinandan.gujjar at intel.com>;
> jerinj at marvell.com; anoobj at marvell.com; Ankur Dwivedi
> <adwivedi at marvell.com>
> Subject: [PATCH v2 3/3] test/event_crypto_adapter: free resources during
> exit
> 
> The resources held by crypto adapter should be freed when the test suite
> exits.
> 
> Signed-off-by: Ankur Dwivedi <adwivedi at marvell.com>
> ---
>  app/test/test_event_crypto_adapter.c | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/app/test/test_event_crypto_adapter.c
> b/app/test/test_event_crypto_adapter.c
> index 572ef9c27..4b09ab3b7 100644
> --- a/app/test/test_event_crypto_adapter.c
> +++ b/app/test/test_event_crypto_adapter.c
> @@ -899,6 +899,20 @@ testsuite_setup(void)
>  	return TEST_SUCCESS;
>  }
> 
> +static void
> +crypto_adapter_teardown(void)
> +{
> +	int ret;
> +
> +	crypto_adapter_setup_done = 0;
Call rte_event_crypto_adapter_stop() here?
> +	ret =
> rte_event_crypto_adapter_queue_pair_del(TEST_ADAPTER_ID,
> +					TEST_CDEV_ID, TEST_CDEV_QP_ID);
> +	if (ret < 0)
> +		RTE_LOG(ERR, USER1, "Failed to delete queue pair!");
> +
> +	rte_event_crypto_adapter_free(TEST_ADAPTER_ID);
> +}
> +
>  static void
>  crypto_teardown(void)
>  {
> @@ -941,6 +955,7 @@ eventdev_teardown(void)  static void
>  testsuite_teardown(void)
>  {
> +	crypto_adapter_teardown();
>  	crypto_teardown();
>  	eventdev_teardown();
>  }
> --
> 2.28.0