Re: [dpdk-dev] [PATCH v2 0/9] Crypto performance app improvements

["De Lara Guarch, Pablo" <pablo.de.lara.guarch@intel.com>]

> -----Original Message-----
> From: Gonzalez Monroy, Sergio
> Sent: Wednesday, March 29, 2017 4:25 PM
> To: De Lara Guarch, Pablo; Doherty, Declan
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2 0/9] Crypto performance app
> improvements
> 
> On 27/03/2017 13:29, De Lara Guarch, Pablo wrote:
> >
> >> -----Original Message-----
> >> From: De Lara Guarch, Pablo
> >> Sent: Monday, March 27, 2017 12:26 PM
> >> To: Doherty, Declan
> >> Cc: dev@dpdk.org; De Lara Guarch, Pablo
> >> Subject: [PATCH v2 0/9] Crypto performance app improvements
> >>
> >> This patchset makes the following changes in the crypto-perf app:
> >>
> >> - Adds the option to run performance tests on different
> >>    buffer and burst sizes, in a single run, instead of having to
> >>    run the application several times.
> >>
> >> - Cleans up the application, by removing unused functions and
> >>    separating the verify option in a new test type
> >>
> >> - Fixes several issues in the application (such as wrong test vectors)
> >>
> >>
> >> Pablo de Lara (8):
> >>    app/crypto-perf: remove cyclecount test type
> >>    app/crypto-perf: remove unused file
> >>    app/crypto-perf: fix AES CBC 128 test vectors
> >>    app/crypto-perf: do not append digest if not used
> >>    app/crypto-perf: display results in test runner
> >>    app/crypto-perf: add range/list of sizes
> >>    app/crypto-perf: add extra option checks
> >>    app/crypto-perf: reorg options structure
> >>
> >> Sergio Gonzalez Monroy (1):
> >>    app/crypto-perf: move verify as single test type
> >>
> >>   app/test-crypto-perf/Makefile                    |   1 +
> >>   app/test-crypto-perf/cperf_ops.c                 |  20 +-
> >>   app/test-crypto-perf/cperf_ops.h                 |   5 -
> >>   app/test-crypto-perf/cperf_options.h             |  35 +-
> >>   app/test-crypto-perf/cperf_options_parsing.c     | 308 ++++++++++--
> >>   app/test-crypto-perf/cperf_test_latency.c        | 508 ++++++++------------
> >>   app/test-crypto-perf/cperf_test_throughput.c     | 502 +++++++----------
> ---
> >>   app/test-crypto-perf/cperf_test_vector_parsing.c |  12 +-
> >>   app/test-crypto-perf/cperf_test_vectors.c        |   8 +-
> >>   app/test-crypto-perf/cperf_test_verify.c         | 579
> >> +++++++++++++++++++++++
> >>   app/test-crypto-perf/cperf_test_verify.h         |  58 +++
> >>   app/test-crypto-perf/cperf_verify_parser.c       | 314 ------------
> >>   app/test-crypto-perf/data/aes_cbc_128_sha.data   | 439 +++++++++---
> -----
> >>   app/test-crypto-perf/main.c                      |  71 ++-
> >>   doc/guides/tools/cryptoperf.rst                  |  27 +-
> >>   15 files changed, 1592 insertions(+), 1295 deletions(-)
> >>   create mode 100644 app/test-crypto-perf/cperf_test_verify.c
> >>   create mode 100644 app/test-crypto-perf/cperf_test_verify.h
> >>   delete mode 100644 app/test-crypto-perf/cperf_verify_parser.c
> >>
> >> --
> >> 2.7.4
> > Changes in v2:
> > - Split previous third patch into new four patches (6-9)
> > - Removed unused functions
> > - Fixed AES CBC 128 test vectors
> >
> Acked-by: Sergio Gonzalez Monroy <sergio.gonzalez.monroy@intel.com>

Applied to dpdk-next-crypto.

Pablo