DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
@ 2019-01-16 13:21 Fan Zhang
  2019-01-16 16:48 ` De Lara Guarch, Pablo
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Fan Zhang @ 2019-01-16 13:21 UTC (permalink / raw)
  To: dev; +Cc: akhil.goyal

This patch fixes the inflexible compile issue AESNI-MB PMD.
Originally the compile of the PMD will fail if IPSec_MB is
not installed in default location.

Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent macros")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
---
 drivers/crypto/aesni_mb/Makefile | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/crypto/aesni_mb/Makefile b/drivers/crypto/aesni_mb/Makefile
index 7ef290dc5..1aae64a3c 100644
--- a/drivers/crypto/aesni_mb/Makefile
+++ b/drivers/crypto/aesni_mb/Makefile
@@ -23,7 +23,9 @@ LDLIBS += -lrte_eal -lrte_mbuf -lrte_mempool -lrte_ring
 LDLIBS += -lrte_cryptodev
 LDLIBS += -lrte_bus_vdev
 
-IMB_HDR = /usr/include/intel-ipsec-mb.h
+IMB_HDR = $(shell echo "\#include <intel-ipsec-mb.h>" | \
+	$(CC) -E $(EXTRA_CFLAGS) - | grep 'intel-ipsec-mb.h' | \
+	head -1 | cut -d'"' -f2)
 
 # Detect library version
 IMB_VERSION = $(shell grep -e "IMB_VERSION_STR" $(IMB_HDR) | cut -d'"' -f2)
-- 
2.13.6

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

* Re: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
  2019-01-16 13:21 [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile Fan Zhang
@ 2019-01-16 16:48 ` De Lara Guarch, Pablo
  2019-01-17 22:26 ` Thomas Monjalon
  2019-01-21 15:56 ` [dpdk-dev] [PATCH v2] " Fan Zhang
  2 siblings, 0 replies; 10+ messages in thread
From: De Lara Guarch, Pablo @ 2019-01-16 16:48 UTC (permalink / raw)
  To: Zhang, Roy Fan, dev; +Cc: akhil.goyal



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Fan Zhang
> Sent: Wednesday, January 16, 2019 1:21 PM
> To: dev@dpdk.org
> Cc: akhil.goyal@nxp.com
> Subject: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
> 
> This patch fixes the inflexible compile issue AESNI-MB PMD.
> Originally the compile of the PMD will fail if IPSec_MB is not installed in
> default location.
> 
> Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent
> macros")
> 
> Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

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

