[dpdk-dev] [PATCH v2 3/3] app/crypto-perf: introduce new performance test application

Mrozowicz, SlawomirX slawomirx.mrozowicz at intel.com
Tue Jan 10 09:52:55 CET 2017


>-----Original Message-----
>From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
>Sent: Monday, January 9, 2017 4:25 PM
>To: Mrozowicz, SlawomirX <slawomirx.mrozowicz at intel.com>; Doherty,
>Declan <declan.doherty at intel.com>
>Cc: dev at dpdk.org; Azarewicz, PiotrX T <piotrx.t.azarewicz at intel.com>;
>Kerlin, Marcin <marcin.kerlin at intel.com>; Kobylinski, MichalX
><michalx.kobylinski at intel.com>
>Subject: Re: [dpdk-dev] [PATCH v2 3/3] app/crypto-perf: introduce new
>performance test application
>
>2017-01-05 17:50, Slawomir Mrozowicz:
>> --- a/app/Makefile
>> +++ b/app/Makefile
>> @@ -38,5 +38,6 @@ DIRS-$(CONFIG_RTE_TEST_PMD) += test-pmd
>>  DIRS-$(CONFIG_RTE_LIBRTE_CMDLINE) += cmdline_test
>>  DIRS-$(CONFIG_RTE_EXEC_ENV_LINUXAPP) += proc_info
>>  DIRS-$(CONFIG_RTE_LIBRTE_PDUMP) += pdump
>> +DIRS-$(CONFIG_RTE_APP_CRYPTO_PERF) += crypto-perf
>
>Could we rename the directory to test-crypto or test-crypto-perf for
>consistency?
>
>> --- /dev/null
>> +++ b/app/crypto-perf/Makefile
>[...]
>> +APP = dpdk-crypto-perf
>
>I think we should have "test" in the name of the app.
>dpdk-test-crypto?
>dpdk-test-crypto-perf?

I am going to change 
the directory to: /app/test-crypto-perf
the app name to: dpdk-test-crypto-perf

Sławomir






More information about the dev mailing list