mbox series

[v8,0/7] add multiple cores feature to test-compress-perf

Message ID 20190708181619.8028-1-arturx.trybula@intel.com (mailing list archive)
Headers
Series add multiple cores feature to test-compress-perf |

Message

Artur Trybula July 8, 2019, 6:16 p.m. UTC
  This patchset adds multiple cores feature to compression perf tool.
All structures have been aligned and are consistent with crypto perf tool.
All test cases have constructor, runner and destructor and can use more
cores and compression devices at the same time.

v8 changes:
  - some minor fixes (e.g. 'magic numbers', spelling mistake, 
    enum naming)
    
v7 changes:
  - rebase as patch failed to apply due to release note clash
  - moved release note change to patch 4 as more appropriate

v6 changes:
  - rebase to the recent master (before applying)
    commit 82b9677cb23c

v5 changes:
  - changed ctx null ptr detection inside constructors

v4 changes:
  - moved release notes update to patch 1
  - removed pmd cyclecount template

v3 changes:
  - merged with commit 2f6527b70c15
    app/compress-perf: add prints for socket id
  - fixed wrong compression/decompression throughput calculation

v2 changes:
  - fixed max_nb_queue_pairs detection. Based on compression API
    if max_nb_queue_pairs = 0 that means there is no limit in
    maximum number of queue pairs
  - fixed qp setup on the last device


Artur Trybula (1):
  app/test-compress-perf: 'magic numbers' removed

Tomasz Jozwiak (6):
  app/test-compress-perf: add weak functions for multi-cores test
  app/test-compress-perf: add ptest command line option
  app/test-compress-perf: add verification test case
  app/test-compress-perf: add benchmark test case
  doc: update dpdk-test-compress-perf description
  app/test-compress-perf: add force process termination

 app/test-compress-perf/Makefile               |   1 +
 app/test-compress-perf/comp_perf.h            |  50 ++
 app/test-compress-perf/comp_perf_options.h    |  51 +-
 .../comp_perf_options_parse.c                 |  54 +-
 .../comp_perf_test_benchmark.c                | 153 +++--
 .../comp_perf_test_benchmark.h                |  25 +-
 .../comp_perf_test_common.c                   | 285 +++++++++
 .../comp_perf_test_common.h                   |  41 ++
 .../comp_perf_test_verify.c                   | 137 +++-
 .../comp_perf_test_verify.h                   |  24 +-
 app/test-compress-perf/main.c                 | 605 +++++++-----------
 app/test-compress-perf/meson.build            |   3 +-
 doc/guides/rel_notes/release_19_08.rst        |  27 +
 doc/guides/tools/comp_perf.rst                |  34 +-
 14 files changed, 1019 insertions(+), 471 deletions(-)
 create mode 100644 app/test-compress-perf/comp_perf.h
 create mode 100644 app/test-compress-perf/comp_perf_test_common.c
 create mode 100644 app/test-compress-perf/comp_perf_test_common.h

Series acked-by: Shally Verma <shallyv@marvell.com>
  

Comments

Akhil Goyal July 15, 2019, 1:12 p.m. UTC | #1
> 
> This patchset adds multiple cores feature to compression perf tool.
> All structures have been aligned and are consistent with crypto perf tool.
> All test cases have constructor, runner and destructor and can use more
> cores and compression devices at the same time.
> 
> v8 changes:
>   - some minor fixes (e.g. 'magic numbers', spelling mistake,
>     enum naming)
> 
> v7 changes:
>   - rebase as patch failed to apply due to release note clash
>   - moved release note change to patch 4 as more appropriate
> 
> v6 changes:
>   - rebase to the recent master (before applying)
>     commit 82b9677cb23c
> 
> v5 changes:
>   - changed ctx null ptr detection inside constructors
> 
> v4 changes:
>   - moved release notes update to patch 1
>   - removed pmd cyclecount template
> 
> v3 changes:
>   - merged with commit 2f6527b70c15
>     app/compress-perf: add prints for socket id
>   - fixed wrong compression/decompression throughput calculation
> 
> v2 changes:
>   - fixed max_nb_queue_pairs detection. Based on compression API
>     if max_nb_queue_pairs = 0 that means there is no limit in
>     maximum number of queue pairs
>   - fixed qp setup on the last device
> 
> 
> Artur Trybula (1):
>   app/test-compress-perf: 'magic numbers' removed
> 
> Tomasz Jozwiak (6):
>   app/test-compress-perf: add weak functions for multi-cores test
>   app/test-compress-perf: add ptest command line option
>   app/test-compress-perf: add verification test case
>   app/test-compress-perf: add benchmark test case
>   doc: update dpdk-test-compress-perf description
>   app/test-compress-perf: add force process termination
> 
>  app/test-compress-perf/Makefile               |   1 +
>  app/test-compress-perf/comp_perf.h            |  50 ++
>  app/test-compress-perf/comp_perf_options.h    |  51 +-
>  .../comp_perf_options_parse.c                 |  54 +-
>  .../comp_perf_test_benchmark.c                | 153 +++--
>  .../comp_perf_test_benchmark.h                |  25 +-
>  .../comp_perf_test_common.c                   | 285 +++++++++
>  .../comp_perf_test_common.h                   |  41 ++
>  .../comp_perf_test_verify.c                   | 137 +++-
>  .../comp_perf_test_verify.h                   |  24 +-
>  app/test-compress-perf/main.c                 | 605 +++++++-----------
>  app/test-compress-perf/meson.build            |   3 +-
>  doc/guides/rel_notes/release_19_08.rst        |  27 +
>  doc/guides/tools/comp_perf.rst                |  34 +-
>  14 files changed, 1019 insertions(+), 471 deletions(-)
>  create mode 100644 app/test-compress-perf/comp_perf.h
>  create mode 100644 app/test-compress-perf/comp_perf_test_common.c
>  create mode 100644 app/test-compress-perf/comp_perf_test_common.h
> 
> Series acked-by: Shally Verma <shallyv@marvell.com>
> --
> 2.17.1

Applied to dpdk-next-crypto