Hi Pablo

>-----Original Message-----
>From: De Lara Guarch, Pablo [mailto:[email protected]]
>Sent: 10 July 2018 03:54
>To: Verma, Shally <[email protected]>
>Cc: [email protected]; Athreya, Narayana Prasad 
><[email protected]>; Murthy, Nidadavolu
><[email protected]>; Sahu, Sunila <[email protected]>; Gupta, 
>Ashish <[email protected]>; Kartha,
>Umesh <[email protected]>
>Subject: RE: [PATCH v4 1/4] lib/cryptodev: add asymmetric algos in cryptodev
>
>External Email
>
>Hi Shally,
>
>A few minor comments:
>
>> -----Original Message-----
>> From: Shally Verma [mailto:[email protected]]
>> Sent: Tuesday, July 3, 2018 4:24 PM
>> To: De Lara Guarch, Pablo <[email protected]>
>> Cc: [email protected]; [email protected];
>> [email protected]; Sunila Sahu
>> <[email protected]>; Ashish Gupta
>> <[email protected]>; Umesh Kartha
>> <[email protected]>
>> Subject: [PATCH v4 1/4] lib/cryptodev: add asymmetric algos in cryptodev
>>
>
>Change title to: "cryptodev: add basic asymmetric structures"?
>At least, don't use lib, and cryptodev twice.

So what's convention for all patches? Just cryptodev: <actual title> ?

Thanks
Shally

>
>> Add rte_crypto_asym.h with supported xfrms and associated op structures and
>> APIs
>>
>> API currently supports:
>> - RSA Encrypt, Decrypt, Sign and Verify
>> - Modular Exponentiation and Inversion
>> - DSA Sign and Verify
>> - Deffie-hellman private key exchange
>
>Diffie-Hellman
>
>> - Deffie-hellman public key exchange
>> - Deffie-hellman shared secret compute
>> - Deffie-hellman public/private key pair generation using xform chain
>>
>> Signed-off-by: Shally Verma <[email protected]>
>> Signed-off-by: Sunila Sahu <[email protected]>
>> Signed-off-by: Ashish Gupta <[email protected]>
>> Signed-off-by: Umesh Kartha <[email protected]>
>

Reply via email to