diff options
| author | Nathan Chancellor <natechancellor@gmail.com> | 2020-10-06 12:58:48 -0700 |
|---|---|---|
| committer | Herbert Xu <herbert@gondor.apana.org.au> | 2020-10-08 16:38:06 +1100 |
| commit | 557f4f05225669b596f7d6ee1577ef87b36f8696 (patch) | |
| tree | 96c21f6a523e867b27e9c498026f2470b8dacc1f /crypto/xor.c | |
| parent | a000b7868c2d35d9a2a6974d2ad26f7c32776fe1 (diff) | |
| download | linux-crypto-557f4f05225669b596f7d6ee1577ef87b36f8696.tar.gz linux-crypto-557f4f05225669b596f7d6ee1577ef87b36f8696.zip | |
crypto: xor - Remove unused variable count in do_xor_speed
Clang warns:
crypto/xor.c:101:4: warning: variable 'count' is uninitialized when used
here [-Wuninitialized]
count++;
^~~~~
crypto/xor.c:86:17: note: initialize the variable 'count' to silence
this warning
int i, j, count;
^
= 0
1 warning generated.
After the refactoring to use ktime that happened in this function, count
is only assigned, never read. Just remove the variable to get rid of the
warning.
Fixes: 839629ce56b9 ("crypto: xor - use ktime for template benchmarking")
Link: https://github.com/ClangBuiltLinux/linux/issues/1171
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto/xor.c')
| -rw-r--r-- | crypto/xor.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/crypto/xor.c b/crypto/xor.c index a0badbc0..eacbf4f9 100644 --- a/crypto/xor.c +++ b/crypto/xor.c @@ -83,7 +83,7 @@ static void __init do_xor_speed(struct xor_block_template *tmpl, void *b1, void *b2) { int speed; - int i, j, count; + int i, j; ktime_t min, start, diff; tmpl->next = template_list; @@ -98,8 +98,6 @@ do_xor_speed(struct xor_block_template *tmpl, void *b1, void *b2) mb(); /* prevent loop optimzation */ tmpl->do_2(BENCH_SIZE, b1, b2); mb(); - count++; - mb(); } diff = ktime_sub(ktime_get(), start); if (diff < min) |