[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <d462e493-c719-4372-8991-cdd860f433d9@linux.ibm.com>
Date: Mon, 18 Mar 2024 18:54:40 -0400
From: Stefan Berger <stefanb@...ux.ibm.com>
To: Jarkko Sakkinen <jarkko@...nel.org>,
Stefan Berger <stefanb@...ux.vnet.ibm.com>, keyrings@...r.kernel.org,
linux-crypto@...r.kernel.org, herbert@...dor.apana.org.au,
davem@...emloft.net
Cc: linux-kernel@...r.kernel.org, saulo.alessandre@....jus.br, lukas@...ner.de,
bbhushan2@...vell.com
Subject: Re: [PATCH v6 08/13] crypto: ecc - Add NIST P521 curve parameters
On 3/18/24 17:05, Jarkko Sakkinen wrote:
> On Tue Mar 12, 2024 at 8:36 PM EET, Stefan Berger wrote:
>> From: Stefan Berger <stefanb@...ux.ibm.com>
>>
>> Add the parameters for the NIST P521 curve and define a new curve ID
>> for it. Make the curve available in ecc_get_curve.
>
> This is rare example of "complete story" in this series despite being
> short, so no complains :-)
Wew :-) Thanks for the reviews.
>
>>
>> Signed-off-by: Stefan Berger <stefanb@...ux.ibm.com>
>> Tested-by: Lukas Wunner <lukas@...ner.de>
>> ---
>> crypto/ecc.c | 2 ++
>> crypto/ecc_curve_defs.h | 45 +++++++++++++++++++++++++++++++++++++++++
>> include/crypto/ecdh.h | 1 +
>> 3 files changed, 48 insertions(+)
>>
>> diff --git a/crypto/ecc.c b/crypto/ecc.c
>> index ead40b5ebb46..4f6fa8617308 100644
>> --- a/crypto/ecc.c
>> +++ b/crypto/ecc.c
>> @@ -60,6 +60,8 @@ const struct ecc_curve *ecc_get_curve(unsigned int curve_id)
>> return &nist_p256;
>> case ECC_CURVE_NIST_P384:
>> return &nist_p384;
>> + case ECC_CURVE_NIST_P521:
>> + return &nist_p521;
>> default:
>> return NULL;
>> }
>> diff --git a/crypto/ecc_curve_defs.h b/crypto/ecc_curve_defs.h
>> index ab1ef3d94be5..0ecade7d02f5 100644
>> --- a/crypto/ecc_curve_defs.h
>> +++ b/crypto/ecc_curve_defs.h
>> @@ -89,6 +89,51 @@ static struct ecc_curve nist_p384 = {
>> .b = nist_p384_b
>> };
>>
>> +/* NIST P-521 */
>> +static u64 nist_p521_g_x[] = { 0xf97e7e31c2e5bd66ull, 0x3348b3c1856a429bull,
>> + 0xfe1dc127a2ffa8deull, 0xa14b5e77efe75928ull,
>> + 0xf828af606b4d3dbaull, 0x9c648139053fb521ull,
>> + 0x9e3ecb662395b442ull, 0x858e06b70404e9cdull,
>> + 0xc6ull };
>> +static u64 nist_p521_g_y[] = { 0x88be94769fd16650ull, 0x353c7086a272c240ull,
>> + 0xc550b9013fad0761ull, 0x97ee72995ef42640ull,
>> + 0x17afbd17273e662cull, 0x98f54449579b4468ull,
>> + 0x5c8a5fb42c7d1bd9ull, 0x39296a789a3bc004ull,
>> + 0x118ull };
>> +static u64 nist_p521_p[] = { 0xffffffffffffffffull, 0xffffffffffffffffull,
>> + 0xffffffffffffffffull, 0xffffffffffffffffull,
>> + 0xffffffffffffffffull, 0xffffffffffffffffull,
>> + 0xffffffffffffffffull, 0xffffffffffffffffull,
>> + 0x1ffull };
>> +static u64 nist_p521_n[] = { 0xbb6fb71e91386409ull, 0x3bb5c9b8899c47aeull,
>> + 0x7fcc0148f709a5d0ull, 0x51868783bf2f966bull,
>> + 0xfffffffffffffffaull, 0xffffffffffffffffull,
>> + 0xffffffffffffffffull, 0xffffffffffffffffull,
>> + 0x1ffull };
>> +static u64 nist_p521_a[] = { 0xfffffffffffffffcull, 0xffffffffffffffffull,
>> + 0xffffffffffffffffull, 0xffffffffffffffffull,
>> + 0xffffffffffffffffull, 0xffffffffffffffffull,
>> + 0xffffffffffffffffull, 0xffffffffffffffffull,
>> + 0x1ffull };
>> +static u64 nist_p521_b[] = { 0xef451fd46b503f00ull, 0x3573df883d2c34f1ull,
>> + 0x1652c0bd3bb1bf07ull, 0x56193951ec7e937bull,
>> + 0xb8b489918ef109e1ull, 0xa2da725b99b315f3ull,
>> + 0x929a21a0b68540eeull, 0x953eb9618e1c9a1full,
>> + 0x051ull };
>> +static struct ecc_curve nist_p521 = {
>> + .name = "nist_521",
>> + .nbits = 521,
>> + .g = {
>> + .x = nist_p521_g_x,
>> + .y = nist_p521_g_y,
>> + .ndigits = 9,
>> + },
>> + .p = nist_p521_p,
>> + .n = nist_p521_n,
>> + .a = nist_p521_a,
>> + .b = nist_p521_b
>> +};
>> +
>> /* curve25519 */
>> static u64 curve25519_g_x[] = { 0x0000000000000009, 0x0000000000000000,
>> 0x0000000000000000, 0x0000000000000000 };
>> diff --git a/include/crypto/ecdh.h b/include/crypto/ecdh.h
>> index a9f98078d29c..9784ecdd2fb4 100644
>> --- a/include/crypto/ecdh.h
>> +++ b/include/crypto/ecdh.h
>> @@ -26,6 +26,7 @@
>> #define ECC_CURVE_NIST_P192 0x0001
>> #define ECC_CURVE_NIST_P256 0x0002
>> #define ECC_CURVE_NIST_P384 0x0003
>> +#define ECC_CURVE_NIST_P521 0x0004
>>
>> /**
>> * struct ecdh - define an ECDH private key
>
>
> Reviewed-by: Jarkko Sakkinen <jarkko@...nel.org>
>
> BR, Jarkko
>
Powered by blists - more mailing lists