> -----Original Message-----
> From: Verma, Shally [mailto:[email protected]]
> Sent: Tuesday, July 10, 2018 6:22 AM
> To: De Lara Guarch, Pablo <[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
>
> 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> ?
Correct. For libraries, title prefix should be just the name of the library.
For other components, we usually need two names, the folder where the belong
and the component name.
For example, for drivers, we use the folder where they belong and their name
(e.g. crypto/openssl).
Pablo
>
> 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]>
> >
