Skip to content

Commit

Permalink
app/crypto-perf: check lcore job failure
Browse files Browse the repository at this point in the history
[ upstream commit 5908e7e ]

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: ce8af1a ("app/crypto-perf: wait for cores launched by app")

Signed-off-by: Tomasz Jozwiak <tjozwiakgm@gmail.com>
Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
  • Loading branch information
tjozwiak authored and kevintraynor committed Aug 26, 2019
1 parent 0f5ef64 commit 08e8a0e
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions app/test-crypto-perf/main.c
Expand Up @@ -627,9 +627,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 */
Expand All @@ -654,10 +657,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;
Expand Down

0 comments on commit 08e8a0e

Please sign in to comment.