On 2/16/24 13:48, Elliott, Robert (Servers) wrote:
-----Original Message-----
From: Stefan Berger <stefanb@xxxxxxxxxxxxx>
Sent: Thursday, February 15, 2024 5:14 PM
Subject: [PATCH v2 06/14] crypto: ecc - Add NIST P521 curve parameters
Add the parameters for the NIST P521 curve and define a new curve ID
for it. Make the curve available in ecc_get_curve.
...
diff --git a/crypto/ecc_curve_defs.h b/crypto/ecc_curve_defs.h
...
+static struct ecc_curve nist_p521 = {
+ .name = "nist_521",
Are the name fields in the ecc_curve structures used anywhere or
exposed to userspace?
It'd be nice if the strings for the nist_p192, nist_p256, and nist_p384
structures and this new nist_p521 structure included "p" before
the number, better matching all the code and the NIST FIPS 186-4 names:
.name = "nist_p192"
.name = "nist_p256"
.name = "nist_p384"
.name = "nist_p521"
This is what is exposed:
$ cat /proc/crypto | grep nist
name : ecdh-nist-p384
driver : ecdh-nist-p384-generic
name : ecdh-nist-p256
driver : ecdh-nist-p256-generic
name : ecdh-nist-p192
driver : ecdh-nist-p192-generic
name : ecdsa-nist-p384
driver : ecdsa-nist-p384-generic
name : ecdsa-nist-p256
driver : ecdsa-nist-p256-generic
name : ecdsa-nist-p192
driver : ecdsa-nist-p192-generic