mbox series

[v2,0/3] add initial version of compress-perf

Message ID 1541151842-8746-1-git-send-email-tomaszx.jozwiak@intel.com (mailing list archive)
Headers
Series add initial version of compress-perf |

Message

Tomasz Jozwiak Nov. 2, 2018, 9:43 a.m. UTC 
  This patchset adds initial version of compression performance
test.

v2 changes:

  -  Added release note
  -  Added new cleanup flow into main function
  -  Blocked dynamic compression test because it hasn't been
     tested enough
  -  Changed `--max-num-sgl-segs' default value to 16
  -  Updated documentation

Opens:  comment from Shally Verma re separating validation from
        benchmarking will be investigated in a later release.
        Support for dynamic Huffman encoding will be added
        in a later release.

Tomasz Jozwiak (3):
  app/compress-perf: add parser
  app/compress-perf: add performance measurement
  doc/guides/tools: add doc files

 MAINTAINERS                                      |   5 +
 app/Makefile                                     |   4 +
 app/meson.build                                  |   1 +
 app/test-compress-perf/Makefile                  |  16 +
 app/test-compress-perf/comp_perf_options.h       |  59 ++
 app/test-compress-perf/comp_perf_options_parse.c | 592 +++++++++++++++
 app/test-compress-perf/main.c                    | 928 +++++++++++++++++++++++
 app/test-compress-perf/meson.build               |   7 +
 config/common_base                               |   5 +
 doc/guides/rel_notes/release_18_11.rst           |   6 +
 doc/guides/tools/comp_perf.rst                   |  75 ++
 11 files changed, 1698 insertions(+)
 create mode 100644 app/test-compress-perf/Makefile
 create mode 100644 app/test-compress-perf/comp_perf_options.h
 create mode 100644 app/test-compress-perf/comp_perf_options_parse.c
 create mode 100644 app/test-compress-perf/main.c
 create mode 100644 app/test-compress-perf/meson.build
 create mode 100644 doc/guides/tools/comp_perf.rst

Comments

Bruce Richardson Nov. 2, 2018, 11:04 a.m. UTC | #1 
On Fri, Nov 02, 2018 at 10:43:59AM +0100, Tomasz Jozwiak wrote:
> This patchset adds initial version of compression performance
> test.
> 
> v2 changes:
> 
>   -  Added release note
>   -  Added new cleanup flow into main function
>   -  Blocked dynamic compression test because it hasn't been
>      tested enough
>   -  Changed `--max-num-sgl-segs' default value to 16
>   -  Updated documentation
> 
> Opens:  comment from Shally Verma re separating validation from
>         benchmarking will be investigated in a later release.
>         Support for dynamic Huffman encoding will be added
>         in a later release.
> 
> Tomasz Jozwiak (3):
>   app/compress-perf: add parser
>   app/compress-perf: add performance measurement
>   doc/guides/tools: add doc files
> 
If this is a performance test app, should it not go in "test" rather than
"app" folder?
Tomasz Jozwiak Nov. 2, 2018, 11:12 a.m. UTC | #2 
Hi Bruce,

It's similar to test-crypto-perf and others like test-eventdev, test-pmd, etc. which are in 'app' folder, as well.