From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 0DC3EA2E1B for ; Thu, 5 Sep 2019 10:11:18 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id AB0531BE98; Thu, 5 Sep 2019 10:11:17 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id 1757E2BE9; Thu, 5 Sep 2019 10:11:13 +0200 (CEST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 5925728; Thu, 5 Sep 2019 01:11:12 -0700 (PDT) Received: from net-arm-c2400-02.shanghai.arm.com (net-arm-c2400-02.shanghai.arm.com [10.169.40.42]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 33A7F3F67D; Thu, 5 Sep 2019 01:11:10 -0700 (PDT) From: Ruifeng Wang To: pablo.de.lara.guarch@intel.com, declan.doherty@intel.com Cc: dev@dpdk.org, roy.fan.zhang@intel.com, gavin.hu@arm.com, honnappa.nagarahalli@arm.com, nd@arm.com, Ruifeng Wang , stable@dpdk.org Date: Thu, 5 Sep 2019 16:10:44 +0800 Message-Id: <20190905081044.1154-1-ruifeng.wang@arm.com> X-Mailer: git-send-email 2.17.1 Subject: [dpdk-stable] [PATCH] test/crypto: fix unit test failure for virtual cryptodev X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" armv8 cryptodev virtual PMD autotest failed with output: CRYPTODEV: [CRYPTODEV_NAME_ARMV8_CRYPTO_PMD] armv8_crypto_pmd_qp_create_processed_ops_ring() line 210: Unable to reuse existing ring armv8_crypto_pmd_0_qp_0 for processed ops CRYPTODEV: rte_cryptodev_stop() line 942: Device with dev_id=0 already stopped + TestCase [ 0] : test_AES_chain_armv8_all failed This is due to the ring size used in ut_setup is bigger than that used in testsuite_setup. Fix this issue by enlarge ring size used in testsuite_setup accordingly. Fixes: 24054e3640a2 ("test/crypto: use separate session mempools") Cc: stable@dpdk.org Signed-off-by: Ruifeng Wang Reviewed-by: Gavin Hu --- app/test/test_cryptodev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c index 4197febb0..9a226bd15 100644 --- a/app/test/test_cryptodev.c +++ b/app/test/test_cryptodev.c @@ -473,7 +473,7 @@ testsuite_setup(void) "Failed to configure cryptodev %u with %u qps", dev_id, ts_params->conf.nb_queue_pairs); - ts_params->qp_conf.nb_descriptors = DEFAULT_NUM_OPS_INFLIGHT; + ts_params->qp_conf.nb_descriptors = MAX_NUM_OPS_INFLIGHT; ts_params->qp_conf.mp_session = ts_params->session_mpool; ts_params->qp_conf.mp_session_private = ts_params->session_priv_mpool; -- 2.17.1