> -----Original Message-----
> From: Tomasz Jozwiak <tjozwia...@gmail.com>
> Sent: Wednesday, June 26, 2019 11:05 PM
> To: Shally Verma <shal...@marvell.com>; dev@dpdk.org;
> fiona.tr...@intel.com; arturx.tryb...@intel.com
> Subject: Re: [EXT] [PATCH v3 2/7] app/test-compress-perf: add ptest
> command line option
>
> Hi Shally,
>
>
> >> -----Original Message-----
> >> From: Tomasz Jozwiak <tjozwia...@gmail.com>
> >> Sent: Wednesday, June 26, 2019 10:00 PM
> >> To: dev@dpdk.org; fiona.tr...@intel.com; tjozwia...@gmail.com;
> Shally
> >> Verma <shal...@marvell.com>; arturx.tryb...@intel.com
> >> Subject: [EXT] [PATCH v3 2/7] app/test-compress-perf: add ptest
> >> command line option
> >>
> >> External Email
> >>
> >> ---------------------------------------------------------------------
> >> -
> >> From: Tomasz Jozwiak <tomaszx.jozw...@intel.com>
> >>
> >> This patch adds --ptest option to make possible a choose of test case
> >> from command line.
> >>
> >> Signed-off-by: Tomasz Jozwiak <tomaszx.jozw...@intel.com>
> >> ---
> >> app/test-compress-perf/comp_perf_options_parse.c | 36
> >> ++++++++++++++++++++++++
> >> 1 file changed, 36 insertions(+)
> >>
> >> diff --git a/app/test-compress-perf/comp_perf_options_parse.c
> >> b/app/test- compress-perf/comp_perf_options_parse.c
> >> index bc4b98a..07672b2 100644
> >> --- a/app/test-compress-perf/comp_perf_options_parse.c
> >> +++ b/app/test-compress-perf/comp_perf_options_parse.c
> >> @@ -15,6 +15,7 @@
> >>
> >> #include "comp_perf_options.h"
> >>
> >> +#define CPERF_PTEST_TYPE ("ptest")
> >> #define CPERF_DRIVER_NAME ("driver-name")
> >> #define CPERF_TEST_FILE ("input-file")
> >> #define CPERF_SEG_SIZE ("seg-sz")
> >> @@ -37,6 +38,7 @@ static void
> >> usage(char *progname)
> >> {
> >> printf("%s [EAL options] --\n"
> >> + " --ptest benchmark / verify :"
> >> " --driver-name NAME: compress driver to use\n"
> >> " --input-file NAME: file to compress and decompress\n"
> >> " --extended-input-sz N: extend file data up to this size
> >> (default: no extension)\n"
> >> @@ -76,6 +78,37 @@ get_str_key_id_mapping(struct name_id_map
> *map,
> >> unsigned int map_len, }
> >>
> >> static int
> >> +parse_cperf_test_type(struct comp_test_data *test_data, const char
> >> +*arg) {
> >> + struct name_id_map cperftest_namemap[] = {
> >> + {
> >> +
> >> cperf_test_type_strs[CPERF_TEST_TYPE_BENCHMARK],
> >> + CPERF_TEST_TYPE_BENCHMARK
> >> + },
> >> + {
> >> + cperf_test_type_strs[CPERF_TEST_TYPE_VERIFY],
> >> + CPERF_TEST_TYPE_VERIFY
> >> + },
> >> + {
> >> + cperf_test_type_strs[CPERF_TEST_TYPE_PMDCC],
> >> + CPERF_TEST_TYPE_PMDCC
> > What is PMDCC here? What kind of test type?
>
> [Tomek] This's for 'pmd_cyclecount_test'. Actually compression_perf tool
>
> doesn't support that test, but we added a sort of 'template' for that, cause
> there's a plan to add
>
> this test in the future.
Isn’t it better we add this .. when we actually add feature?
>
>
> Thx
>
> --
>
> Tomek
>
>
