DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] app/crypto-perf: fix build with Ofast
@ 2017-10-13  9:20 Pablo de Lara
  2017-10-13 11:03 ` Thomas Monjalon
  0 siblings, 1 reply; 2+ messages in thread
From: Pablo de Lara @ 2017-10-13  9:20 UTC (permalink / raw)
  To: declan.doherty; +Cc: dev, 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>
---
 app/test-crypto-perf/main.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

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;
-- 
2.9.4

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-dev] [PATCH] app/crypto-perf: fix build with Ofast
  2017-10-13  9:20 [dpdk-dev] [PATCH] app/crypto-perf: fix build with Ofast Pablo de Lara
@ 2017-10-13 11:03 ` Thomas Monjalon
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Monjalon @ 2017-10-13 11:03 UTC (permalink / raw)
  To: Pablo de Lara; +Cc: dev, declan.doherty

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-10-13 11:03 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-13  9:20 [dpdk-dev] [PATCH] app/crypto-perf: fix build with Ofast Pablo de Lara
2017-10-13 11:03 ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).