Hi Shally,
On 7/5/2018 4:57 PM, Verma, Shally wrote:
-----Original Message-----
From: Pablo de Lara [mailto:pablo.de.lara.gua...@intel.com]
Sent: 05 July 2018 07:38
To: declan.dohe...@intel.com; akhil.go...@nxp.com; Verma, Shally
<shally.ve...@cavium.com>; ravi1.ku...@amd.com; Jacob,
Jerin <jerin.jacobkollanukka...@cavium.com>; roy.fan.zh...@intel.com;
fiona.tr...@intel.com; t...@semihalf.com;
jianjay.z...@huawei.com
Cc: dev@dpdk.org; Pablo de Lara <pablo.de.lara.gua...@intel.com>
Subject: [PATCH v5 04/16] test/crypto: limit number of sessions
External Email
Instead of using the maximum number of sessions
allowed by the PMDs (which will change to unlimited most
of the PMDs), limit the number to a small sufficient amount.
Signed-off-by: Pablo de Lara <pablo.de.lara.gua...@intel.com>
Acked-by: Akhil Goyal <akhil.go...@nxp.com>
---
test/test/test_cryptodev.c | 27 +++++++++++++++++++++------
1 file changed, 21 insertions(+), 6 deletions(-)
diff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c
index 389f79677..5c906cfae 100644
--- a/test/test/test_cryptodev.c
+++ b/test/test/test_cryptodev.c
@@ -39,6 +39,7 @@
#include "test_cryptodev_hmac_test_vectors.h"
#define VDEV_ARGS_SIZE 100
+#define MAX_NB_SESSIONS 4
static int gbl_driver_id;
@@ -435,9 +436,16 @@ testsuite_setup(void)
* Create mempool with maximum number of sessions * 2,
* to include the session headers
*/
+ if (info.sym.max_nb_sessions < MAX_NB_SESSIONS) {
+ RTE_LOG(ERR, USER1, "Device does not support "
+ "at least %u sessions\n",
+ MAX_NB_SESSIONS);
+ return TEST_FAILED;
+ }
+
If info.sym.max_nb_sessions = 0, won't it fail from here?
It is added in 8/16 of this series. Till this patch 0 is not a valid value.
Thanks
Shally