* Re: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
  2019-01-16 13:21 [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile Fan Zhang
  2019-01-16 16:48 ` De Lara Guarch, Pablo
@ 2019-01-17 22:26 ` Thomas Monjalon
  2019-01-18 10:57   ` Ananyev, Konstantin
  2019-01-21 15:56 ` [dpdk-dev] [PATCH v2] " Fan Zhang
  2 siblings, 1 reply; 10+ messages in thread
From: Thomas Monjalon @ 2019-01-17 22:26 UTC (permalink / raw)
  To: Fan Zhang; +Cc: dev, akhil.goyal, pablo.de.lara.guarch

16/01/2019 14:21, Fan Zhang:
> This patch fixes the inflexible compile issue AESNI-MB PMD.
> Originally the compile of the PMD will fail if IPSec_MB is
> not installed in default location.
> 
> Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent macros")
> 
> Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
> ---
> -IMB_HDR = /usr/include/intel-ipsec-mb.h
> +IMB_HDR = $(shell echo "\#include <intel-ipsec-mb.h>" | \

What is the backslash? Looks like a typo.

> +	$(CC) -E $(EXTRA_CFLAGS) - | grep 'intel-ipsec-mb.h' | \
> +	head -1 | cut -d'"' -f2)

I think the most standard invocation of head is "head -n1".
You can also use "sed 1q".

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

* Re: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
  2019-01-17 22:26 ` Thomas Monjalon
@ 2019-01-18 10:57   ` Ananyev, Konstantin
  2019-01-18 11:03     ` Thomas Monjalon
  0 siblings, 1 reply; 10+ messages in thread
From: Ananyev, Konstantin @ 2019-01-18 10:57 UTC (permalink / raw)
  To: Thomas Monjalon, Zhang, Roy Fan; +Cc: dev, akhil.goyal, De Lara Guarch, Pablo



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas Monjalon
> Sent: Thursday, January 17, 2019 10:27 PM
> To: Zhang, Roy Fan <roy.fan.zhang@intel.com>
> Cc: dev@dpdk.org; akhil.goyal@nxp.com; De Lara Guarch, Pablo <pablo.de.lara.guarch@intel.com>
> Subject: Re: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
> 
> 16/01/2019 14:21, Fan Zhang:
> > This patch fixes the inflexible compile issue AESNI-MB PMD.
> > Originally the compile of the PMD will fail if IPSec_MB is
> > not installed in default location.
> >
> > Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent macros")
> >
> > Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
> > ---
> > -IMB_HDR = /usr/include/intel-ipsec-mb.h
> > +IMB_HDR = $(shell echo "\#include <intel-ipsec-mb.h>" | \
> 
> What is the backslash? Looks like a typo.

Wouldn't make get confused without it?
Konstantin

> 
> > +	$(CC) -E $(EXTRA_CFLAGS) - | grep 'intel-ipsec-mb.h' | \
> > +	head -1 | cut -d'"' -f2)
> 
> I think the most standard invocation of head is "head -n1".
> You can also use "sed 1q".
> 

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

* Re: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
  2019-01-18 10:57   ` Ananyev, Konstantin
@ 2019-01-18 11:03     ` Thomas Monjalon
  2019-01-18 11:55       ` Ananyev, Konstantin
  0 siblings, 1 reply; 10+ messages in thread
From: Thomas Monjalon @ 2019-01-18 11:03 UTC (permalink / raw)
  To: Ananyev, Konstantin
  Cc: Zhang, Roy Fan, dev, akhil.goyal, De Lara Guarch, Pablo

18/01/2019 11:57, Ananyev, Konstantin:
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas Monjalon
> > 16/01/2019 14:21, Fan Zhang:
> > > This patch fixes the inflexible compile issue AESNI-MB PMD.
> > > Originally the compile of the PMD will fail if IPSec_MB is
> > > not installed in default location.
> > >
> > > Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent macros")
> > >
> > > Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > ---
> > > -IMB_HDR = /usr/include/intel-ipsec-mb.h
> > > +IMB_HDR = $(shell echo "\#include <intel-ipsec-mb.h>" | \
> > 
> > What is the backslash? Looks like a typo.
> 
> Wouldn't make get confused without it?

Confused about what?
I tested without backslash, it works.

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

* Re: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
  2019-01-18 11:03     ` Thomas Monjalon
@ 2019-01-18 11:55       ` Ananyev, Konstantin
  2019-01-21 10:08         ` Zhang, Roy Fan
  0 siblings, 1 reply; 10+ messages in thread
From: Ananyev, Konstantin @ 2019-01-18 11:55 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: Zhang, Roy Fan, dev, akhil.goyal, De Lara Guarch, Pablo



> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas@monjalon.net]
> Sent: Friday, January 18, 2019 11:03 AM
> To: Ananyev, Konstantin <konstantin.ananyev@intel.com>
> Cc: Zhang, Roy Fan <roy.fan.zhang@intel.com>; dev@dpdk.org; akhil.goyal@nxp.com; De Lara Guarch, Pablo
> <pablo.de.lara.guarch@intel.com>
> Subject: Re: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
> 
> 18/01/2019 11:57, Ananyev, Konstantin:
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas Monjalon
> > > 16/01/2019 14:21, Fan Zhang:
> > > > This patch fixes the inflexible compile issue AESNI-MB PMD.
> > > > Originally the compile of the PMD will fail if IPSec_MB is
> > > > not installed in default location.
> > > >
> > > > Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent macros")
> > > >
> > > > Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > ---
> > > > -IMB_HDR = /usr/include/intel-ipsec-mb.h
> > > > +IMB_HDR = $(shell echo "\#include <intel-ipsec-mb.h>" | \
> > >
> > > What is the backslash? Looks like a typo.
> >
> > Wouldn't make get confused without it?
> 
> Confused about what?

My guess was - about '#'.

> I tested without backslash, it works.

Then I was wrong with my guess.
Konstantin

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

* Re: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
  2019-01-18 11:55       ` Ananyev, Konstantin
@ 2019-01-21 10:08         ` Zhang, Roy Fan
  0 siblings, 0 replies; 10+ messages in thread
From: Zhang, Roy Fan @ 2019-01-21 10:08 UTC (permalink / raw)
  To: Ananyev, Konstantin, Thomas Monjalon
  Cc: dev, akhil.goyal, De Lara Guarch, Pablo

Hi, 

Thanks guys. I will send a v2 to remove the backslash.

Regards,
Fan

> -----Original Message-----
> From: Ananyev, Konstantin
> Sent: Friday, January 18, 2019 11:56 AM
> To: Thomas Monjalon <thomas@monjalon.net>
> Cc: Zhang, Roy Fan <roy.fan.zhang@intel.com>; dev@dpdk.org;
> akhil.goyal@nxp.com; De Lara Guarch, Pablo
> <pablo.de.lara.guarch@intel.com>
> Subject: RE: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
> 
> 
> 
> > -----Original Message-----
> > From: Thomas Monjalon [mailto:thomas@monjalon.net]
> > Sent: Friday, January 18, 2019 11:03 AM
> > To: Ananyev, Konstantin <konstantin.ananyev@intel.com>
> > Cc: Zhang, Roy Fan <roy.fan.zhang@intel.com>; dev@dpdk.org;
> > akhil.goyal@nxp.com; De Lara Guarch, Pablo
> > <pablo.de.lara.guarch@intel.com>
> > Subject: Re: [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile
> >
> > 18/01/2019 11:57, Ananyev, Konstantin:
> > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas
> Monjalon
> > > > 16/01/2019 14:21, Fan Zhang:
> > > > > This patch fixes the inflexible compile issue AESNI-MB PMD.
> > > > > Originally the compile of the PMD will fail if IPSec_MB is not
> > > > > installed in default location.
> > > > >
> > > > > Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture
> > > > > independent macros")
> > > > >
> > > > > Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
> > > > > ---
> > > > > -IMB_HDR = /usr/include/intel-ipsec-mb.h
> > > > > +IMB_HDR = $(shell echo "\#include <intel-ipsec-mb.h>" | \
> > > >
> > > > What is the backslash? Looks like a typo.
> > >
> > > Wouldn't make get confused without it?
> >
> > Confused about what?
> 
> My guess was - about '#'.
> 
> > I tested without backslash, it works.
> 
> Then I was wrong with my guess.
> Konstantin

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

* [dpdk-dev] [PATCH v2] crypto/aesni_mb: fix compile
  2019-01-16 13:21 [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile Fan Zhang
  2019-01-16 16:48 ` De Lara Guarch, Pablo
  2019-01-17 22:26 ` Thomas Monjalon
@ 2019-01-21 15:56 ` Fan Zhang
  2019-01-21 22:42   ` Ananyev, Konstantin
  2 siblings, 1 reply; 10+ messages in thread
From: Fan Zhang @ 2019-01-21 15:56 UTC (permalink / raw)
  To: dev; +Cc: akhil.goyal

This patch fixes the inflexible compile issue AESNI-MB PMD.
Originally the compile of the PMD will fail if IPSec_MB is
not installed in default location.

Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent macros")

Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
---
v2:
- uses standard head invocation

 drivers/crypto/aesni_mb/Makefile | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/crypto/aesni_mb/Makefile b/drivers/crypto/aesni_mb/Makefile
index 7ef290dc5..8d2024c9e 100644
--- a/drivers/crypto/aesni_mb/Makefile
+++ b/drivers/crypto/aesni_mb/Makefile
@@ -23,7 +23,9 @@ LDLIBS += -lrte_eal -lrte_mbuf -lrte_mempool -lrte_ring
 LDLIBS += -lrte_cryptodev
 LDLIBS += -lrte_bus_vdev
 
-IMB_HDR = /usr/include/intel-ipsec-mb.h
+IMB_HDR = $(shell echo '\#include <intel-ipsec-mb.h>' | \
+	$(CC) -E $(EXTRA_CFLAGS) - | grep 'intel-ipsec-mb.h' | \
+	head -n1 | cut -d'"' -f2)
 
 # Detect library version
 IMB_VERSION = $(shell grep -e "IMB_VERSION_STR" $(IMB_HDR) | cut -d'"' -f2)
-- 
2.13.6

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

* Re: [dpdk-dev] [PATCH v2] crypto/aesni_mb: fix compile
  2019-01-21 15:56 ` [dpdk-dev] [PATCH v2] " Fan Zhang
@ 2019-01-21 22:42   ` Ananyev, Konstantin
  2019-01-22 16:29     ` Thomas Monjalon
  0 siblings, 1 reply; 10+ messages in thread
From: Ananyev, Konstantin @ 2019-01-21 22:42 UTC (permalink / raw)
  To: Zhang, Roy Fan, dev; +Cc: akhil.goyal



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Fan Zhang
> Sent: Monday, January 21, 2019 3:56 PM
> To: dev@dpdk.org
> Cc: akhil.goyal@nxp.com
> Subject: [dpdk-dev] [PATCH v2] crypto/aesni_mb: fix compile
> 
> This patch fixes the inflexible compile issue AESNI-MB PMD.
> Originally the compile of the PMD will fail if IPSec_MB is
> not installed in default location.
> 
> Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent macros")
> 
> Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
> Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> ---
> v2:
> - uses standard head invocation
> 
>  drivers/crypto/aesni_mb/Makefile | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/crypto/aesni_mb/Makefile b/drivers/crypto/aesni_mb/Makefile
> index 7ef290dc5..8d2024c9e 100644
> --- a/drivers/crypto/aesni_mb/Makefile
> +++ b/drivers/crypto/aesni_mb/Makefile
> @@ -23,7 +23,9 @@ LDLIBS += -lrte_eal -lrte_mbuf -lrte_mempool -lrte_ring
>  LDLIBS += -lrte_cryptodev
>  LDLIBS += -lrte_bus_vdev
> 
> -IMB_HDR = /usr/include/intel-ipsec-mb.h
> +IMB_HDR = $(shell echo '\#include <intel-ipsec-mb.h>' | \
> +	$(CC) -E $(EXTRA_CFLAGS) - | grep 'intel-ipsec-mb.h' | \
> +	head -n1 | cut -d'"' -f2)
> 
>  # Detect library version
>  IMB_VERSION = $(shell grep -e "IMB_VERSION_STR" $(IMB_HDR) | cut -d'"' -f2)
> --

Tested-by: Konstantin Ananyev <konstantin.ananyev@intel.com>

> 2.13.6

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

* Re: [dpdk-dev] [PATCH v2] crypto/aesni_mb: fix compile
  2019-01-21 22:42   ` Ananyev, Konstantin
@ 2019-01-22 16:29     ` Thomas Monjalon
  0 siblings, 0 replies; 10+ messages in thread
From: Thomas Monjalon @ 2019-01-22 16:29 UTC (permalink / raw)
  To: Zhang, Roy Fan; +Cc: dev, Ananyev, Konstantin, akhil.goyal

> > This patch fixes the inflexible compile issue AESNI-MB PMD.
> > Originally the compile of the PMD will fail if IPSec_MB is
> > not installed in default location.
> > 
> > Fixes: c68d7aa354f6 ("crypto/aesni_mb: use architecture independent macros")
> > 
> > Signed-off-by: Fan Zhang <roy.fan.zhang@intel.com>
> > Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> 
> Tested-by: Konstantin Ananyev <konstantin.ananyev@intel.com>

Applied, thanks

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

end of thread, other threads:[~2019-01-22 16:29 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-16 13:21 [dpdk-dev] [PATCH] crypto/aesni_mb: fix compile Fan Zhang
2019-01-16 16:48 ` De Lara Guarch, Pablo
2019-01-17 22:26 ` Thomas Monjalon
2019-01-18 10:57   ` Ananyev, Konstantin
2019-01-18 11:03     ` Thomas Monjalon
2019-01-18 11:55       ` Ananyev, Konstantin
2019-01-21 10:08         ` Zhang, Roy Fan
2019-01-21 15:56 ` [dpdk-dev] [PATCH v2] " Fan Zhang
2019-01-21 22:42   ` Ananyev, Konstantin
2019-01-22 16:29     ` 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).