From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fan Zhang Subject: [PATCH v2] app/test-crypto-perf: fix crypto operation resubmission Date: Tue, 18 Apr 2017 15:37:06 +0100 Message-ID: <1492526226-109903-1-git-send-email-roy.fan.zhang@intel.com> References: <1491836242-124659-1-git-send-email-roy.fan.zhang@intel.com> Cc: pablo.de.lara.guarch@intel.com, stable@dpdk.com To: dev@dpdk.org Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 0735756A1 for ; Tue, 18 Apr 2017 16:35:19 +0200 (CEST) In-Reply-To: <1491836242-124659-1-git-send-email-roy.fan.zhang@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" This patch fixes the crypto operation resubmission problem in crypto perferformance test. Originally, when needed crypto ops amount is smaller than the enqueued crypto ops in the last round, one or more processed crypto operations will be re-enqueued. Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test application") Signed-off-by: Fan Zhang --- v2: Fixed memmove src and size calculation app/test-crypto-perf/cperf_test_throughput.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/app/test-crypto-perf/cperf_test_throughput.c b/app/test-crypto-perf/cperf_test_throughput.c index 70ec4ff..0f635b9 100644 --- a/app/test-crypto-perf/cperf_test_throughput.c +++ b/app/test-crypto-perf/cperf_test_throughput.c @@ -348,6 +348,16 @@ cperf_throughput_test_runner(void *test_ctx) ops_needed, ctx->sess, ctx->options, ctx->test_vector); + /** + * When ops_needed is smaller than ops_enqd, the + * unused ops need to be moved to the front for + * next round use. + */ + if (unlikely(ops_enqd > ops_needed)) + memmove(&ops[ops_needed], + &ops[ops_enqd - ops_needed], + burst_size - ops_needed); + #ifdef CPERF_LINEARIZATION_ENABLE if (linearize) { /* PMD doesn't support scatter-gather and source buffer -- 2.7.4