> > > /** > > > + * Asymmetric crypto key exchange operation type */ enum > > > +rte_crypto_asym_ke_type { > > > + RTE_CRYPTO_ASYM_KE_PRIVATE_KEY_GENERATE, > > > > Is it better to shorten it to > > RTE_CRYPTO_ASYM_KE_PRIV_KEY_GENERATE > > RTE_CRYPTO_ASYM_KE_PUB_KEY_GENERATE > +1, Actually I am not big fan of having asym everywhere too. > RTE_CRYPTO_KE_PRIV_KEY_GENERATE would be equally good.
It is better to keep ASYM, or else people might relate with IKE > > > > > + /**< Private Key generation operation */ > > > + RTE_CRYPTO_ASYM_KE_PUBLIC_KEY_GENERATE, > > > + /**< Public Key generation operation */ > > > + RTE_CRYPTO_ASYM_KE_SHARED_SECRET_COMPUTE > > > + /**< Shared Secret compute operation */ }; > > > + > > > +/** > > > * Padding types for RSA signature. > > > */ > > > enum rte_crypto_rsa_padding_type { > > > @@ -238,7 +248,7 @@ struct rte_crypto_modinv_xform { > > > * > > > */ > > > struct rte_crypto_dh_xform { > > > - enum rte_crypto_asym_op_type type; > > > + enum rte_crypto_asym_ke_type type; > > > /**< Setup xform for key generate or shared secret compute */ > > > rte_crypto_uint p; > > > /**< Prime modulus data */ > > > @@ -375,26 +385,27 @@ struct rte_crypto_rsa_op_param { struct > > > rte_crypto_dh_op_param { > > > rte_crypto_uint pub_key; > > > /**< > > > - * Output generated public key when xform type is > > > - * DH PUB_KEY_GENERATION. > > > - * Input peer public key when xform type is DH > > > - * SHARED_SECRET_COMPUTATION > > > + * Output - generated public key, when xform type is > > > > It is not xform type, Right? > > It should be key exchange type. > Yes, I meant xform op_type. Will change, leter it will be overwritten by move > dh > op patch too. OK > > Check at other places also. > > > > > + * RTE_CRYPTO_ASYM_KE_PUBLIC_KEY_GENERATE. > > > * > > > + * Input - peer's public key, when xform type is > > > + * RTE_CRYPTO_ASYM_KE_SHARED_SECRET_COMPUTE. > > > */ > > > > > > rte_crypto_uint priv_key; > > > /**< > > > - * Output generated private key if xform type is > > > - * DH PRIVATE_KEY_GENERATION > > > - * Input when xform type is DH SHARED_SECRET_COMPUTATION. > > > + * Output - generated private key, when xform type is > > > + * RTE_CRYPTO_ASYM_KE_PRIVATE_KEY_GENERATE. > > > * > > > + * Input - private key, when xform type is one of: > > > + * RTE_CRYPTO_ASYM_KE_PUBLIC_KEY_GENERATE, > > > + * RTE_CRYPTO_ASYM_KE_SHARED_SECRET_COMPUTE. > > > */ > > > > > > rte_crypto_uint shared_secret; > > > /**< > > > - * Output with calculated shared secret > > > - * when dh xform set up with op type = > > > SHARED_SECRET_COMPUTATION. > > > - * > > > + * Output - calculated shared secret when xform type is > > > + * RTE_CRYPTO_ASYM_KE_SHARED_SECRET_COMPUTE. > > > */ > > > }; > > >