[dpdk-dev] [EXT] [PATCH v3 2/7] app/test-compress-perf: add ptest command line option
Tomasz Jozwiak
tjozwiakgm at gmail.com
Thu Jun 27 23:27:50 CEST 2019
Hi Shally,
>
>> -----Original Message-----
>> From: Tomasz Jozwiak <tjozwiakgm at gmail.com>
>> Sent: Wednesday, June 26, 2019 11:05 PM
>> To: Shally Verma <shallyv at marvell.com>; dev at dpdk.org;
>> fiona.trahe at intel.com; arturx.trybula at 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 <tjozwiakgm at gmail.com>
>>>> Sent: Wednesday, June 26, 2019 10:00 PM
>>>> To: dev at dpdk.org; fiona.trahe at intel.com; tjozwiakgm at gmail.com;
>> Shally
>>>> Verma <shallyv at marvell.com>; arturx.trybula at intel.com
>>>> Subject: [EXT] [PATCH v3 2/7] app/test-compress-perf: add ptest
>>>> command line option
>>>>
>>>> External Email
>>>>
>>>> ---------------------------------------------------------------------
>>>> -
>>>> From: Tomasz Jozwiak <tomaszx.jozwiak at intel.com>
>>>>
>>>> This patch adds --ptest option to make possible a choose of test case
>>>> from command line.
>>>>
>>>> Signed-off-by: Tomasz Jozwiak <tomaszx.jozwiak at 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?
[Tomek] True , I'll remove this in V4.
Thx, Tomek
More information about the dev
mailing list