Re: [dpdk-dev] [PATCH v6 0/6] add multiple cores feature to test-compress-perf

[Shally Verma <shallyv@marvell.com>]

> -----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