From 6cef3265af80ca365421b87d4a447a5e637fb6c4 Mon Sep 17 00:00:00 2001 From: Fiona Trahe Date: Mon, 19 Nov 2018 22:10:11 +0000 Subject: [PATCH] test/compress: use bulk free operation API 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 Acked-by: Shally Verma Acked-by: Lee Daly --- 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 dab3c12ecb..83f115d200 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]; @@ -959,12 +967,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.20.1