Whamcloud - gitweb
git://git.whamcloud.com
/
fs
/
lustre-release.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
LU-9116 libcfs: avoid overflow of crypto bandwidth caculation
[fs/lustre-release.git]
/
libcfs
/
libcfs
/
linux
/
linux-crypto.c
diff --git
a/libcfs/libcfs/linux/linux-crypto.c
b/libcfs/libcfs/linux/linux-crypto.c
index
8a6c01f
..
1a0a105
100644
(file)
--- a/
libcfs/libcfs/linux/linux-crypto.c
+++ b/
libcfs/libcfs/linux/linux-crypto.c
@@
-27,11
+27,24
@@
* Copyright (c) 2012, 2014, Intel Corporation.
*/
* Copyright (c) 2012, 2014, Intel Corporation.
*/
-#include <
linux/crypto
.h>
+#include <
crypto/hash
.h>
#include <linux/scatterlist.h>
#include <libcfs/libcfs.h>
#include <libcfs/libcfs_crypto.h>
#include <libcfs/linux/linux-crypto.h>
#include <linux/scatterlist.h>
#include <libcfs/libcfs.h>
#include <libcfs/libcfs_crypto.h>
#include <libcfs/linux/linux-crypto.h>
+
+#ifndef HAVE_CRYPTO_HASH_HELPERS
+static inline const char *crypto_ahash_alg_name(struct crypto_ahash *tfm)
+{
+ return crypto_tfm_alg_name(crypto_ahash_tfm(tfm));
+}
+
+static inline const char *crypto_ahash_driver_name(struct crypto_ahash *tfm)
+{
+ return crypto_tfm_alg_driver_name(crypto_ahash_tfm(tfm));
+}
+#endif
+
/**
* Array of hash algorithm speed in MByte per second
*/
/**
* Array of hash algorithm speed in MByte per second
*/
@@
-47,7
+60,7
@@
static int cfs_crypto_hash_speeds[CFS_HASH_ALG_MAX];
*
* \param[in] hash_alg hash algorithm id (CFS_HASH_ALG_*)
* \param[out] type pointer to the hash description in hash_types[] array
*
* \param[in] hash_alg hash algorithm id (CFS_HASH_ALG_*)
* \param[out] type pointer to the hash description in hash_types[] array
- * \param[in,out]
hdesc hash state descriptor
to be initialized
+ * \param[in,out]
req ahash request
to be initialized
* \param[in] key initial hash value/state, NULL to use default value
* \param[in] key_len length of \a key
*
* \param[in] key initial hash value/state, NULL to use default value
* \param[in] key_len length of \a key
*
@@
-56,9
+69,11
@@
static int cfs_crypto_hash_speeds[CFS_HASH_ALG_MAX];
*/
static int cfs_crypto_hash_alloc(enum cfs_crypto_hash_alg hash_alg,
const struct cfs_crypto_hash_type **type,
*/
static int cfs_crypto_hash_alloc(enum cfs_crypto_hash_alg hash_alg,
const struct cfs_crypto_hash_type **type,
- struct hash_desc *hdesc, unsigned char *key,
+ struct ahash_request **req,
+ unsigned char *key,
unsigned int key_len)
{
unsigned int key_len)
{
+ struct crypto_ahash *tfm;
int err = 0;
*type = cfs_crypto_hash_type(hash_alg);
int err = 0;
*type = cfs_crypto_hash_type(hash_alg);
@@
-68,37
+83,46
@@
static int cfs_crypto_hash_alloc(enum cfs_crypto_hash_alg hash_alg,
hash_alg, CFS_HASH_ALG_MAX);
return -EINVAL;
}
hash_alg, CFS_HASH_ALG_MAX);
return -EINVAL;
}
- hdesc->tfm = crypto_alloc_hash((*type)->cht_name, 0, 0);
-
- if (hdesc->tfm == NULL)
- return -EINVAL;
-
- if (IS_ERR(hdesc->tfm)) {
+ tfm = crypto_alloc_ahash((*type)->cht_name, 0, CRYPTO_ALG_ASYNC);
+ if (IS_ERR(tfm)) {
CDEBUG(D_INFO, "Failed to alloc crypto hash %s\n",
(*type)->cht_name);
CDEBUG(D_INFO, "Failed to alloc crypto hash %s\n",
(*type)->cht_name);
- return PTR_ERR(
hdesc->
tfm);
+ return PTR_ERR(tfm);
}
}
- hdesc->flags = 0;
+ *req = ahash_request_alloc(tfm, GFP_KERNEL);
+ if (!*req) {
+ CDEBUG(D_INFO, "Failed to alloc ahash_request for %s\n",
+ (*type)->cht_name);
+ crypto_free_ahash(tfm);
+ return -ENOMEM;
+ }
+
+ ahash_request_set_callback(*req, 0, NULL, NULL);
- if (key
!= NULL
)
- err = crypto_
hash_setkey(hdesc->
tfm, key, key_len);
+ if (key)
+ err = crypto_
ahash_setkey(
tfm, key, key_len);
else if ((*type)->cht_key != 0)
else if ((*type)->cht_key != 0)
- err = crypto_
hash_setkey(hdesc->
tfm,
+ err = crypto_
ahash_setkey(
tfm,
(unsigned char *)&((*type)->cht_key),
(*type)->cht_size);
if (err != 0) {
(unsigned char *)&((*type)->cht_key),
(*type)->cht_size);
if (err != 0) {
- crypto_free_hash(hdesc->tfm);
+ ahash_request_free(*req);
+ crypto_free_ahash(tfm);
return err;
}
CDEBUG(D_INFO, "Using crypto hash: %s (%s) speed %d MB/s\n",
return err;
}
CDEBUG(D_INFO, "Using crypto hash: %s (%s) speed %d MB/s\n",
- (crypto_hash_tfm(hdesc->tfm))->__crt_alg->cra_name,
- (crypto_hash_tfm(hdesc->tfm))->__crt_alg->cra_driver_name,
+ crypto_ahash_alg_name(tfm), crypto_ahash_driver_name(tfm),
cfs_crypto_hash_speeds[hash_alg]);
cfs_crypto_hash_speeds[hash_alg]);
- return crypto_hash_init(hdesc);
+ err = crypto_ahash_init(*req);
+ if (err) {
+ ahash_request_free(*req);
+ crypto_free_ahash(tfm);
+ }
+ return err;
}
/**
}
/**
@@
-117,7
+141,7
@@
static int cfs_crypto_hash_alloc(enum cfs_crypto_hash_alg hash_alg,
* \a hash_len is to digest size in bytes, retval -ENOSPC
* \param[in,out] hash_len size of \a hash buffer
*
* \a hash_len is to digest size in bytes, retval -ENOSPC
* \param[in,out] hash_len size of \a hash buffer
*
- * \retval -EINVAL \a buf, \a buf_len, \a hash_len, \a
alg_id
invalid
+ * \retval -EINVAL \a buf, \a buf_len, \a hash_len, \a
hash_alg
invalid
* \retval -ENOENT \a hash_alg is unsupported
* \retval -ENOSPC \a hash is NULL, or \a hash_len less than digest size
* \retval 0 for success
* \retval -ENOENT \a hash_alg is unsupported
* \retval -ENOSPC \a hash is NULL, or \a hash_len less than digest size
* \retval 0 for success
@@
-129,27
+153,29
@@
int cfs_crypto_hash_digest(enum cfs_crypto_hash_alg hash_alg,
unsigned char *hash, unsigned int *hash_len)
{
struct scatterlist sl;
unsigned char *hash, unsigned int *hash_len)
{
struct scatterlist sl;
- struct
hash_desc hdesc
;
+ struct
ahash_request *req
;
int err;
const struct cfs_crypto_hash_type *type;
int err;
const struct cfs_crypto_hash_type *type;
- if (
buf == NULL || buf_len == 0 || hash_len == NULL
)
+ if (
!buf || buf_len == 0 || !hash_len
)
return -EINVAL;
return -EINVAL;
- err = cfs_crypto_hash_alloc(hash_alg, &type, &
hdesc
, key, key_len);
+ err = cfs_crypto_hash_alloc(hash_alg, &type, &
req
, key, key_len);
if (err != 0)
return err;
if (err != 0)
return err;
- if (
hash == NULL
|| *hash_len < type->cht_size) {
+ if (
!hash
|| *hash_len < type->cht_size) {
*hash_len = type->cht_size;
*hash_len = type->cht_size;
- crypto_free_hash(hdesc.tfm);
+ crypto_free_ahash(crypto_ahash_reqtfm(req));
+ ahash_request_free(req);
return -ENOSPC;
}
sg_init_one(&sl, (void *)buf, buf_len);
return -ENOSPC;
}
sg_init_one(&sl, (void *)buf, buf_len);
- hdesc.flags = 0;
- err = crypto_hash_digest(&hdesc, &sl, sl.length, hash);
- crypto_free_hash(hdesc.tfm);
+ ahash_request_set_crypt(req, &sl, hash, sl.length);
+ err = crypto_ahash_digest(req);
+ crypto_free_ahash(crypto_ahash_reqtfm(req));
+ ahash_request_free(req);
return err;
}
return err;
}
@@
-176,22
+202,14
@@
struct cfs_crypto_hash_desc *
cfs_crypto_hash_init(enum cfs_crypto_hash_alg hash_alg,
unsigned char *key, unsigned int key_len)
{
cfs_crypto_hash_init(enum cfs_crypto_hash_alg hash_alg,
unsigned char *key, unsigned int key_len)
{
-
- struct hash_desc *hdesc;
+ struct ahash_request *req;
int err;
const struct cfs_crypto_hash_type *type;
int err;
const struct cfs_crypto_hash_type *type;
- hdesc = kmalloc(sizeof(*hdesc), 0);
- if (hdesc == NULL)
- return ERR_PTR(-ENOMEM);
-
- err = cfs_crypto_hash_alloc(hash_alg, &type, hdesc, key, key_len);
-
- if (err) {
- kfree(hdesc);
- hdesc = ERR_PTR(err);
- }
- return (struct cfs_crypto_hash_desc *)hdesc;
+ err = cfs_crypto_hash_alloc(hash_alg, &type, &req, key, key_len);
+ if (err)
+ return ERR_PTR(err);
+ return (struct cfs_crypto_hash_desc *)req;
}
EXPORT_SYMBOL(cfs_crypto_hash_init);
}
EXPORT_SYMBOL(cfs_crypto_hash_init);
@@
-210,12
+228,14
@@
int cfs_crypto_hash_update_page(struct cfs_crypto_hash_desc *hdesc,
struct page *page, unsigned int offset,
unsigned int len)
{
struct page *page, unsigned int offset,
unsigned int len)
{
+ struct ahash_request *req = (void *)hdesc;
struct scatterlist sl;
sg_init_table(&sl, 1);
sg_set_page(&sl, page, len, offset & ~PAGE_MASK);
struct scatterlist sl;
sg_init_table(&sl, 1);
sg_set_page(&sl, page, len, offset & ~PAGE_MASK);
- return crypto_hash_update((struct hash_desc *)hdesc, &sl, sl.length);
+ ahash_request_set_crypt(req, &sl, NULL, sl.length);
+ return crypto_ahash_update(req);
}
EXPORT_SYMBOL(cfs_crypto_hash_update_page);
}
EXPORT_SYMBOL(cfs_crypto_hash_update_page);
@@
-232,11
+252,13
@@
EXPORT_SYMBOL(cfs_crypto_hash_update_page);
int cfs_crypto_hash_update(struct cfs_crypto_hash_desc *hdesc,
const void *buf, unsigned int buf_len)
{
int cfs_crypto_hash_update(struct cfs_crypto_hash_desc *hdesc,
const void *buf, unsigned int buf_len)
{
+ struct ahash_request *req = (void *)hdesc;
struct scatterlist sl;
sg_init_one(&sl, (void *)buf, buf_len);
struct scatterlist sl;
sg_init_one(&sl, (void *)buf, buf_len);
- return crypto_hash_update((struct hash_desc *)hdesc, &sl, sl.length);
+ ahash_request_set_crypt(req, &sl, NULL, sl.length);
+ return crypto_ahash_update(req);
}
EXPORT_SYMBOL(cfs_crypto_hash_update);
}
EXPORT_SYMBOL(cfs_crypto_hash_update);
@@
-256,10
+278,11
@@
EXPORT_SYMBOL(cfs_crypto_hash_update);
int cfs_crypto_hash_final(struct cfs_crypto_hash_desc *hdesc,
unsigned char *hash, unsigned int *hash_len)
{
int cfs_crypto_hash_final(struct cfs_crypto_hash_desc *hdesc,
unsigned char *hash, unsigned int *hash_len)
{
- int size = crypto_hash_digestsize(((struct hash_desc *)hdesc)->tfm);
- int err;
+ struct ahash_request *req = (void *)hdesc;
+ int size = crypto_ahash_digestsize(crypto_ahash_reqtfm(req));
+ int err;
- if (
hash == NULL || hash_len == NULL
) {
+ if (
!hash || !hash_len
) {
err = 0;
goto free;
}
err = 0;
goto free;
}
@@
-268,12
+291,13
@@
int cfs_crypto_hash_final(struct cfs_crypto_hash_desc *hdesc,
goto free;
}
goto free;
}
- err = crypto_hash_final((struct hash_desc *)hdesc, hash);
+ ahash_request_set_crypt(req, NULL, hash, 0);
+ err = crypto_ahash_final(req);
if (err == 0)
*hash_len = size;
free:
if (err == 0)
*hash_len = size;
free:
- crypto_free_
hash(((struct hash_desc *)hdesc)->tfm
);
-
kfree(hdesc
);
+ crypto_free_
ahash(crypto_ahash_reqtfm(req)
);
+
ahash_request_free(req
);
return err;
}
return err;
}
@@
-295,7
+319,8
@@
static void cfs_crypto_performance_test(enum cfs_crypto_hash_alg hash_alg)
int buf_len = max(PAGE_SIZE, 1048576UL);
void *buf;
unsigned long start, end;
int buf_len = max(PAGE_SIZE, 1048576UL);
void *buf;
unsigned long start, end;
- int bcount, err = 0;
+ int err = 0;
+ unsigned long bcount;
struct page *page;
unsigned char hash[CFS_CRYPTO_HASH_DIGESTSIZE_MAX];
unsigned int hash_len = sizeof(hash);
struct page *page;
unsigned char hash[CFS_CRYPTO_HASH_DIGESTSIZE_MAX];
unsigned int hash_len = sizeof(hash);