mbox series

[v6,0/6] add multiple cores feature to test-compress-perf

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

Message

Artur Trybula July 3, 2019, 3:24 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.

v2 changes:

  - fixed checkpatch warning
  - 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

v3 changes:

  - merged with commit 2f6527b70c15
    app/compress-perf: add prints for socket id

  - fixed wrong compression/decompression throughput calculation

v4 changes:

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

v5 changes:

  - changed ctx null ptr detection inside constructors

v6 changes:

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

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    |  45 +-
 .../comp_perf_options_parse.c                 |  54 +-
 .../comp_perf_test_benchmark.c                | 153 +++--
 .../comp_perf_test_benchmark.h                |  25 +-
 .../comp_perf_test_common.c                   | 284 ++++++++
 .../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        |   4 +
 doc/guides/tools/comp_perf.rst                |  34 +-
 14 files changed, 993 insertions(+), 467 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
  

Comments

Shally Verma July 5, 2019, 9:50 a.m. UTC | #1
> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Artur Trybula
> Sent: Wednesday, July 3, 2019 8:54 PM
> To: dev@dpdk.org; fiona.trahe@intel.com; akhil.goyal@nxp.com;
> adamx.dybkowski@intel.com; arturx.trybula@intel.com;
> tjozwiakgm@gmail.com
> Subject: [dpdk-dev] [PATCH v6 0/6] add multiple cores feature to test-
> compress-perf
> 
> 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.
> 
> v2 changes:
> 
>   - fixed checkpatch warning
>   - 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
> 
> v3 changes:
> 
>   - merged with commit 2f6527b70c15
>     app/compress-perf: add prints for socket id
> 
>   - fixed wrong compression/decompression throughput calculation
> 
> v4 changes:
> 
>   - moved release notes update to patch 1
>   - removed pmd cyclecount template
> 
> v5 changes:
> 
>   - changed ctx null ptr detection inside constructors
> 
> v6 changes:
> 
>   - rebase to the recent master (before applying)
>     commit 82b9677cb23c
> 
> 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    |  45 +-
>  .../comp_perf_options_parse.c                 |  54 +-
>  .../comp_perf_test_benchmark.c                | 153 +++--
>  .../comp_perf_test_benchmark.h                |  25 +-
>  .../comp_perf_test_common.c                   | 284 ++++++++
>  .../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        |   4 +
>  doc/guides/tools/comp_perf.rst                |  34 +-
>  14 files changed, 993 insertions(+), 467 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
> 
> --
It will be nice if we could have version history in descending order. Other than that

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

> 2.17.1