From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id B76762A6C for ; Thu, 6 Oct 2016 19:45:48 +0200 (CEST) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga105.jf.intel.com with ESMTP; 06 Oct 2016 10:45:47 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,454,1473145200"; d="scan'208";a="1061389083" Received: from irvmail001.ir.intel.com ([163.33.26.43]) by orsmga002.jf.intel.com with ESMTP; 06 Oct 2016 10:45:46 -0700 Received: from linux.site (sisvmlab045.ir.intel.com [10.237.216.52]) by irvmail001.ir.intel.com (8.14.3/8.13.6/MailSET/Hub) with ESMTP id u96Hjjnu026586; Thu, 6 Oct 2016 18:45:45 +0100 Received: by linux.site (Postfix, from userid 11342333) id 03E24E3BC4; Thu, 6 Oct 2016 18:34:50 +0100 (IST) From: Fiona Trahe To: dev@dpdk.org Cc: pablo.de.lara.guarch@intel.com, fiona.trahe@intel.com, akhil.goyal@nxp.com Date: Thu, 6 Oct 2016 18:34:25 +0100 Message-Id: <1475775269-21454-1-git-send-email-fiona.trahe@intel.com> X-Mailer: git-send-email 1.7.0.7 In-Reply-To: <20160926163300.22990-1-akhil.goyal@nxp.com> References: <20160926163300.22990-1-akhil.goyal@nxp.com> Subject: [dpdk-dev] [PATCH v3 0/4] remove hard-coding of crypto num qps and cleanup X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 06 Oct 2016 17:45:49 -0000 ts_params->conf.nb_queue_pairs should not be hard coded with device specific number. It should be retrieved from the device info. Any test which changes it should restore it to orig value. Also related cleanup of test code setting number and size of queue-pairs on a device, e.g. * Removed irrelevant “for” loop – was hardcoded to only loop once. * Removed obsolete comment re inability to free and re-allocate queu memory and obsolete workaround for it which used to create maximum size queues. And added freeing of ring memory on queue-pair release in aesni_mb PMD, else releasing and setting up queue-pair of a different size fails. v3: separate out into 4 patches v2: Fix for broken QAT PMD unit tests exposed by v1 i.e. In test_device_configure_invalid_queue_pair_ids() after running tests for invalid values restore original nb_queue_pairs. Also cleanup of test code setting number and size of queue-pairs on a device Also fix for aesni_mb PMD not freeing ring memory on qp release Fiona Trahe (4): crypto/aesni_mb: free ring memory on qp release in PMD app/test: remove pointless for loop app/test: cleanup unnecessary ring size setup app/test: remove hard-coding of crypto num qps Akhil Goyal (1): app/test: remove hard-coding of crypto num qps app/test/test_cryptodev.c | 53 ++++++++++---------------- app/test/test_cryptodev_perf.c | 19 +-------- drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c | 10 ++++- 3 files changed, 31 insertions(+), 51 deletions(-) -- 2.5.0