From: Antoine Tenart <antoine.tenart@bootlin.com>
To: Pascal van Leeuwen <pascalvanl@gmail.com>
Cc: linux-crypto@vger.kernel.org, antoine.tenart@bootlin.com,
herbert@gondor.apana.org.au, davem@davemloft.net,
Pascal van Leeuwen <pvanleeuwen@verimatrix.com>
Subject: Re: [PATCHv3 2/4] crypto: inside-secure - Remove redundant algo to engine mapping code
Date: Mon, 5 Aug 2019 09:51:03 +0200 [thread overview]
Message-ID: <20190805075103.GE14470@kwain> (raw)
In-Reply-To: <1564586959-9963-3-git-send-email-pvanleeuwen@verimatrix.com>
Hi Pascal,
On Wed, Jul 31, 2019 at 05:29:17PM +0200, Pascal van Leeuwen wrote:
> This removes some code determine which engine has which algorithms which
> was effectively redundant (may have been forward-looking?) due to always
> enabling all algorithms for all currently supported engines.
> A future patch will use a different, more scalable approach to achieve
> this. This is removed now because otherwise the next patch will add new
> hardware which would otherwise have to be added to all algorithms, so
> now is a convenient time to just get rid of this.
>
> Signed-off-by: Pascal van Leeuwen <pvanleeuwen@verimatrix.com>
Acked-by: Antoine Tenart <antoine.tenart@bootlin.com>
Thanks!
Antoine
> ---
> drivers/crypto/inside-secure/safexcel.c | 9 ---------
> drivers/crypto/inside-secure/safexcel.h | 1 -
> drivers/crypto/inside-secure/safexcel_cipher.c | 11 -----------
> drivers/crypto/inside-secure/safexcel_hash.c | 12 ------------
> 4 files changed, 33 deletions(-)
>
> diff --git a/drivers/crypto/inside-secure/safexcel.c b/drivers/crypto/inside-secure/safexcel.c
> index df43a2c..a066152 100644
> --- a/drivers/crypto/inside-secure/safexcel.c
> +++ b/drivers/crypto/inside-secure/safexcel.c
> @@ -869,9 +869,6 @@ static int safexcel_register_algorithms(struct safexcel_crypto_priv *priv)
> for (i = 0; i < ARRAY_SIZE(safexcel_algs); i++) {
> safexcel_algs[i]->priv = priv;
>
> - if (!(safexcel_algs[i]->engines & priv->version))
> - continue;
> -
> if (safexcel_algs[i]->type == SAFEXCEL_ALG_TYPE_SKCIPHER)
> ret = crypto_register_skcipher(&safexcel_algs[i]->alg.skcipher);
> else if (safexcel_algs[i]->type == SAFEXCEL_ALG_TYPE_AEAD)
> @@ -887,9 +884,6 @@ static int safexcel_register_algorithms(struct safexcel_crypto_priv *priv)
>
> fail:
> for (j = 0; j < i; j++) {
> - if (!(safexcel_algs[j]->engines & priv->version))
> - continue;
> -
> if (safexcel_algs[j]->type == SAFEXCEL_ALG_TYPE_SKCIPHER)
> crypto_unregister_skcipher(&safexcel_algs[j]->alg.skcipher);
> else if (safexcel_algs[j]->type == SAFEXCEL_ALG_TYPE_AEAD)
> @@ -906,9 +900,6 @@ static void safexcel_unregister_algorithms(struct safexcel_crypto_priv *priv)
> int i;
>
> for (i = 0; i < ARRAY_SIZE(safexcel_algs); i++) {
> - if (!(safexcel_algs[i]->engines & priv->version))
> - continue;
> -
> if (safexcel_algs[i]->type == SAFEXCEL_ALG_TYPE_SKCIPHER)
> crypto_unregister_skcipher(&safexcel_algs[i]->alg.skcipher);
> else if (safexcel_algs[i]->type == SAFEXCEL_ALG_TYPE_AEAD)
> diff --git a/drivers/crypto/inside-secure/safexcel.h b/drivers/crypto/inside-secure/safexcel.h
> index e0c202f..e53c232 100644
> --- a/drivers/crypto/inside-secure/safexcel.h
> +++ b/drivers/crypto/inside-secure/safexcel.h
> @@ -655,7 +655,6 @@ struct safexcel_ahash_export_state {
> struct safexcel_alg_template {
> struct safexcel_crypto_priv *priv;
> enum safexcel_alg_type type;
> - u32 engines;
> union {
> struct skcipher_alg skcipher;
> struct aead_alg aead;
> diff --git a/drivers/crypto/inside-secure/safexcel_cipher.c b/drivers/crypto/inside-secure/safexcel_cipher.c
> index 8cdbdbe..ee8a0c3 100644
> --- a/drivers/crypto/inside-secure/safexcel_cipher.c
> +++ b/drivers/crypto/inside-secure/safexcel_cipher.c
> @@ -881,7 +881,6 @@ static void safexcel_aead_cra_exit(struct crypto_tfm *tfm)
>
> struct safexcel_alg_template safexcel_alg_ecb_aes = {
> .type = SAFEXCEL_ALG_TYPE_SKCIPHER,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.skcipher = {
> .setkey = safexcel_skcipher_aes_setkey,
> .encrypt = safexcel_ecb_aes_encrypt,
> @@ -920,7 +919,6 @@ static int safexcel_cbc_aes_decrypt(struct skcipher_request *req)
>
> struct safexcel_alg_template safexcel_alg_cbc_aes = {
> .type = SAFEXCEL_ALG_TYPE_SKCIPHER,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.skcipher = {
> .setkey = safexcel_skcipher_aes_setkey,
> .encrypt = safexcel_cbc_aes_encrypt,
> @@ -990,7 +988,6 @@ static int safexcel_des_setkey(struct crypto_skcipher *ctfm, const u8 *key,
>
> struct safexcel_alg_template safexcel_alg_cbc_des = {
> .type = SAFEXCEL_ALG_TYPE_SKCIPHER,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.skcipher = {
> .setkey = safexcel_des_setkey,
> .encrypt = safexcel_cbc_des_encrypt,
> @@ -1030,7 +1027,6 @@ static int safexcel_ecb_des_decrypt(struct skcipher_request *req)
>
> struct safexcel_alg_template safexcel_alg_ecb_des = {
> .type = SAFEXCEL_ALG_TYPE_SKCIPHER,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.skcipher = {
> .setkey = safexcel_des_setkey,
> .encrypt = safexcel_ecb_des_encrypt,
> @@ -1093,7 +1089,6 @@ static int safexcel_des3_ede_setkey(struct crypto_skcipher *ctfm,
>
> struct safexcel_alg_template safexcel_alg_cbc_des3_ede = {
> .type = SAFEXCEL_ALG_TYPE_SKCIPHER,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.skcipher = {
> .setkey = safexcel_des3_ede_setkey,
> .encrypt = safexcel_cbc_des3_ede_encrypt,
> @@ -1133,7 +1128,6 @@ static int safexcel_ecb_des3_ede_decrypt(struct skcipher_request *req)
>
> struct safexcel_alg_template safexcel_alg_ecb_des3_ede = {
> .type = SAFEXCEL_ALG_TYPE_SKCIPHER,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.skcipher = {
> .setkey = safexcel_des3_ede_setkey,
> .encrypt = safexcel_ecb_des3_ede_encrypt,
> @@ -1203,7 +1197,6 @@ static int safexcel_aead_sha1_cra_init(struct crypto_tfm *tfm)
>
> struct safexcel_alg_template safexcel_alg_authenc_hmac_sha1_cbc_aes = {
> .type = SAFEXCEL_ALG_TYPE_AEAD,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.aead = {
> .setkey = safexcel_aead_aes_setkey,
> .encrypt = safexcel_aead_encrypt,
> @@ -1238,7 +1231,6 @@ static int safexcel_aead_sha256_cra_init(struct crypto_tfm *tfm)
>
> struct safexcel_alg_template safexcel_alg_authenc_hmac_sha256_cbc_aes = {
> .type = SAFEXCEL_ALG_TYPE_AEAD,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.aead = {
> .setkey = safexcel_aead_aes_setkey,
> .encrypt = safexcel_aead_encrypt,
> @@ -1273,7 +1265,6 @@ static int safexcel_aead_sha224_cra_init(struct crypto_tfm *tfm)
>
> struct safexcel_alg_template safexcel_alg_authenc_hmac_sha224_cbc_aes = {
> .type = SAFEXCEL_ALG_TYPE_AEAD,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.aead = {
> .setkey = safexcel_aead_aes_setkey,
> .encrypt = safexcel_aead_encrypt,
> @@ -1308,7 +1299,6 @@ static int safexcel_aead_sha512_cra_init(struct crypto_tfm *tfm)
>
> struct safexcel_alg_template safexcel_alg_authenc_hmac_sha512_cbc_aes = {
> .type = SAFEXCEL_ALG_TYPE_AEAD,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.aead = {
> .setkey = safexcel_aead_aes_setkey,
> .encrypt = safexcel_aead_encrypt,
> @@ -1343,7 +1333,6 @@ static int safexcel_aead_sha384_cra_init(struct crypto_tfm *tfm)
>
> struct safexcel_alg_template safexcel_alg_authenc_hmac_sha384_cbc_aes = {
> .type = SAFEXCEL_ALG_TYPE_AEAD,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.aead = {
> .setkey = safexcel_aead_aes_setkey,
> .encrypt = safexcel_aead_encrypt,
> diff --git a/drivers/crypto/inside-secure/safexcel_hash.c b/drivers/crypto/inside-secure/safexcel_hash.c
> index a80a5e7..2c31536 100644
> --- a/drivers/crypto/inside-secure/safexcel_hash.c
> +++ b/drivers/crypto/inside-secure/safexcel_hash.c
> @@ -802,7 +802,6 @@ static void safexcel_ahash_cra_exit(struct crypto_tfm *tfm)
>
> struct safexcel_alg_template safexcel_alg_sha1 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_sha1_init,
> .update = safexcel_ahash_update,
> @@ -1035,7 +1034,6 @@ static int safexcel_hmac_sha1_setkey(struct crypto_ahash *tfm, const u8 *key,
>
> struct safexcel_alg_template safexcel_alg_hmac_sha1 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_hmac_sha1_init,
> .update = safexcel_ahash_update,
> @@ -1099,7 +1097,6 @@ static int safexcel_sha256_digest(struct ahash_request *areq)
>
> struct safexcel_alg_template safexcel_alg_sha256 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_sha256_init,
> .update = safexcel_ahash_update,
> @@ -1162,7 +1159,6 @@ static int safexcel_sha224_digest(struct ahash_request *areq)
>
> struct safexcel_alg_template safexcel_alg_sha224 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_sha224_init,
> .update = safexcel_ahash_update,
> @@ -1218,7 +1214,6 @@ static int safexcel_hmac_sha224_digest(struct ahash_request *areq)
>
> struct safexcel_alg_template safexcel_alg_hmac_sha224 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_hmac_sha224_init,
> .update = safexcel_ahash_update,
> @@ -1275,7 +1270,6 @@ static int safexcel_hmac_sha256_digest(struct ahash_request *areq)
>
> struct safexcel_alg_template safexcel_alg_hmac_sha256 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_hmac_sha256_init,
> .update = safexcel_ahash_update,
> @@ -1347,7 +1341,6 @@ static int safexcel_sha512_digest(struct ahash_request *areq)
>
> struct safexcel_alg_template safexcel_alg_sha512 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_sha512_init,
> .update = safexcel_ahash_update,
> @@ -1418,7 +1411,6 @@ static int safexcel_sha384_digest(struct ahash_request *areq)
>
> struct safexcel_alg_template safexcel_alg_sha384 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_sha384_init,
> .update = safexcel_ahash_update,
> @@ -1474,7 +1466,6 @@ static int safexcel_hmac_sha512_digest(struct ahash_request *areq)
>
> struct safexcel_alg_template safexcel_alg_hmac_sha512 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_hmac_sha512_init,
> .update = safexcel_ahash_update,
> @@ -1531,7 +1522,6 @@ static int safexcel_hmac_sha384_digest(struct ahash_request *areq)
>
> struct safexcel_alg_template safexcel_alg_hmac_sha384 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_hmac_sha384_init,
> .update = safexcel_ahash_update,
> @@ -1591,7 +1581,6 @@ static int safexcel_md5_digest(struct ahash_request *areq)
>
> struct safexcel_alg_template safexcel_alg_md5 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_md5_init,
> .update = safexcel_ahash_update,
> @@ -1647,7 +1636,6 @@ static int safexcel_hmac_md5_digest(struct ahash_request *areq)
>
> struct safexcel_alg_template safexcel_alg_hmac_md5 = {
> .type = SAFEXCEL_ALG_TYPE_AHASH,
> - .engines = EIP97IES | EIP197B | EIP197D,
> .alg.ahash = {
> .init = safexcel_hmac_md5_init,
> .update = safexcel_ahash_update,
> --
> 1.8.3.1
--
Antoine Ténart, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
next prev parent reply other threads:[~2019-08-05 7:51 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-31 15:29 [PATCHv3 0/4] crypto: inside-secure - broaden driver scope Pascal van Leeuwen
2019-07-31 15:29 ` [PATCHv3 1/4] crypto: inside-secure - make driver selectable for non-Marvell hardware Pascal van Leeuwen
2019-08-05 8:04 ` Antoine Tenart
2019-07-31 15:29 ` [PATCHv3 2/4] crypto: inside-secure - Remove redundant algo to engine mapping code Pascal van Leeuwen
2019-08-05 7:51 ` Antoine Tenart [this message]
2019-07-31 15:29 ` [PATCHv3 3/4] crypto: inside-secure - add support for PCI based FPGA development board Pascal van Leeuwen
2019-08-05 8:36 ` Antoine Tenart
2019-08-05 8:47 ` Pascal Van Leeuwen
2019-08-05 9:09 ` Antoine Tenart
2019-08-05 9:49 ` Pascal Van Leeuwen
2019-07-31 15:29 ` [PATCHv3 4/4] crypto: inside-secure - add support for using the EIP197 without vendor firmware Pascal van Leeuwen
2019-08-05 9:07 ` Antoine Tenart
2019-08-05 9:48 ` Pascal Van Leeuwen
2019-08-05 9:54 ` Antoine Tenart
2019-08-05 10:12 ` Pascal Van Leeuwen
2019-08-05 12:43 ` Antoine Tenart
2019-08-05 13:02 ` Pascal Van Leeuwen
2019-08-05 13:12 ` Antoine Tenart
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190805075103.GE14470@kwain \
--to=antoine.tenart@bootlin.com \
--cc=davem@davemloft.net \
--cc=herbert@gondor.apana.org.au \
--cc=linux-crypto@vger.kernel.org \
--cc=pascalvanl@gmail.com \
--cc=pvanleeuwen@verimatrix.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox