[dpdk-dev] [PATCH v1 2/2] test/crypto: remove data verification at rsa verify operation
Verma, Shally
Shally.Verma at cavium.com
Mon Oct 22 14:29:30 CEST 2018
>-----Original Message-----
>From: dev <dev-bounces at dpdk.org> On Behalf Of Akhil Goyal
>Sent: 16 October 2018 18:51
>To: Saxena, Akash <Akash.Saxena at cavium.com>; pablo.de.lara.guarch at intel.com
>Cc: dev at dpdk.org; Verma, Shally <Shally.Verma at cavium.com>; Verma, Ayuj <Ayuj.Verma at cavium.com>
>Subject: Re: [dpdk-dev] [PATCH v1 2/2] test/crypto: remove data verification at rsa verify operation
>
>External Email
>
>On 10/11/2018 7:13 PM, Akash Saxena wrote:
>> Change unit test app to check only for op->status =
>> RTE_CRYPTO_OP_STATUS_SUCCESS/ERROR instead of calling rsa_verify().
>>
>> Signed-off-by: Ayuj Verma <ayuj.verma at caviumnetworks.com>
>> Signed-off-by: Shally Verma <shally.verma at caviumnetworks.com>
>> ---
>> test/test/test_cryptodev_asym.c | 9 ++++++---
>> 1 file changed, 6 insertions(+), 3 deletions(-)
>>
>> diff --git a/test/test/test_cryptodev_asym.c b/test/test/test_cryptodev_asym.c
>> index 2fdfc1d..a899f99 100644
>> --- a/test/test/test_cryptodev_asym.c
>> +++ b/test/test/test_cryptodev_asym.c
>> @@ -153,10 +153,13 @@ test_rsa_sign_verify(void)
>> goto error_exit;
>> }
>> status = TEST_SUCCESS;
>> - int ret = 0;
>> - ret = rsa_verify(&rsaplaintext, result_op);
>> - if (ret)
>> + if (result_op->status != RTE_CRYPTO_OP_STATUS_SUCCESS) {
>> + RTE_LOG(ERR, USER1,
>> + "line %u FAILED: %s",
>> + __LINE__, "Failed to process asym crypto op");
>> status = TEST_FAILED;
>> + goto error_exit;
>> + }
>>
>> error_exit:
>>
>What about test_rsa_enc_dec? Do you need to remove verify from that as well?
No. in case of enc_dec, PMD just return decrypted o/p so, app need to do cross-verify data.
In case of RSA_VERIFY OP PMD internally does memcmp of plaintext to decrypted o/p and ensure
Pass / fail. So, doing it again at app level is redundant.
Thanks
Shally
More information about the dev
mailing list