> This commit adds support for following elliptic curves:
> 1) Curve25519
> 2) Curve448
> 
> Signed-off-by: Arek Kusztal <arkadiuszx.kusz...@intel.com>
> ---
>  lib/cryptodev/rte_crypto_asym.h | 14 ++++++++++++--
>  1 file changed, 12 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/cryptodev/rte_crypto_asym.h b/lib/cryptodev/rte_crypto_asym.h
> index cd24d4b07b..775b2f6277 100644
> --- a/lib/cryptodev/rte_crypto_asym.h
> +++ b/lib/cryptodev/rte_crypto_asym.h
> @@ -48,6 +48,8 @@ enum rte_crypto_ec_group {
>       RTE_CRYPTO_EC_GROUP_SECP256R1 = 23,
>       RTE_CRYPTO_EC_GROUP_SECP384R1 = 24,
>       RTE_CRYPTO_EC_GROUP_SECP521R1 = 25,
> +     RTE_CRYPTO_EC_GROUP_CURVE25519 = 29,
> +     RTE_CRYPTO_EC_GROUP_CURVE448 = 30,
>  };
> 
>  /**
> @@ -180,9 +182,17 @@ typedef rte_crypto_param rte_crypto_uint;
>   */
>  struct rte_crypto_ec_point {
>       rte_crypto_param x;
> -     /**< X coordinate */
> +     /**<
> +      * X coordinate
> +      * For curve25519 and curve448 - little-endian integer
> +      * otherwise, big-endian integer
> +      */
>       rte_crypto_param y;
> -     /**< Y coordinate */
> +     /**<
> +      * Y coordinate
> +      * For curve25519 and curve448 - little-endian integer
> +      * otherwise, big-endian integer
> +      */
Can you give reference of the document which specify this endianness?

And if it is implicit as per the protocol, do we need to add explicit comments 
here?

Reply via email to