From 61bec3623ceba3fda30a1dd74adc67cdb6dc9466 Mon Sep 17 00:00:00 2001 From: Pablo de Lara Date: Mon, 7 Aug 2017 04:16:54 +0100 Subject: [PATCH] app/crypto-perf: fix operation free In crypto latency test, when some crypto operations cannot be enqueued, they are returned to its mempool. However, instead of freeing the operations in the array passed to the enqueue function, the app was freeing the operations in the array passed to the dequeue function, leading to incorrect results. Fixes: 15b55dd75120 ("app/crypto-perf: return crypto ops to mempool in bulk") Signed-off-by: Pablo de Lara --- app/test-crypto-perf/cperf_test_latency.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/test-crypto-perf/cperf_test_latency.c b/app/test-crypto-perf/cperf_test_latency.c index 7e610d9318..58b21abd9a 100644 --- a/app/test-crypto-perf/cperf_test_latency.c +++ b/app/test-crypto-perf/cperf_test_latency.c @@ -433,7 +433,7 @@ cperf_latency_test_runner(void *arg) /* Free memory for not enqueued operations */ if (ops_enqd != burst_size) rte_mempool_put_bulk(ctx->crypto_op_pool, - (void **)&ops_processed[ops_enqd], + (void **)&ops[ops_enqd], burst_size - ops_enqd); for (i = 0; i < ops_enqd; i++) { -- 2.20.1