diff mbox series

[2/7] app/crypto-perf: fix excess crypto device error

Message ID 1522152039-32493-2-git-send-email-hemant.agrawal@nxp.com
State Superseded
Headers show
Series [1/7] test/crypto: add macro for dpaa sec device name | expand

Commit Message

Hemant Agrawal March 27, 2018, noon UTC 
If number of available devices are more than logical core,
no need to through an error. Just use the less number of
devices.

Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test application")
Cc: stable@dpdk.org

Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>

---
 app/test-crypto-perf/main.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

-- 
2.7.4

Comments

De Lara Guarch, Pablo March 30, 2018, 4:01 p.m. UTC | #1 
> -----Original Message-----

> From: Hemant Agrawal [mailto:hemant.agrawal@nxp.com]

> Sent: Tuesday, March 27, 2018 1:01 PM

> To: dev@dpdk.org

> Cc: De Lara Guarch, Pablo <pablo.de.lara.guarch@intel.com>;

> akhil.goyal@nxp.com

> Subject: [PATCH 2/7] app/crypto-perf: fix excess crypto device error

> 

> If number of available devices are more than logical core, no need to through an

> error. Just use the less number of devices.

> 

> Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test

> application")

> Cc: stable@dpdk.org

> 

> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>


Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
diff mbox series

Patch

diff --git a/app/test-crypto-perf/main.c b/app/test-crypto-perf/main.c
index 019d835..dd666f4 100644
--- a/app/test-crypto-perf/main.c
+++ b/app/test-crypto-perf/main.c
@@ -80,12 +80,9 @@  cperf_initialize_cryptodev(struct cperf_options *opts, uint8_t *enabled_cdevs,
 
 	nb_lcores = rte_lcore_count() - 1;
 
-	if (enabled_cdev_count > nb_lcores) {
-		printf("Number of capable crypto devices (%d) "
-				"has to be less or equal to number of slave "
-				"cores (%d)\n", enabled_cdev_count, nb_lcores);
-		return -EINVAL;
-	}
+	/* Use less number of devices, if more area available than cores.*/
+	if (enabled_cdev_count > nb_lcores)
+		enabled_cdev_count = nb_lcores;
 
 	/* Create a mempool shared by all the devices */
 	uint32_t max_sess_size = 0, sess_size;