[dpdk-dev] [PATCH v3 31/32] test: adds validation test

Akhil Goyal akhil.goyal at nxp.com
Mon Oct 8 14:56:10 CEST 2018


one more... title should be test/crypto: add octeontx unit tests

On 10/8/2018 6:03 PM, Akhil Goyal wrote:
>
> Hi Anoob,
>
> On 10/5/2018 6:29 PM, Anoob Joseph wrote:
>> From: Ankur Dwivedi <ankur.dwivedi at caviumnetworks.com>
>>
>> This patch adds validation tests for octeontx crypto device.
>>
>> Signed-off-by: Ankur Dwivedi <ankur.dwivedi at caviumnetworks.com>
>> Signed-off-by: Anoob Joseph <anoob.joseph at caviumnetworks.com>
>> Signed-off-by: Murthy NSSR <nidadavolu.murthy at caviumnetworks.com>
>> Signed-off-by: Nithin Dabilpuram <nithin.dabilpuram at caviumnetworks.com>
>> Signed-off-by: Ragothaman Jayaraman <rjayaraman at caviumnetworks.com>
>> Signed-off-by: Srisivasubramanian S <ssrinivasan at caviumnetworks.com>
>> Signed-off-by: Tejasree Kondoj <kondoj.tejasree at caviumnetworks.com>
>> ---
>>   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



More information about the dev mailing list