From mboxrd@z Thu Jan 1 00:00:00 1970 From: Akhil Goyal Subject: Re: [PATCH v3 31/32] test: adds validation test Date: Mon, 8 Oct 2018 18:03:11 +0530 Message-ID: <407d40a1-1ac6-3a28-dfcb-aaf3f1829182@nxp.com> References: <1536033560-21541-1-git-send-email-ajoseph@caviumnetworks.com> <1538744363-30340-1-git-send-email-anoob.joseph@caviumnetworks.com> <1538744363-30340-32-git-send-email-anoob.joseph@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Ankur Dwivedi , Jerin Jacob , Narayana Prasad , dev@dpdk.org, Murthy NSSR , Nithin Dabilpuram , Ragothaman Jayaraman , Srisivasubramanian S , Tejasree Kondoj To: Anoob Joseph , Pablo de Lara , Thomas Monjalon Return-path: Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-eopbgr80042.outbound.protection.outlook.com [40.107.8.42]) by dpdk.org (Postfix) with ESMTP id 9B505683E for ; Mon, 8 Oct 2018 14:33:28 +0200 (CEST) In-Reply-To: <1538744363-30340-32-git-send-email-anoob.joseph@caviumnetworks.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Anoob, On 10/5/2018 6:29 PM, Anoob Joseph wrote: > From: Ankur Dwivedi > > This patch adds validation tests for octeontx crypto device. > > Signed-off-by: Ankur Dwivedi > Signed-off-by: Anoob Joseph > Signed-off-by: Murthy NSSR > Signed-off-by: Nithin Dabilpuram > Signed-off-by: Ragothaman Jayaraman > Signed-off-by: Srisivasubramanian S > Signed-off-by: Tejasree Kondoj > --- > test/test/meson.build | 1 + > test/test/test_cryptodev.c | 316 +++++++++++++++++++++++++++ > test/test/test_cryptodev.h | 1 + > test/test/test_cryptodev_aes_test_vectors.h | 114 ++++++---- > test/test/test_cryptodev_blockcipher.c | 9 +- > test/test/test_cryptodev_blockcipher.h | 1 + > test/test/test_cryptodev_des_test_vectors.h | 12 +- > test/test/test_cryptodev_hash_test_vectors.h | 78 ++++--- > 8 files changed, 463 insertions(+), 69 deletions(-) > > diff --git a/test/test/meson.build b/test/test/meson.build > index b1dd6ec..2d59041 100644 > --- a/test/test/meson.build > +++ b/test/test/meson.build > @@ -141,6 +141,7 @@ test_names = [ > 'cryptodev_sw_mvsam_autotest', > 'cryptodev_dpaa2_sec_autotest', > 'cryptodev_dpaa_sec_autotest', > + 'cryptodev_octeontx_autotest', > 'cycles_autotest', > 'debug_autotest', > 'devargs_autotest', > diff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c > index c63662d..b9731b9 100644 > --- a/test/test/test_cryptodev.c > +++ b/test/test/test_cryptodev.c > @@ -2142,6 +2142,96 @@ test_3DES_cipheronly_mrvl_all(void) > return TEST_SUCCESS; > } > > +static int > +test_AES_chain_cpt_all(void) > +{ > + struct crypto_testsuite_params *ts_params = &testsuite_params; > + int status; > + > + status = test_blockcipher_all_tests(ts_params->mbuf_pool, > + ts_params->op_mpool, ts_params->session_mpool, > + ts_params->valid_devs[0], > + rte_cryptodev_driver_id_get( > + RTE_STR(CRYPTODEV_NAME_OCTEONTX_SYM_PMD)), > + BLKCIPHER_AES_CHAIN_TYPE); > + > + TEST_ASSERT_EQUAL(status, 0, "Test failed"); > + > + return TEST_SUCCESS; > +} > + > +static int > +test_AES_cipheronly_cpt_all(void) > +{ > + struct crypto_testsuite_params *ts_params = &testsuite_params; > + int status; > + > + status = test_blockcipher_all_tests(ts_params->mbuf_pool, > + ts_params->op_mpool, ts_params->session_mpool, > + ts_params->valid_devs[0], > + rte_cryptodev_driver_id_get( > + RTE_STR(CRYPTODEV_NAME_OCTEONTX_SYM_PMD)), > + BLKCIPHER_AES_CIPHERONLY_TYPE); > + > + TEST_ASSERT_EQUAL(status, 0, "Test failed"); > + > + return TEST_SUCCESS; > +} > + > +static int > +test_3DES_chain_cpt_all(void) > +{ > + struct crypto_testsuite_params *ts_params = &testsuite_params; > + int status; > + > + status = test_blockcipher_all_tests(ts_params->mbuf_pool, > + ts_params->op_mpool, ts_params->session_mpool, > + ts_params->valid_devs[0], > + rte_cryptodev_driver_id_get( > + RTE_STR(CRYPTODEV_NAME_OCTEONTX_SYM_PMD)), > + BLKCIPHER_3DES_CHAIN_TYPE); > + > + TEST_ASSERT_EQUAL(status, 0, "Test failed"); > + > + return TEST_SUCCESS; > +} > + > +static int > +test_3DES_cipheronly_cpt_all(void) > +{ > + struct crypto_testsuite_params *ts_params = &testsuite_params; > + int status; > + > + status = test_blockcipher_all_tests(ts_params->mbuf_pool, > + ts_params->op_mpool, ts_params->session_mpool, > + ts_params->valid_devs[0], > + rte_cryptodev_driver_id_get( > + RTE_STR(CRYPTODEV_NAME_OCTEONTX_SYM_PMD)), > + BLKCIPHER_3DES_CIPHERONLY_TYPE); > + > + TEST_ASSERT_EQUAL(status, 0, "Test failed"); > + > + return TEST_SUCCESS; > +} > + > +static int > +test_authonly_cpt_all(void) > +{ > + struct crypto_testsuite_params *ts_params = &testsuite_params; > + int status; > + > + status = test_blockcipher_all_tests(ts_params->mbuf_pool, > + ts_params->op_mpool, ts_params->session_mpool, > + ts_params->valid_devs[0], > + rte_cryptodev_driver_id_get( > + RTE_STR(CRYPTODEV_NAME_OCTEONTX_SYM_PMD)), > + BLKCIPHER_AUTHONLY_TYPE); > + > + TEST_ASSERT_EQUAL(status, 0, "Test failed"); > + > + return TEST_SUCCESS; > +} > + > /* ***** SNOW 3G Tests ***** */ > static int > create_wireless_algo_hash_session(uint8_t dev_id, > @@ -9951,6 +10041,216 @@ static struct unit_test_suite cryptodev_ccp_testsuite = { > } > }; > > +static struct unit_test_suite cryptodev_cpt_testsuite = { Shouldn't this be cryptodev_octeontx_testsuite and for other functions as well? I believe cpt is helper API set, and main PMD is octeontx, so the test shall also have octeontx reference. Thanks, Akhil