Hi Akhil,
Please see inline.
Thanks,
Anoob
> -----Original Message-----
> From: dev <dev-boun...@dpdk.org> On Behalf Of Akhil Goyal
> Sent: Tuesday, October 1, 2019 7:44 PM
> To: Anoob Joseph <ano...@marvell.com>; Pablo de Lara
> <pablo.de.lara.gua...@intel.com>; Thomas Monjalon <tho...@monjalon.net>
> Cc: Sunila Sahu <ss...@marvell.com>; Jerin Jacob Kollanukkaran
> <jer...@marvell.com>; Narayana Prasad Raju Athreya
> <pathr...@marvell.com>; Fiona Trahe <fiona.tr...@intel.com>; Kanaka Durga
> Kotamarthy <kkotamar...@marvell.com>; Shally Verma
> <shal...@marvell.com>; dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH 5/6] app/test: register octeontx2 PMD to asym
> testsuite
>
> >
> > From: Sunila Sahu <ss...@marvell.com>
> >
> > This patch updates asymmetric crypto unit-test application to validate
> > asymmetric crypto operation supported by octeontx2 PMD.
> >
> > Signed-off-by: Anoob Joseph <ano...@marvell.com>
> > Signed-off-by: Kanaka Durga Kotamarthy <kkotamar...@marvell.com>
> > Signed-off-by: Sunila Sahu <ss...@marvell.com>
> > ---
> > app/test/test_cryptodev_asym.c | 18 ++++++++++++++++++
> > 1 file changed, 18 insertions(+)
> >
> > diff --git a/app/test/test_cryptodev_asym.c
> > b/app/test/test_cryptodev_asym.c index 241c384..edbd34e 100644
> > --- a/app/test/test_cryptodev_asym.c
> > +++ b/app/test/test_cryptodev_asym.c
> > @@ -1851,6 +1851,21 @@ test_cryptodev_octeontx_asym(void)
> > return unit_test_suite_runner(&cryptodev_octeontx_asym_testsuite);
> > }
> >
> > +static int
> > +test_cryptodev_octeontx2_asym(void)
> > +{
> > + gbl_driver_id = rte_cryptodev_driver_id_get(
> > + RTE_STR(CRYPTODEV_NAME_OCTEONTX2_PMD));
> > + if (gbl_driver_id == -1) {
> > + RTE_LOG(ERR, USER1, "OCTEONTX2 PMD must be loaded.
> > Check if "
> > +
> > "CONFIG_RTE_LIBRTE_PMD_OCTEONTX2_CRYPTO is "
> > + "enabled in config file to run this "
> > + "testsuite.\n");
> > + return TEST_FAILED;
> > + }
> > + return unit_test_suite_runner(&cryptodev_octeontx_asym_testsuite);
> > +}
> > +
> > REGISTER_TEST_COMMAND(cryptodev_openssl_asym_autotest,
> > test_cryptodev_openssl_asym);
> >
> > @@ -1858,3 +1873,6 @@
> > REGISTER_TEST_COMMAND(cryptodev_qat_asym_autotest,
> > test_cryptodev_qat_asym);
> >
> > REGISTER_TEST_COMMAND(cryptodev_octeontx_asym_autotest,
> > test_cryptodev_octeontx_asym);
>
> It looks a typo but it is intentional. Better to add a comment here and in the
> description That you are reusing the octeontx test suite.
[Anoob] Will make this change in v2.
>
> > +
> > +REGISTER_TEST_COMMAND(cryptodev_octeontx2_asym_autotest,
> > + test_cryptodev_octeontx2_asym);
> > --
> > 2.7.4
