Re: [PATCH v2 3/3] app/crypto-perf: introduce new performance test application
From: Thomas Monjalon <hidden>
Date: 2017-01-09 15:24:53
2017-01-05 17:50, Slawomir Mrozowicz:
quoted hunk ↗ jump to hunk
--- 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 hunk ↗ jump to hunk
--- /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?