From 5908e7e837a62fe62c48094d8c2e947242f8fac3 Mon Sep 17 00:00:00 2001 From: Tomasz Jozwiak Date: Fri, 14 Jun 2019 22:56:45 +0200 Subject: [PATCH] app/crypto-perf: check lcore job failure Currently, there's no return status check from lcore's jobs. In case of fail - crypto-perf tool returns success anyway. This patch adds such a detection and returns proper status at the end. Fixes: ce8af1a4398d ("app/crypto-perf: wait for cores launched by app") Cc: stable@dpdk.org Signed-off-by: Tomasz Jozwiak Acked-by: Akhil Goyal --- app/test-crypto-perf/main.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/app/test-crypto-perf/main.c b/app/test-crypto-perf/main.c index db0290751c..ce92a96872 100644 --- a/app/test-crypto-perf/main.c +++ b/app/test-crypto-perf/main.c @@ -665,9 +665,12 @@ main(int argc, char **argv) if (i == total_nb_qps) break; - rte_eal_wait_lcore(lcore_id); + ret |= rte_eal_wait_lcore(lcore_id); i++; } + + if (ret != EXIT_SUCCESS) + goto err; } else { /* Get next size from range or list */ @@ -692,10 +695,13 @@ main(int argc, char **argv) if (i == total_nb_qps) break; - rte_eal_wait_lcore(lcore_id); + ret |= rte_eal_wait_lcore(lcore_id); i++; } + if (ret != EXIT_SUCCESS) + goto err; + /* Get next size from range or list */ if (opts.inc_buffer_size != 0) opts.test_buffer_size += opts.inc_buffer_size; -- 2.20.1