From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fiona Trahe Subject: [PATCH 2/2] test/compress: use bulk free operations api Date: Mon, 19 Nov 2018 22:10:11 +0000 Message-ID: <1542665411-23489-3-git-send-email-fiona.trahe@intel.com> References: <1542665411-23489-1-git-send-email-fiona.trahe@intel.com> Cc: akhil.goyal@nxp.com, tomaszx.jozwiak@intel.com, shally.verma@caviumnetworks.com, ashish.gupta@caviumnetworks.com, lee.daly@intel.com, fiona.trahe@intel.com To: dev@dpdk.org Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 3BF245323 for ; Mon, 19 Nov 2018 23:10:35 +0100 (CET) In-Reply-To: <1542665411-23489-1-git-send-email-fiona.trahe@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Use the new rte_comp_op_bulk_free API. Add trace to catch any mempool elements not freed at test end. Signed-off-by: Fiona Trahe --- test/test/test_compressdev.c | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/test/test/test_compressdev.c b/test/test/test_compressdev.c index 5d5e519..3ea726d 100644 --- a/test/test/test_compressdev.c +++ b/test/test/test_compressdev.c @@ -69,6 +69,13 @@ testsuite_teardown(void) { struct comp_testsuite_params *ts_params = &testsuite_params; + if (rte_mempool_in_use_count(ts_params->large_mbuf_pool)) + RTE_LOG(ERR, USER1, "Large mbuf pool still has unfreed bufs\n"); + if (rte_mempool_in_use_count(ts_params->small_mbuf_pool)) + RTE_LOG(ERR, USER1, "Small mbuf pool still has unfreed bufs\n"); + if (rte_mempool_in_use_count(ts_params->op_pool)) + RTE_LOG(ERR, USER1, "op pool still has unfreed ops\n"); + rte_mempool_free(ts_params->large_mbuf_pool); rte_mempool_free(ts_params->small_mbuf_pool); rte_mempool_free(ts_params->op_pool); @@ -731,6 +738,7 @@ test_deflate_comp_decomp(const char * const test_bufs[], goto exit; } + for (i = 0; i < num_bufs; i++) { ops[i]->m_src = uncomp_bufs[i]; ops[i]->m_dst = comp_bufs[i]; @@ -961,12 +969,9 @@ test_deflate_comp_decomp(const char * const test_bufs[], /* * Free the previous compress operations, - * as it is not needed anymore + * as they are not needed anymore */ - for (i = 0; i < num_bufs; i++) { - rte_comp_op_free(ops_processed[i]); - ops_processed[i] = NULL; - } + rte_comp_op_bulk_free(ops_processed, num_bufs); /* Decompress data (either with Zlib API or compressdev API */ if (zlib_dir == ZLIB_DECOMPRESS || zlib_dir == ZLIB_ALL) { -- 2.7.4