diff mbox

[dpdk-dev] app/crypto-perf: fix build with Ofast

Message ID 20171013092014.23194-1-pablo.de.lara.guarch@intel.com (mailing list archive)
State Accepted, archived
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

De Lara Guarch, Pablo Oct. 13, 2017, 9:20 a.m. UTC 
  app/test-crypto-perf/main.c:596:6: error: ‘total_nb_qps’ may be
used uninitialized in this function [-Werror=maybe-uninitialized]
   if (i == total_nb_qps)
      ^

Fixes: c4f916e33226 ("app/crypto-perf: support multiple queue pairs")

Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
---
 app/test-crypto-perf/main.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Thomas Monjalon Oct. 13, 2017, 11:03 a.m. UTC | #1 
13/10/2017 11:20, Pablo de Lara:
> app/test-crypto-perf/main.c:596:6: error: ‘total_nb_qps’ may be
> used uninitialized in this function [-Werror=maybe-uninitialized]
>    if (i == total_nb_qps)
>       ^
> 
> Fixes: c4f916e33226 ("app/crypto-perf: support multiple queue pairs")
> 
> Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

Applied, thanks
diff mbox

Patch

diff --git a/app/test-crypto-perf/main.c b/app/test-crypto-perf/main.c
index aaa5830..29373f5 100644
--- a/app/test-crypto-perf/main.c
+++ b/app/test-crypto-perf/main.c
@@ -424,6 +424,7 @@  main(int argc, char **argv)
 	struct rte_mempool *session_pool_socket[RTE_MAX_NUMA_NODES] = { 0 };
 
 	int nb_cryptodevs = 0;
+	uint16_t total_nb_qps = 0;
 	uint8_t cdev_id, i;
 	uint8_t enabled_cdevs[RTE_CRYPTO_MAX_DEVS] = { 0 };
 
@@ -509,7 +510,7 @@  main(int argc, char **argv)
 	if (!opts.silent)
 		show_test_vector(t_vec);
 
-	uint16_t total_nb_qps = nb_cryptodevs * opts.nb_qps;
+	total_nb_qps = nb_cryptodevs * opts.nb_qps;
 
 	i = 0;
 	uint8_t qp_id = 0, cdev_index = 0;