test/compress: improve debug logs
authorFiona Trahe <fiona.trahe@intel.com>
Wed, 31 Oct 2018 00:42:06 +0000 (00:42 +0000)
committerAkhil Goyal <akhil.goyal@nxp.com>
Fri, 2 Nov 2018 11:26:06 +0000 (12:26 +0100)
Make clear which engine is compressing and which is decompressing
in debug output. Also add newline and print ratio = 0 if test fails.

Signed-off-by: Fiona Trahe <fiona.trahe@intel.com>
Acked-by: Lee Daly <lee.daly@intel.com>
Acked-by: Tomasz Jozwiak <tomaszx.jozwiak@intel.com>
test/test/test_compressdev.c

index 8645388..5d5e519 100644 (file)
@@ -10,6 +10,7 @@
 #include <rte_mempool.h>
 #include <rte_mbuf.h>
 #include <rte_compressdev.h>
+#include <rte_string_fns.h>
 
 #include "test_compressdev_test_buffer.h"
 #include "test.h"
@@ -856,13 +857,20 @@ test_deflate_comp_decomp(const char * const test_bufs[],
                                &compress_xforms[xform_idx]->compress;
                enum rte_comp_huffman huffman_type =
                        compress_xform->deflate.huffman;
-               RTE_LOG(DEBUG, USER1, "Buffer %u compressed from %u to %u bytes "
-                       "(level = %d, huffman = %s)\n",
-                       buf_idx[priv_data->orig_idx],
+               char engine[22];
+               if (zlib_dir == ZLIB_COMPRESS || zlib_dir == ZLIB_ALL)
+                       strlcpy(engine, "zlib (direct, no pmd)", 22);
+               else
+                       strlcpy(engine, "pmd", 22);
+
+               RTE_LOG(DEBUG, USER1, "Buffer %u compressed by %s from %u to"
+                       " %u bytes (level = %d, huffman = %s)\n",
+                       buf_idx[priv_data->orig_idx], engine,
                        ops_processed[i]->consumed, ops_processed[i]->produced,
                        compress_xform->level,
                        huffman_type_strings[huffman_type]);
-               RTE_LOG(DEBUG, USER1, "Compression ratio = %.2f",
+               RTE_LOG(DEBUG, USER1, "Compression ratio = %.2f\n",
+                       ops_processed[i]->consumed == 0 ? 0 :
                        (float)ops_processed[i]->produced /
                        ops_processed[i]->consumed * 100);
                ops[i] = NULL;
@@ -1058,8 +1066,14 @@ test_deflate_comp_decomp(const char * const test_bufs[],
 
        for (i = 0; i < num_bufs; i++) {
                priv_data = (struct priv_op_data *)(ops_processed[i] + 1);
-               RTE_LOG(DEBUG, USER1, "Buffer %u decompressed from %u to %u bytes\n",
-                       buf_idx[priv_data->orig_idx],
+               char engine[22];
+               if (zlib_dir == ZLIB_DECOMPRESS || zlib_dir == ZLIB_ALL)
+                       strlcpy(engine, "zlib (direct, no pmd)", 22);
+               else
+                       strlcpy(engine, "pmd", 22);
+               RTE_LOG(DEBUG, USER1,
+                       "Buffer %u decompressed by %s from %u to %u bytes\n",
+                       buf_idx[priv_data->orig_idx], engine,
                        ops_processed[i]->consumed, ops_processed[i]->produced);
                ops[i] = NULL;
        }