> -----Original Message-----
> From: Gujjar, Abhinandan S
> Sent: Tuesday, October 6, 2020 3:52 PM
> To: 'Ankur Dwivedi' <adwiv...@marvell.com>; dev@dpdk.org
> Cc: jer...@marvell.com; ano...@marvell.com
> Subject: RE: [PATCH v2 3/3] test/event_crypto_adapter: free resources
> during exit
>
> Hi Ankur,
>
> Please see comment below.
>
> > -----Original Message-----
> > From: Ankur Dwivedi <adwiv...@marvell.com>
> > Sent: Friday, September 25, 2020 12:59 PM
> > To: dev@dpdk.org
> > Cc: Gujjar, Abhinandan S <abhinandan.guj...@intel.com>;
> > jer...@marvell.com; ano...@marvell.com; Ankur Dwivedi
> > <adwiv...@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 <adwiv...@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;
> > +
Below APIs could return error. So, set this flag at the end?
> > + 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
