Commit ce05ffe1 authored by Christian Lamparter's avatar Christian Lamparter Committed by Herbert Xu

crypto: crypto4xx - convert to skcipher

The ablkcipher APIs have been effectively deprecated since [1].
This patch converts the crypto4xx driver to the new skcipher APIs.

[1] <https://www.spinics.net/lists/linux-crypto/msg18133.html>
Signed-off-by: default avatarChristian Lamparter <chunkeey@gmail.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent a8d79d7b
...@@ -31,6 +31,7 @@ ...@@ -31,6 +31,7 @@
#include <crypto/gcm.h> #include <crypto/gcm.h>
#include <crypto/sha.h> #include <crypto/sha.h>
#include <crypto/ctr.h> #include <crypto/ctr.h>
#include <crypto/skcipher.h>
#include "crypto4xx_reg_def.h" #include "crypto4xx_reg_def.h"
#include "crypto4xx_core.h" #include "crypto4xx_core.h"
#include "crypto4xx_sa.h" #include "crypto4xx_sa.h"
...@@ -74,36 +75,37 @@ static void set_dynamic_sa_command_1(struct dynamic_sa_ctl *sa, u32 cm, ...@@ -74,36 +75,37 @@ static void set_dynamic_sa_command_1(struct dynamic_sa_ctl *sa, u32 cm,
sa->sa_command_1.bf.copy_hdr = cp_hdr; sa->sa_command_1.bf.copy_hdr = cp_hdr;
} }
static inline int crypto4xx_crypt(struct ablkcipher_request *req, static inline int crypto4xx_crypt(struct skcipher_request *req,
const unsigned int ivlen, bool decrypt) const unsigned int ivlen, bool decrypt)
{ {
struct crypto4xx_ctx *ctx = crypto_tfm_ctx(req->base.tfm); struct crypto_skcipher *cipher = crypto_skcipher_reqtfm(req);
struct crypto4xx_ctx *ctx = crypto_skcipher_ctx(cipher);
__le32 iv[ivlen]; __le32 iv[ivlen];
if (ivlen) if (ivlen)
crypto4xx_memcpy_to_le32(iv, req->info, ivlen); crypto4xx_memcpy_to_le32(iv, req->iv, ivlen);
return crypto4xx_build_pd(&req->base, ctx, req->src, req->dst, return crypto4xx_build_pd(&req->base, ctx, req->src, req->dst,
req->nbytes, iv, ivlen, decrypt ? ctx->sa_in : ctx->sa_out, req->cryptlen, iv, ivlen, decrypt ? ctx->sa_in : ctx->sa_out,
ctx->sa_len, 0); ctx->sa_len, 0);
} }
int crypto4xx_encrypt_noiv(struct ablkcipher_request *req) int crypto4xx_encrypt_noiv(struct skcipher_request *req)
{ {
return crypto4xx_crypt(req, 0, false); return crypto4xx_crypt(req, 0, false);
} }
int crypto4xx_encrypt_iv(struct ablkcipher_request *req) int crypto4xx_encrypt_iv(struct skcipher_request *req)
{ {
return crypto4xx_crypt(req, AES_IV_SIZE, false); return crypto4xx_crypt(req, AES_IV_SIZE, false);
} }
int crypto4xx_decrypt_noiv(struct ablkcipher_request *req) int crypto4xx_decrypt_noiv(struct skcipher_request *req)
{ {
return crypto4xx_crypt(req, 0, true); return crypto4xx_crypt(req, 0, true);
} }
int crypto4xx_decrypt_iv(struct ablkcipher_request *req) int crypto4xx_decrypt_iv(struct skcipher_request *req)
{ {
return crypto4xx_crypt(req, AES_IV_SIZE, true); return crypto4xx_crypt(req, AES_IV_SIZE, true);
} }
...@@ -111,20 +113,19 @@ int crypto4xx_decrypt_iv(struct ablkcipher_request *req) ...@@ -111,20 +113,19 @@ int crypto4xx_decrypt_iv(struct ablkcipher_request *req)
/** /**
* AES Functions * AES Functions
*/ */
static int crypto4xx_setkey_aes(struct crypto_ablkcipher *cipher, static int crypto4xx_setkey_aes(struct crypto_skcipher *cipher,
const u8 *key, const u8 *key,
unsigned int keylen, unsigned int keylen,
unsigned char cm, unsigned char cm,
u8 fb) u8 fb)
{ {
struct crypto_tfm *tfm = crypto_ablkcipher_tfm(cipher); struct crypto4xx_ctx *ctx = crypto_skcipher_ctx(cipher);
struct crypto4xx_ctx *ctx = crypto_tfm_ctx(tfm);
struct dynamic_sa_ctl *sa; struct dynamic_sa_ctl *sa;
int rc; int rc;
if (keylen != AES_KEYSIZE_256 && if (keylen != AES_KEYSIZE_256 &&
keylen != AES_KEYSIZE_192 && keylen != AES_KEYSIZE_128) { keylen != AES_KEYSIZE_192 && keylen != AES_KEYSIZE_128) {
crypto_ablkcipher_set_flags(cipher, crypto_skcipher_set_flags(cipher,
CRYPTO_TFM_RES_BAD_KEY_LEN); CRYPTO_TFM_RES_BAD_KEY_LEN);
return -EINVAL; return -EINVAL;
} }
...@@ -164,39 +165,38 @@ static int crypto4xx_setkey_aes(struct crypto_ablkcipher *cipher, ...@@ -164,39 +165,38 @@ static int crypto4xx_setkey_aes(struct crypto_ablkcipher *cipher,
return 0; return 0;
} }
int crypto4xx_setkey_aes_cbc(struct crypto_ablkcipher *cipher, int crypto4xx_setkey_aes_cbc(struct crypto_skcipher *cipher,
const u8 *key, unsigned int keylen) const u8 *key, unsigned int keylen)
{ {
return crypto4xx_setkey_aes(cipher, key, keylen, CRYPTO_MODE_CBC, return crypto4xx_setkey_aes(cipher, key, keylen, CRYPTO_MODE_CBC,
CRYPTO_FEEDBACK_MODE_NO_FB); CRYPTO_FEEDBACK_MODE_NO_FB);
} }
int crypto4xx_setkey_aes_cfb(struct crypto_ablkcipher *cipher, int crypto4xx_setkey_aes_cfb(struct crypto_skcipher *cipher,
const u8 *key, unsigned int keylen) const u8 *key, unsigned int keylen)
{ {
return crypto4xx_setkey_aes(cipher, key, keylen, CRYPTO_MODE_CFB, return crypto4xx_setkey_aes(cipher, key, keylen, CRYPTO_MODE_CFB,
CRYPTO_FEEDBACK_MODE_128BIT_CFB); CRYPTO_FEEDBACK_MODE_128BIT_CFB);
} }
int crypto4xx_setkey_aes_ecb(struct crypto_ablkcipher *cipher, int crypto4xx_setkey_aes_ecb(struct crypto_skcipher *cipher,
const u8 *key, unsigned int keylen) const u8 *key, unsigned int keylen)
{ {
return crypto4xx_setkey_aes(cipher, key, keylen, CRYPTO_MODE_ECB, return crypto4xx_setkey_aes(cipher, key, keylen, CRYPTO_MODE_ECB,
CRYPTO_FEEDBACK_MODE_NO_FB); CRYPTO_FEEDBACK_MODE_NO_FB);
} }
int crypto4xx_setkey_aes_ofb(struct crypto_ablkcipher *cipher, int crypto4xx_setkey_aes_ofb(struct crypto_skcipher *cipher,
const u8 *key, unsigned int keylen) const u8 *key, unsigned int keylen)
{ {
return crypto4xx_setkey_aes(cipher, key, keylen, CRYPTO_MODE_OFB, return crypto4xx_setkey_aes(cipher, key, keylen, CRYPTO_MODE_OFB,
CRYPTO_FEEDBACK_MODE_64BIT_OFB); CRYPTO_FEEDBACK_MODE_64BIT_OFB);
} }
int crypto4xx_setkey_rfc3686(struct crypto_ablkcipher *cipher, int crypto4xx_setkey_rfc3686(struct crypto_skcipher *cipher,
const u8 *key, unsigned int keylen) const u8 *key, unsigned int keylen)
{ {
struct crypto_tfm *tfm = crypto_ablkcipher_tfm(cipher); struct crypto4xx_ctx *ctx = crypto_skcipher_ctx(cipher);
struct crypto4xx_ctx *ctx = crypto_tfm_ctx(tfm);
int rc; int rc;
rc = crypto4xx_setkey_aes(cipher, key, keylen - CTR_RFC3686_NONCE_SIZE, rc = crypto4xx_setkey_aes(cipher, key, keylen - CTR_RFC3686_NONCE_SIZE,
...@@ -210,31 +210,33 @@ int crypto4xx_setkey_rfc3686(struct crypto_ablkcipher *cipher, ...@@ -210,31 +210,33 @@ int crypto4xx_setkey_rfc3686(struct crypto_ablkcipher *cipher,
return 0; return 0;
} }
int crypto4xx_rfc3686_encrypt(struct ablkcipher_request *req) int crypto4xx_rfc3686_encrypt(struct skcipher_request *req)
{ {
struct crypto4xx_ctx *ctx = crypto_tfm_ctx(req->base.tfm); struct crypto_skcipher *cipher = crypto_skcipher_reqtfm(req);
struct crypto4xx_ctx *ctx = crypto_skcipher_ctx(cipher);
__le32 iv[AES_IV_SIZE / 4] = { __le32 iv[AES_IV_SIZE / 4] = {
ctx->iv_nonce, ctx->iv_nonce,
cpu_to_le32p((u32 *) req->info), cpu_to_le32p((u32 *) req->iv),
cpu_to_le32p((u32 *) (req->info + 4)), cpu_to_le32p((u32 *) (req->iv + 4)),
cpu_to_le32(1) }; cpu_to_le32(1) };
return crypto4xx_build_pd(&req->base, ctx, req->src, req->dst, return crypto4xx_build_pd(&req->base, ctx, req->src, req->dst,
req->nbytes, iv, AES_IV_SIZE, req->cryptlen, iv, AES_IV_SIZE,
ctx->sa_out, ctx->sa_len, 0); ctx->sa_out, ctx->sa_len, 0);
} }
int crypto4xx_rfc3686_decrypt(struct ablkcipher_request *req) int crypto4xx_rfc3686_decrypt(struct skcipher_request *req)
{ {
struct crypto4xx_ctx *ctx = crypto_tfm_ctx(req->base.tfm); struct crypto_skcipher *cipher = crypto_skcipher_reqtfm(req);
struct crypto4xx_ctx *ctx = crypto_skcipher_ctx(cipher);
__le32 iv[AES_IV_SIZE / 4] = { __le32 iv[AES_IV_SIZE / 4] = {
ctx->iv_nonce, ctx->iv_nonce,
cpu_to_le32p((u32 *) req->info), cpu_to_le32p((u32 *) req->iv),
cpu_to_le32p((u32 *) (req->info + 4)), cpu_to_le32p((u32 *) (req->iv + 4)),
cpu_to_le32(1) }; cpu_to_le32(1) };
return crypto4xx_build_pd(&req->base, ctx, req->src, req->dst, return crypto4xx_build_pd(&req->base, ctx, req->src, req->dst,
req->nbytes, iv, AES_IV_SIZE, req->cryptlen, iv, AES_IV_SIZE,
ctx->sa_out, ctx->sa_len, 0); ctx->sa_out, ctx->sa_len, 0);
} }
......
This diff is collapsed.
...@@ -25,6 +25,7 @@ ...@@ -25,6 +25,7 @@
#include <linux/ratelimit.h> #include <linux/ratelimit.h>
#include <crypto/internal/hash.h> #include <crypto/internal/hash.h>
#include <crypto/internal/aead.h> #include <crypto/internal/aead.h>
#include <crypto/internal/skcipher.h>
#include "crypto4xx_reg_def.h" #include "crypto4xx_reg_def.h"
#include "crypto4xx_sa.h" #include "crypto4xx_sa.h"
...@@ -134,7 +135,7 @@ struct crypto4xx_ctx { ...@@ -134,7 +135,7 @@ struct crypto4xx_ctx {
struct crypto4xx_alg_common { struct crypto4xx_alg_common {
u32 type; u32 type;
union { union {
struct crypto_alg cipher; struct skcipher_alg cipher;
struct ahash_alg hash; struct ahash_alg hash;
struct aead_alg aead; struct aead_alg aead;
} u; } u;
...@@ -158,22 +159,22 @@ int crypto4xx_build_pd(struct crypto_async_request *req, ...@@ -158,22 +159,22 @@ int crypto4xx_build_pd(struct crypto_async_request *req,
const struct dynamic_sa_ctl *sa, const struct dynamic_sa_ctl *sa,
const unsigned int sa_len, const unsigned int sa_len,
const unsigned int assoclen); const unsigned int assoclen);
int crypto4xx_setkey_aes_cbc(struct crypto_ablkcipher *cipher, int crypto4xx_setkey_aes_cbc(struct crypto_skcipher *cipher,
const u8 *key, unsigned int keylen); const u8 *key, unsigned int keylen);
int crypto4xx_setkey_aes_cfb(struct crypto_ablkcipher *cipher, int crypto4xx_setkey_aes_cfb(struct crypto_skcipher *cipher,
const u8 *key, unsigned int keylen); const u8 *key, unsigned int keylen);
int crypto4xx_setkey_aes_ecb(struct crypto_ablkcipher *cipher, int crypto4xx_setkey_aes_ecb(struct crypto_skcipher *cipher,
const u8 *key, unsigned int keylen); const u8 *key, unsigned int keylen);
int crypto4xx_setkey_aes_ofb(struct crypto_ablkcipher *cipher, int crypto4xx_setkey_aes_ofb(struct crypto_skcipher *cipher,
const u8 *key, unsigned int keylen); const u8 *key, unsigned int keylen);
int crypto4xx_setkey_rfc3686(struct crypto_ablkcipher *cipher, int crypto4xx_setkey_rfc3686(struct crypto_skcipher *cipher,
const u8 *key, unsigned int keylen); const u8 *key, unsigned int keylen);
int crypto4xx_encrypt_iv(struct ablkcipher_request *req); int crypto4xx_encrypt_iv(struct skcipher_request *req);
int crypto4xx_decrypt_iv(struct ablkcipher_request *req); int crypto4xx_decrypt_iv(struct skcipher_request *req);
int crypto4xx_encrypt_noiv(struct ablkcipher_request *req); int crypto4xx_encrypt_noiv(struct skcipher_request *req);
int crypto4xx_decrypt_noiv(struct ablkcipher_request *req); int crypto4xx_decrypt_noiv(struct skcipher_request *req);
int crypto4xx_rfc3686_encrypt(struct ablkcipher_request *req); int crypto4xx_rfc3686_encrypt(struct skcipher_request *req);
int crypto4xx_rfc3686_decrypt(struct ablkcipher_request *req); int crypto4xx_rfc3686_decrypt(struct skcipher_request *req);
int crypto4xx_sha1_alg_init(struct crypto_tfm *tfm); int crypto4xx_sha1_alg_init(struct crypto_tfm *tfm);
int crypto4xx_hash_digest(struct ahash_request *req); int crypto4xx_hash_digest(struct ahash_request *req);
int crypto4xx_hash_final(struct ahash_request *req); int crypto4xx_hash_final(struct ahash_request *req);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment