[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4227.1430496240@warthog.procyon.org.uk>
Date: Fri, 01 May 2015 17:04:00 +0100
From: David Howells <dhowells@...hat.com>
To: Tadeusz Struk <tadeusz.struk@...el.com>
Cc: dhowells@...hat.com, herbert@...dor.apana.org.au, corbet@....net,
keescook@...omium.org, qat-linux@...el.com, jwboyer@...hat.com,
richard@....at, d.kasatkin@...sung.com,
linux-kernel@...r.kernel.org, steved@...hat.com, vgoyal@...hat.com,
james.l.morris@...cle.com, jkosina@...e.cz,
zohar@...ux.vnet.ibm.com, davem@...emloft.net,
linux-crypto@...r.kernel.org
Subject: Re: [PATCH RFC 1/2] crypto: add PKE API
Tadeusz Struk <tadeusz.struk@...el.com> wrote:
> +config CRYPTO_PKE
I would prefer CRYPTO_PKEY and pkey rather than pke generally and algo rather
than alg where possible - this will have more consistency with what we have
now.
I understand, however, that in core crypto code, 'alg' is used.
+ seq_printf(m, "can encrypt : %s\n", cap & PKEY_CAN_ENCRYPT ?
+ "yes" : "no");
Please split the line after the comma.
> #define CRYPTO_ALG_TYPE_AHASH 0x0000000a
> +#define CRYPTO_ALG_TYPE_PKE 0x0000000b
> #define CRYPTO_ALG_TYPE_RNG 0x0000000c
Will filling a hole cause a problem with something that got obsoleted?
> +struct pke_request {
> + struct crypto_async_request base;
> + const struct public_key *pk;
> + const struct public_key_signature *pks;
> +};
I recommend:
pk -> key or pkey
pks -> sig or signature
> + u8 pub_mpis; /* Number of MPIs in public key */
> + u8 sec_mpis; /* Number of MPIs in secret key */
> + u8 sig_mpis; /* Number of MPIs in a signature */
Keep member names as:
u8 n_pub_mpi; /* Number of MPIs in public key */
u8 n_sec_mpi; /* Number of MPIs in secret key */
u8 n_sig_mpi; /* Number of MPIs in a signature */
please.
David
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists