Skip to content

Commit

Permalink
app/crypto-perf: fix latency CSV output
Browse files Browse the repository at this point in the history
[ upstream commit 2f04e82 ]

The csv output for the latency performance test had an extra header,
"Packet Size", which is a duplicate of "Buffer Size", and had no
corresponding value in the output. This is now removed.

Fixes: f6cefe2 ("app/crypto-perf: add range/list of sizes")

Signed-off-by: Ciara Power <ciara.power@intel.com>
Acked-by: Declan Doherty <declan.doherty@intel.com>
Acked-by: Adam Dybkowski <adamx.dybkowski@intel.com>
  • Loading branch information
ciarapow authored and bluca committed Feb 4, 2021
1 parent e12e46e commit c0bf6fb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion app/test-crypto-perf/cperf_test_latency.c
Original file line number Diff line number Diff line change
Expand Up @@ -310,7 +310,7 @@ cperf_latency_test_runner(void *arg)
if (ctx->options->csv) {
if (rte_atomic16_test_and_set(&display_once))
printf("\n# lcore, Buffer Size, Burst Size, Pakt Seq #, "
"Packet Size, cycles, time (us)");
"cycles, time (us)");

for (i = 0; i < ctx->options->total_ops; i++) {

Expand Down

0 comments on commit c0bf6fb

Please sign in to comment.