* [dpdk-stable] [PATCH] app/test-crypto-perf: fix crypto operation resubmission
@ 2017-04-10 14:57 Fan Zhang
2017-04-17 20:28 ` De Lara Guarch, Pablo
0 siblings, 1 reply; 2+ messages in thread
From: Fan Zhang @ 2017-04-10 14:57 UTC (permalink / raw)
To: dev; +Cc: pablo.de.lara.guarch, stable
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 <roy.fan.zhang@intel.com>
---
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
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [dpdk-stable] [PATCH] app/test-crypto-perf: fix crypto operation resubmission
2017-04-10 14:57 [dpdk-stable] [PATCH] app/test-crypto-perf: fix crypto operation resubmission Fan Zhang
@ 2017-04-17 20:28 ` De Lara Guarch, Pablo
0 siblings, 0 replies; 2+ messages in thread
From: De Lara Guarch, Pablo @ 2017-04-17 20:28 UTC (permalink / raw)
To: Zhang, Roy Fan, dev; +Cc: stable
> -----Original Message-----
> From: Zhang, Roy Fan
> Sent: Monday, April 10, 2017 3:57 PM
> To: dev@dpdk.org
> Cc: De Lara Guarch, Pablo; stable@dpdk.org
> Subject: [PATCH] app/test-crypto-perf: fix crypto operation resubmission
>
> 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 <roy.fan.zhang@intel.com>
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-04-17 20:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-10 14:57 [dpdk-stable] [PATCH] app/test-crypto-perf: fix crypto operation resubmission Fan Zhang
2017-04-17 20:28 ` De Lara Guarch, Pablo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).