-----Original Message-----
From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
Sent: Monday, January 9, 2017 4:25 PM
To: Mrozowicz, SlawomirX <redacted>; Doherty,
Declan [off-list ref]
Cc: dev@dpdk.org; Azarewicz, PiotrX T <redacted>;
Kerlin, Marcin [off-list ref]; Kobylinski, MichalX
[off-list ref]
Subject: Re: [dpdk-dev] [PATCH v2 3/3] app/crypto-perf: introduce new
performance test application

2017-01-05 17:50, Slawomir Mrozowicz:

quoted

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

quoted

--- /dev/null
+++ b/app/crypto-perf/Makefile

[...]

quoted

+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