[dpdk-dev,v2] app/test-crypto-perf: fix index

Message ID 20180123142255.68449-1-roy.fan.zhang@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Pablo de Lara Guarch
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation fail Compilation issues

Commit Message

Fan Zhang Jan. 23, 2018, 2:22 p.m. UTC
  Fixes: 27c2e7471961 ("app/crypto-perf: support IMIX")

A bug caused index out-of-range error. This simple patch is to fix
this.

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
---
v2:
Removed unnecessary cdev_id assignment line.

 app/test-crypto-perf/main.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
  

Comments

De Lara Guarch, Pablo Jan. 23, 2018, 3:28 p.m. UTC | #1
> -----Original Message-----
> From: Zhang, Roy Fan
> Sent: Tuesday, January 23, 2018 2:23 PM
> To: dev@dpdk.org
> Cc: De Lara Guarch, Pablo <pablo.de.lara.guarch@intel.com>
> Subject: [PATCH v2] app/test-crypto-perf: fix index
> 
> Fixes: 27c2e7471961 ("app/crypto-perf: support IMIX")
> 
> A bug caused index out-of-range error. This simple patch is to fix this.
> 
> Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>

Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
  
De Lara Guarch, Pablo Jan. 23, 2018, 3:39 p.m. UTC | #2
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of De Lara Guarch,
> Pablo
> Sent: Tuesday, January 23, 2018 3:29 PM
> To: Zhang, Roy Fan <roy.fan.zhang@intel.com>; dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2] app/test-crypto-perf: fix index
> 
> 
> 
> > -----Original Message-----
> > From: Zhang, Roy Fan
> > Sent: Tuesday, January 23, 2018 2:23 PM
> > To: dev@dpdk.org
> > Cc: De Lara Guarch, Pablo <pablo.de.lara.guarch@intel.com>
> > Subject: [PATCH v2] app/test-crypto-perf: fix index
> >
> > Fixes: 27c2e7471961 ("app/crypto-perf: support IMIX")
> >
> > A bug caused index out-of-range error. This simple patch is to fix this.
> >
> > Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
> 
> Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

Applied to dpdk-next-crypto.
Thanks,

Pablo
  

Patch

diff --git a/app/test-crypto-perf/main.c b/app/test-crypto-perf/main.c
index 83a9d7b73..dd16fbe24 100644
--- a/app/test-crypto-perf/main.c
+++ b/app/test-crypto-perf/main.c
@@ -582,10 +582,8 @@  main(int argc, char **argv)
 				if (i == nb_cryptodevs)
 					break;
 
-				cdev_id = enabled_cdevs[i];
-
 				rte_eal_remote_launch(cperf_testmap[opts.test].runner,
-					ctx[cdev_id], lcore_id);
+					ctx[i], lcore_id);
 				i++;
 			}
 			i = 0;