Re: [PATCH] crypto: tcrypt - remove all multibuffer ahash tests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 28 Jan 2022 at 09:38, Tianjia Zhang
<tianjia.zhang@xxxxxxxxxxxxxxxxx> wrote:
>
> The multibuffer algorithms was removed already in 2018, so it is
> necessary to clear the test code left by tcrypt.
>
> Suggested-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Tianjia Zhang <tianjia.zhang@xxxxxxxxxxxxxxxxx>

Acked-by: Ard Biesheuvel <ardb@xxxxxxxxxx>

> ---
>  crypto/tcrypt.c | 224 ------------------------------------------------
>  1 file changed, 224 deletions(-)
>
> diff --git a/crypto/tcrypt.c b/crypto/tcrypt.c
> index 82b5eef2246a..2a808e843de5 100644
> --- a/crypto/tcrypt.c
> +++ b/crypto/tcrypt.c
> @@ -724,200 +724,6 @@ static inline int do_one_ahash_op(struct ahash_request *req, int ret)
>         return crypto_wait_req(ret, wait);
>  }
>
> -struct test_mb_ahash_data {
> -       struct scatterlist sg[XBUFSIZE];
> -       char result[64];
> -       struct ahash_request *req;
> -       struct crypto_wait wait;
> -       char *xbuf[XBUFSIZE];
> -};
> -
> -static inline int do_mult_ahash_op(struct test_mb_ahash_data *data, u32 num_mb,
> -                                  int *rc)
> -{
> -       int i, err = 0;
> -
> -       /* Fire up a bunch of concurrent requests */
> -       for (i = 0; i < num_mb; i++)
> -               rc[i] = crypto_ahash_digest(data[i].req);
> -
> -       /* Wait for all requests to finish */
> -       for (i = 0; i < num_mb; i++) {
> -               rc[i] = crypto_wait_req(rc[i], &data[i].wait);
> -
> -               if (rc[i]) {
> -                       pr_info("concurrent request %d error %d\n", i, rc[i]);
> -                       err = rc[i];
> -               }
> -       }
> -
> -       return err;
> -}
> -
> -static int test_mb_ahash_jiffies(struct test_mb_ahash_data *data, int blen,
> -                                int secs, u32 num_mb)
> -{
> -       unsigned long start, end;
> -       int bcount;
> -       int ret = 0;
> -       int *rc;
> -
> -       rc = kcalloc(num_mb, sizeof(*rc), GFP_KERNEL);
> -       if (!rc)
> -               return -ENOMEM;
> -
> -       for (start = jiffies, end = start + secs * HZ, bcount = 0;
> -            time_before(jiffies, end); bcount++) {
> -               ret = do_mult_ahash_op(data, num_mb, rc);
> -               if (ret)
> -                       goto out;
> -       }
> -
> -       pr_cont("%d operations in %d seconds (%llu bytes)\n",
> -               bcount * num_mb, secs, (u64)bcount * blen * num_mb);
> -
> -out:
> -       kfree(rc);
> -       return ret;
> -}
> -
> -static int test_mb_ahash_cycles(struct test_mb_ahash_data *data, int blen,
> -                               u32 num_mb)
> -{
> -       unsigned long cycles = 0;
> -       int ret = 0;
> -       int i;
> -       int *rc;
> -
> -       rc = kcalloc(num_mb, sizeof(*rc), GFP_KERNEL);
> -       if (!rc)
> -               return -ENOMEM;
> -
> -       /* Warm-up run. */
> -       for (i = 0; i < 4; i++) {
> -               ret = do_mult_ahash_op(data, num_mb, rc);
> -               if (ret)
> -                       goto out;
> -       }
> -
> -       /* The real thing. */
> -       for (i = 0; i < 8; i++) {
> -               cycles_t start, end;
> -
> -               start = get_cycles();
> -               ret = do_mult_ahash_op(data, num_mb, rc);
> -               end = get_cycles();
> -
> -               if (ret)
> -                       goto out;
> -
> -               cycles += end - start;
> -       }
> -
> -       pr_cont("1 operation in %lu cycles (%d bytes)\n",
> -               (cycles + 4) / (8 * num_mb), blen);
> -
> -out:
> -       kfree(rc);
> -       return ret;
> -}
> -
> -static void test_mb_ahash_speed(const char *algo, unsigned int secs,
> -                               struct hash_speed *speed, u32 num_mb)
> -{
> -       struct test_mb_ahash_data *data;
> -       struct crypto_ahash *tfm;
> -       unsigned int i, j, k;
> -       int ret;
> -
> -       data = kcalloc(num_mb, sizeof(*data), GFP_KERNEL);
> -       if (!data)
> -               return;
> -
> -       tfm = crypto_alloc_ahash(algo, 0, 0);
> -       if (IS_ERR(tfm)) {
> -               pr_err("failed to load transform for %s: %ld\n",
> -                       algo, PTR_ERR(tfm));
> -               goto free_data;
> -       }
> -
> -       for (i = 0; i < num_mb; ++i) {
> -               if (testmgr_alloc_buf(data[i].xbuf))
> -                       goto out;
> -
> -               crypto_init_wait(&data[i].wait);
> -
> -               data[i].req = ahash_request_alloc(tfm, GFP_KERNEL);
> -               if (!data[i].req) {
> -                       pr_err("alg: hash: Failed to allocate request for %s\n",
> -                              algo);
> -                       goto out;
> -               }
> -
> -               ahash_request_set_callback(data[i].req, 0, crypto_req_done,
> -                                          &data[i].wait);
> -
> -               sg_init_table(data[i].sg, XBUFSIZE);
> -               for (j = 0; j < XBUFSIZE; j++) {
> -                       sg_set_buf(data[i].sg + j, data[i].xbuf[j], PAGE_SIZE);
> -                       memset(data[i].xbuf[j], 0xff, PAGE_SIZE);
> -               }
> -       }
> -
> -       pr_info("\ntesting speed of multibuffer %s (%s)\n", algo,
> -               get_driver_name(crypto_ahash, tfm));
> -
> -       for (i = 0; speed[i].blen != 0; i++) {
> -               /* For some reason this only tests digests. */
> -               if (speed[i].blen != speed[i].plen)
> -                       continue;
> -
> -               if (speed[i].blen > XBUFSIZE * PAGE_SIZE) {
> -                       pr_err("template (%u) too big for tvmem (%lu)\n",
> -                              speed[i].blen, XBUFSIZE * PAGE_SIZE);
> -                       goto out;
> -               }
> -
> -               if (klen)
> -                       crypto_ahash_setkey(tfm, tvmem[0], klen);
> -
> -               for (k = 0; k < num_mb; k++)
> -                       ahash_request_set_crypt(data[k].req, data[k].sg,
> -                                               data[k].result, speed[i].blen);
> -
> -               pr_info("test%3u "
> -                       "(%5u byte blocks,%5u bytes per update,%4u updates): ",
> -                       i, speed[i].blen, speed[i].plen,
> -                       speed[i].blen / speed[i].plen);
> -
> -               if (secs) {
> -                       ret = test_mb_ahash_jiffies(data, speed[i].blen, secs,
> -                                                   num_mb);
> -                       cond_resched();
> -               } else {
> -                       ret = test_mb_ahash_cycles(data, speed[i].blen, num_mb);
> -               }
> -
> -
> -               if (ret) {
> -                       pr_err("At least one hashing failed ret=%d\n", ret);
> -                       break;
> -               }
> -       }
> -
> -out:
> -       for (k = 0; k < num_mb; ++k)
> -               ahash_request_free(data[k].req);
> -
> -       for (k = 0; k < num_mb; ++k)
> -               testmgr_free_buf(data[k].xbuf);
> -
> -       crypto_free_ahash(tfm);
> -
> -free_data:
> -       kfree(data);
> -}
> -
>  static int test_ahash_jiffies_digest(struct ahash_request *req, int blen,
>                                      char *out, int secs)
>  {
> @@ -2574,36 +2380,6 @@ static int do_test(const char *alg, u32 type, u32 mask, int m, u32 num_mb)
>                 test_ahash_speed("sm3", sec, generic_hash_speed_template);
>                 if (mode > 400 && mode < 500) break;
>                 fallthrough;
> -       case 450:
> -               test_mb_ahash_speed("sha1", sec, generic_hash_speed_template,
> -                                   num_mb);
> -               if (mode > 400 && mode < 500) break;
> -               fallthrough;
> -       case 451:
> -               test_mb_ahash_speed("sha256", sec, generic_hash_speed_template,
> -                                   num_mb);
> -               if (mode > 400 && mode < 500) break;
> -               fallthrough;
> -       case 452:
> -               test_mb_ahash_speed("sha512", sec, generic_hash_speed_template,
> -                                   num_mb);
> -               if (mode > 400 && mode < 500) break;
> -               fallthrough;
> -       case 453:
> -               test_mb_ahash_speed("sm3", sec, generic_hash_speed_template,
> -                                   num_mb);
> -               if (mode > 400 && mode < 500) break;
> -               fallthrough;
> -       case 454:
> -               test_mb_ahash_speed("streebog256", sec,
> -                                   generic_hash_speed_template, num_mb);
> -               if (mode > 400 && mode < 500) break;
> -               fallthrough;
> -       case 455:
> -               test_mb_ahash_speed("streebog512", sec,
> -                                   generic_hash_speed_template, num_mb);
> -               if (mode > 400 && mode < 500) break;
> -               fallthrough;
>         case 499:
>                 break;
>
> --
> 2.34.1
>



[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]

  Powered by Linux