Whamcloud - gitweb
LU-5053 libcfs: clean up cfs_crypto_hash code
[fs/lustre-release.git] / libcfs / libcfs / linux / linux-crypto.c
index 8398e9e..b5f376d 100644 (file)
 #include <linux/crypto.h>
 #include <linux/scatterlist.h>
 #include <libcfs/libcfs.h>
+#include <libcfs/libcfs_crypto.h>
 #include <libcfs/linux/linux-crypto.h>
 /**
- *  Array of  hash algorithm speed in MByte per second
+ *  Array of hash algorithm speed in MByte per second
  */
 static int cfs_crypto_hash_speeds[CFS_HASH_ALG_MAX];
 
-static int cfs_crypto_hash_alloc(unsigned char alg_id,
+/**
+ * Initialize the state descriptor for the specified hash algorithm.
+ *
+ * An internal routine to allocate the hash-specific state in \a hdesc for
+ * use with cfs_crypto_hash_digest() to compute the hash of a single message,
+ * though possibly in multiple chunks.  The descriptor internal state should
+ * be freed with cfs_crypto_hash_final().
+ *
+ * \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]  key     initial hash value/state, NULL to use default value
+ * \param[in]  key_len length of \a key
+ *
+ * \retval             0 on success
+ * \retval             negative errno on failure
+ */
+static int cfs_crypto_hash_alloc(enum cfs_crypto_hash_alg hash_alg,
                                 const struct cfs_crypto_hash_type **type,
-                                struct hash_desc *desc, unsigned char *key,
+                                struct hash_desc *hdesc, unsigned char *key,
                                 unsigned int key_len)
 {
-       int     err = 0;
+       int err = 0;
 
-       *type = cfs_crypto_hash_type(alg_id);
+       *type = cfs_crypto_hash_type(hash_alg);
 
        if (*type == NULL) {
                CWARN("Unsupported hash algorithm id = %d, max id is %d\n",
-                     alg_id, CFS_HASH_ALG_MAX);
+                     hash_alg, CFS_HASH_ALG_MAX);
                return -EINVAL;
        }
-       desc->tfm = crypto_alloc_hash((*type)->cht_name, 0, 0);
+       hdesc->tfm = crypto_alloc_hash((*type)->cht_name, 0, 0);
 
-       if (desc->tfm == NULL)
+       if (hdesc->tfm == NULL)
                return -EINVAL;
 
-       if (IS_ERR(desc->tfm)) {
+       if (IS_ERR(hdesc->tfm)) {
                CDEBUG(D_INFO, "Failed to alloc crypto hash %s\n",
                       (*type)->cht_name);
-               return PTR_ERR(desc->tfm);
+               return PTR_ERR(hdesc->tfm);
        }
 
-       desc->flags = 0;
+       hdesc->flags = 0;
 
-       if (key != NULL) {
-               err = crypto_hash_setkey(desc->tfm, key, key_len);
-       } else if ((*type)->cht_key != 0) {
-               err = crypto_hash_setkey(desc->tfm,
+       if (key != NULL)
+               err = crypto_hash_setkey(hdesc->tfm, key, key_len);
+       else if ((*type)->cht_key != 0)
+               err = crypto_hash_setkey(hdesc->tfm,
                                         (unsigned char *)&((*type)->cht_key),
                                         (*type)->cht_size);
-       }
 
        if (err != 0) {
-               crypto_free_hash(desc->tfm);
+               crypto_free_hash(hdesc->tfm);
                return err;
        }
 
        CDEBUG(D_INFO, "Using crypto hash: %s (%s) speed %d MB/s\n",
-              (crypto_hash_tfm(desc->tfm))->__crt_alg->cra_name,
-              (crypto_hash_tfm(desc->tfm))->__crt_alg->cra_driver_name,
-              cfs_crypto_hash_speeds[alg_id]);
+              (crypto_hash_tfm(hdesc->tfm))->__crt_alg->cra_name,
+              (crypto_hash_tfm(hdesc->tfm))->__crt_alg->cra_driver_name,
+              cfs_crypto_hash_speeds[hash_alg]);
 
-       return crypto_hash_init(desc);
+       return crypto_hash_init(hdesc);
 }
 
-int cfs_crypto_hash_digest(unsigned char alg_id,
+/**
+ * Calculate hash digest for the passed buffer.
+ *
+ * This should be used when computing the hash on a single contiguous buffer.
+ * It combines the hash initialization, computation, and cleanup.
+ *
+ * \param[in] hash_alg id of hash algorithm (CFS_HASH_ALG_*)
+ * \param[in] buf      data buffer on which to compute hash
+ * \param[in] buf_len  length of \a buf in bytes
+ * \param[in] key      initial value/state for algorithm, if \a key = NULL
+ *                     use default initial value
+ * \param[in] key_len  length of \a key in bytes
+ * \param[out] hash    pointer to computed hash value, if \a hash = NULL then
+ *                     \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 -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             negative errno for other errors from lower layers.
+ */
+int cfs_crypto_hash_digest(enum cfs_crypto_hash_alg hash_alg,
                           const void *buf, unsigned int buf_len,
                           unsigned char *key, unsigned int key_len,
                           unsigned char *hash, unsigned int *hash_len)
@@ -97,7 +136,7 @@ int cfs_crypto_hash_digest(unsigned char alg_id,
        if (buf == NULL || buf_len == 0 || hash_len == NULL)
                return -EINVAL;
 
-       err = cfs_crypto_hash_alloc(alg_id, &type, &hdesc, key, key_len);
+       err = cfs_crypto_hash_alloc(hash_alg, &type, &hdesc, key, key_len);
        if (err != 0)
                return err;
 
@@ -116,29 +155,57 @@ int cfs_crypto_hash_digest(unsigned char alg_id,
 }
 EXPORT_SYMBOL(cfs_crypto_hash_digest);
 
+/**
+ * Allocate and initialize desriptor for hash algorithm.
+ *
+ * This should be used to initialize a hash descriptor for multiple calls
+ * to a single hash function when computing the hash across multiple
+ * separate buffers or pages using cfs_crypto_hash_update{,_page}().
+ *
+ * The hash descriptor should be freed with cfs_crypto_hash_final().
+ *
+ * \param[in] hash_alg algorithm id (CFS_HASH_ALG_*)
+ * \param[in] key      initial value/state for algorithm, if \a key = NULL
+ *                     use default initial value
+ * \param[in] key_len  length of \a key in bytes
+ *
+ * \retval             pointer to descriptor of hash instance
+ * \retval             ERR_PTR(errno) in case of error
+ */
 struct cfs_crypto_hash_desc *
-       cfs_crypto_hash_init(unsigned char alg_id,
+       cfs_crypto_hash_init(enum cfs_crypto_hash_alg hash_alg,
                             unsigned char *key, unsigned int key_len)
 {
 
-       struct  hash_desc       *hdesc;
-       int                  err;
+       struct  hash_desc                       *hdesc;
+       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(alg_id, &type, hdesc, key, key_len);
+       err = cfs_crypto_hash_alloc(hash_alg, &type, hdesc, key, key_len);
 
        if (err) {
                kfree(hdesc);
-               return ERR_PTR(err);
+               hdesc = ERR_PTR(err);
        }
        return (struct cfs_crypto_hash_desc *)hdesc;
 }
 EXPORT_SYMBOL(cfs_crypto_hash_init);
 
+/**
+ * Update hash digest computed on data within the given \a page
+ *
+ * \param[in] hdesc    hash state descriptor
+ * \param[in] page     data page on which to compute the hash
+ * \param[in] offset   offset within \a page at which to start hash
+ * \param[in] len      length of data on which to compute hash
+ *
+ * \retval             0 for success
+ * \retval             negative errno on failure
+ */
 int cfs_crypto_hash_update_page(struct cfs_crypto_hash_desc *hdesc,
                                struct page *page, unsigned int offset,
                                unsigned int len)
@@ -152,6 +219,16 @@ int cfs_crypto_hash_update_page(struct cfs_crypto_hash_desc *hdesc,
 }
 EXPORT_SYMBOL(cfs_crypto_hash_update_page);
 
+/**
+ * Update hash digest computed on the specified data
+ *
+ * \param[in] hdesc    hash state descriptor
+ * \param[in] buf      data buffer on which to compute the hash
+ * \param[in] buf_len  length of \buf on which to compute hash
+ *
+ * \retval             0 for success
+ * \retval             negative errno on failure
+ */
 int cfs_crypto_hash_update(struct cfs_crypto_hash_desc *hdesc,
                           const void *buf, unsigned int buf_len)
 {
@@ -163,102 +240,146 @@ int cfs_crypto_hash_update(struct cfs_crypto_hash_desc *hdesc,
 }
 EXPORT_SYMBOL(cfs_crypto_hash_update);
 
-/*      If hash_len pointer is NULL - destroy descriptor. */
+/**
+ * Finish hash calculation, copy hash digest to buffer, clean up hash descriptor
+ *
+ * \param[in] hdesc    hash descriptor
+ * \param[out] hash    pointer to hash buffer to store hash digest
+ * \param[in,out] hash_len pointer to hash buffer size, if \a hdesc = NULL
+ *                     only free \a hdesc instead of computing the hash
+ *
+ * \retval             -ENOSPC if \a hash = NULL, or \a hash_len < digest size
+ * \retval             0 for success
+ * \retval             negative errno for other errors from lower layers
+ */
 int cfs_crypto_hash_final(struct cfs_crypto_hash_desc *hdesc,
                          unsigned char *hash, unsigned int *hash_len)
 {
-       int     err;
        int     size = crypto_hash_digestsize(((struct hash_desc *)hdesc)->tfm);
+       int     err;
 
        if (hash_len == NULL) {
-               crypto_free_hash(((struct hash_desc *)hdesc)->tfm);
-               kfree(hdesc);
-               return 0;
+               err = 0;
+               goto free;
        }
        if (hash == NULL || *hash_len < size) {
-               *hash_len = size;
-               return -ENOSPC;
-       }
-       err = crypto_hash_final((struct hash_desc *) hdesc, hash);
-
-       if (err < 0) {
-               /* May be caller can fix error */
-               return err;
+               err = -ENOSPC;
+               goto free;
        }
+       err = crypto_hash_final((struct hash_desc *)hdesc, hash);
+free:
        crypto_free_hash(((struct hash_desc *)hdesc)->tfm);
        kfree(hdesc);
+
        return err;
 }
 EXPORT_SYMBOL(cfs_crypto_hash_final);
 
-static void cfs_crypto_performance_test(unsigned char alg_id,
+/**
+ * Compute the speed of specified hash function
+ *
+ * Run a speed test on the given hash algorithm on buffer of the given size.
+ * The speed is stored internally in the cfs_crypto_hash_speeds[] array, and
+ * is available through the cfs_crypto_hash_speed() function.
+ *
+ * \param[in] hash_alg hash algorithm id (CFS_HASH_ALG_*)
+ * \param[in] buf      data buffer on which to compute the hash
+ * \param[in] buf_len  length of \buf on which to compute hash
+ */
+static void cfs_crypto_performance_test(enum cfs_crypto_hash_alg hash_alg,
                                        const unsigned char *buf,
                                        unsigned int buf_len)
 {
-       unsigned long              start, end;
-       int                          bcount, err = 0;
-       int                          sec = 1; /* do test only 1 sec */
-       unsigned char              hash[64];
-       unsigned int                hash_len = 64;
+       unsigned long           start, end;
+       int                     bcount, err = 0;
+       int                     sec = 1; /* do test only 1 sec */
+       unsigned char           hash[64];
+       unsigned int            hash_len = sizeof(hash);
 
        for (start = jiffies, end = start + sec * HZ, bcount = 0;
             time_before(jiffies, end); bcount++) {
-               err = cfs_crypto_hash_digest(alg_id, buf, buf_len, NULL, 0,
+               err = cfs_crypto_hash_digest(hash_alg, buf, buf_len, NULL, 0,
                                             hash, &hash_len);
-               if (err)
+               if (err != 0)
                        break;
 
        }
        end = jiffies;
 
-       if (err) {
-               cfs_crypto_hash_speeds[alg_id] =  -1;
-               CDEBUG(D_INFO, "Crypto hash algorithm %s, err = %d\n",
-                      cfs_crypto_hash_name(alg_id), err);
+       if (err != 0) {
+               cfs_crypto_hash_speeds[hash_alg] = err;
+               CDEBUG(D_INFO, "Crypto hash algorithm %s test error: rc = %d\n",
+                      cfs_crypto_hash_name(hash_alg), err);
        } else {
                unsigned long   tmp;
+
                tmp = ((bcount * buf_len / jiffies_to_msecs(end - start)) *
                       1000) / (1024 * 1024);
-               cfs_crypto_hash_speeds[alg_id] = (int)tmp;
+               cfs_crypto_hash_speeds[hash_alg] = (int)tmp;
+               CDEBUG(D_CONFIG, "Crypto hash algorithm %s speed = %d MB/s\n",
+                      cfs_crypto_hash_name(hash_alg),
+                      cfs_crypto_hash_speeds[hash_alg]);
        }
-       CDEBUG(D_CONFIG, "Crypto hash algorithm %s speed = %d MB/s\n",
-              cfs_crypto_hash_name(alg_id), cfs_crypto_hash_speeds[alg_id]);
 }
 
-int cfs_crypto_hash_speed(unsigned char hash_alg)
+/**
+ * hash speed in Mbytes per second for valid hash algorithm
+ *
+ * Return the performance of the specified \a hash_alg that was previously
+ * computed using cfs_crypto_performance_test().
+ *
+ * \param[in] hash_alg hash algorithm id (CFS_HASH_ALG_*)
+ *
+ * \retval             positive speed of the hash function in MB/s
+ * \retval             -ENOENT if \a hash_alg is unsupported
+ * \retval             negative errno if \a hash_alg speed is unavailable
+ */
+int cfs_crypto_hash_speed(enum cfs_crypto_hash_alg hash_alg)
 {
        if (hash_alg < CFS_HASH_ALG_MAX)
                return cfs_crypto_hash_speeds[hash_alg];
-       else
-               return -1;
+
+       return -ENOENT;
 }
 EXPORT_SYMBOL(cfs_crypto_hash_speed);
 
 /**
- * Do performance test for all hash algorithms.
+ * Run the performance test for all hash algorithms.
+ *
+ * Run the cfs_crypto_performance_test() benchmark for all of the available
+ * hash functions using a 1MB buffer size.  This is a reasonable buffer size
+ * for Lustre RPCs, even if the actual RPC size is larger or smaller.
+ *
+ * Since the setup cost and computation speed of various hash algorithms is
+ * a function of the buffer size (and possibly internal contention of offload
+ * engines), this speed only represents an estimate of the actual speed under
+ * actual usage, but is reasonable for comparing available algorithms.
+ *
+ * The actual speeds are available via cfs_crypto_hash_speed() for later
+ * comparison.
+ *
+ * \retval             0 on success
+ * \retval             -ENOMEM if no memory is available for test buffer
  */
 static int cfs_crypto_test_hashes(void)
 {
-       unsigned char      i;
-       unsigned char      *data;
-       unsigned int        j;
-       /* Data block size for testing hash. Maximum
-        * kmalloc size for 2.6.18 kernel is 128K */
-       unsigned int        data_len = 1 * 128 * 1024;
-
-       data = kmalloc(data_len, 0);
+       enum cfs_crypto_hash_alg hash_alg;
+       unsigned char           *data;
+       /* Data block size for testing hash. Use bulk RPC size. */
+       unsigned int             data_len = 1024 * 1024;
+
+       data = vmalloc(data_len);
        if (data == NULL) {
-               CERROR("Failed to allocate mem\n");
+               CERROR("Failed to allocate buffer for hash speed test\n");
                return -ENOMEM;
        }
 
-       for (j = 0; j < data_len; j++)
-               data[j] = j & 0xff;
+       memset(data, 0xAD, data_len);
 
-       for (i = 0; i < CFS_HASH_ALG_MAX; i++)
-               cfs_crypto_performance_test(i, data, data_len);
+       for (hash_alg = 0; hash_alg < CFS_HASH_ALG_MAX; hash_alg++)
+               cfs_crypto_performance_test(hash_alg, data, data_len);
 
-       kfree(data);
+       vfree(data);
        return 0;
 }
 
@@ -269,13 +390,18 @@ static int crc32;
 #endif
 #ifdef HAVE_PCLMULQDQ
 #ifdef NEED_CRC32_ACCEL
-static int crc32pclmul;
+static int crc32_pclmul;
 #endif
 #ifdef NEED_CRC32C_ACCEL
 static int crc32c_pclmul;
 #endif
-#endif
+#endif /* HAVE_PCLMULQDQ */
 
+/**
+ * Register available hash functions
+ *
+ * \retval             0
+ */
 int cfs_crypto_register(void)
 {
        request_module("crc32c");
@@ -287,16 +413,22 @@ int cfs_crypto_register(void)
 #endif
 #ifdef HAVE_PCLMULQDQ
 #ifdef NEED_CRC32_ACCEL
-       crc32pclmul = cfs_crypto_crc32_pclmul_register();
+       crc32_pclmul = cfs_crypto_crc32_pclmul_register();
 #endif
 #ifdef NEED_CRC32C_ACCEL
        crc32c_pclmul = cfs_crypto_crc32c_pclmul_register();
 #endif
-#endif
+#endif /* HAVE_PCLMULQDQ */
+
        /* check all algorithms and do performance test */
        cfs_crypto_test_hashes();
+
        return 0;
 }
+
+/**
+ * Unregister previously registered hash functions
+ */
 void cfs_crypto_unregister(void)
 {
        if (adler32 == 0)
@@ -308,13 +440,12 @@ void cfs_crypto_unregister(void)
 #endif
 #ifdef HAVE_PCLMULQDQ
 #ifdef NEED_CRC32_ACCEL
-       if (crc32pclmul == 0)
+       if (crc32_pclmul == 0)
                cfs_crypto_crc32_pclmul_unregister();
 #endif
 #ifdef NEED_CRC32C_ACCEL
        if (crc32c_pclmul == 0)
                cfs_crypto_crc32c_pclmul_unregister();
 #endif
-#endif
-       return;
+#endif /* HAVE_PCLMULQDQ */
 }