DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] app: don't try to build test-crypto-perf if cryptodev disabled
       [not found] <CGME20170410081425eucas1p24667f00bc981db49558c1ba8dd1d9abe@eucas1p2.samsung.com>
@ 2017-04-10  8:14 ` Ilya Maximets
  2017-04-10  9:51   ` De Lara Guarch, Pablo
  0 siblings, 1 reply; 3+ messages in thread
From: Ilya Maximets @ 2017-04-10  8:14 UTC (permalink / raw)
  To: dev, Declan Doherty; +Cc: Heetae Ahn, Thomas Monjalon, Ilya Maximets

This fixes build in following configuration:

	CONFIG_RTE_LIBRTE_CRYPTODEV=n
	CONFIG_RTE_APP_CRYPTO_PERF=y

Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test application")

Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
---
 app/Makefile | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/app/Makefile b/app/Makefile
index 4b3a448..c3aeebf 100644
--- a/app/Makefile
+++ b/app/Makefile
@@ -32,8 +32,11 @@
 include $(RTE_SDK)/mk/rte.vars.mk
 
 DIRS-$(CONFIG_RTE_TEST_PMD) += test-pmd
-DIRS-$(CONFIG_RTE_APP_CRYPTO_PERF) += test-crypto-perf
 DIRS-$(CONFIG_RTE_EXEC_ENV_LINUXAPP) += proc_info
 DIRS-$(CONFIG_RTE_LIBRTE_PDUMP) += pdump
 
+ifeq ($(CONFIG_RTE_LIBRTE_CRYPTODEV),y)
+DIRS-$(CONFIG_RTE_APP_CRYPTO_PERF) += test-crypto-perf
+endif
+
 include $(RTE_SDK)/mk/rte.subdir.mk
-- 
2.7.4

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] app: don't try to build test-crypto-perf if cryptodev disabled
  2017-04-10  8:14 ` [dpdk-dev] [PATCH] app: don't try to build test-crypto-perf if cryptodev disabled Ilya Maximets
@ 2017-04-10  9:51   ` De Lara Guarch, Pablo
  2017-04-10 10:01     ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: De Lara Guarch, Pablo @ 2017-04-10  9:51 UTC (permalink / raw)
  To: Ilya Maximets, dev, Doherty, Declan; +Cc: Heetae Ahn, Thomas Monjalon



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ilya Maximets
> Sent: Monday, April 10, 2017 9:14 AM
> To: dev@dpdk.org; Doherty, Declan
> Cc: Heetae Ahn; Thomas Monjalon; Ilya Maximets
> Subject: [dpdk-dev] [PATCH] app: don't try to build test-crypto-perf if
> cryptodev disabled
> 
> This fixes build in following configuration:
> 
> 	CONFIG_RTE_LIBRTE_CRYPTODEV=n
> 	CONFIG_RTE_APP_CRYPTO_PERF=y
> 
> Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test
> application")
> 
> Signed-off-by: Ilya Maximets <i.maximets@samsung.com>

Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] app: don't try to build test-crypto-perf if cryptodev disabled
  2017-04-10  9:51   ` De Lara Guarch, Pablo
@ 2017-04-10 10:01     ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2017-04-10 10:01 UTC (permalink / raw)
  To: Ilya Maximets; +Cc: De Lara Guarch, Pablo, dev, Doherty, Declan, Heetae Ahn

2017-04-10 09:51, De Lara Guarch, Pablo:
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ilya Maximets
> > 
> > This fixes build in following configuration:
> > 
> > 	CONFIG_RTE_LIBRTE_CRYPTODEV=n
> > 	CONFIG_RTE_APP_CRYPTO_PERF=y
> > 
> > Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test
> > application")

Cc: stable@dpdk.org

> > Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
> 
> Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

Applied, thanks

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-04-10 10:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CGME20170410081425eucas1p24667f00bc981db49558c1ba8dd1d9abe@eucas1p2.samsung.com>
2017-04-10  8:14 ` [dpdk-dev] [PATCH] app: don't try to build test-crypto-perf if cryptodev disabled Ilya Maximets
2017-04-10  9:51   ` De Lara Guarch, Pablo
2017-04-10 10:01     ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).