* [dpdk-dev v1] crypto/qat: add in libcrypto version check
@ 2022-07-26 9:59 Kai Ji
2022-07-26 10:05 ` [dpdk-dev v2] " Kai Ji
2022-07-26 10:14 ` [EXT] [dpdk-dev v1] crypto/qat: add in libcrypto version check Akhil Goyal
0 siblings, 2 replies; 6+ messages in thread
From: Kai Ji @ 2022-07-26 9:59 UTC (permalink / raw)
To: dev; +Cc: gakhil, Kai Ji
This patch add in libcrypto version check before enable libipsec-mb for
QAT. The inter-ipsec-mb lib for partial hash and AES cacluation should
only be enabled when both OpensSSL 3.0 and IPSec_MB 1.2.0 are installed
on the system.
Signed-off-by: Kai Ji <kai.ji@intel.com>
---
drivers/common/qat/meson.build | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/common/qat/meson.build b/drivers/common/qat/meson.build
index 245c0fbe61..8247bdd017 100644
--- a/drivers/common/qat/meson.build
+++ b/drivers/common/qat/meson.build
@@ -37,7 +37,9 @@ endif
IMB_required_ver = '1.2.0'
libipsecmb = cc.find_library('IPSec_MB', required: false)
-if libipsecmb.found()
+libcrypto_3 = dependency('libcrypto', required: false,
+ method: 'pkg-config', version : '>=3.0.0')
+if libipsecmb.found() and libcrypto_3.found()
# version comes with quotes, so we split based on " and take the middle
imb_ver = cc.get_define('IMB_VERSION_STR',
prefix : '#include<intel-ipsec-mb.h>').split('"')[1]
--
2.17.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-dev v2] crypto/qat: add in libcrypto version check
2022-07-26 9:59 [dpdk-dev v1] crypto/qat: add in libcrypto version check Kai Ji
@ 2022-07-26 10:05 ` Kai Ji
2022-07-26 11:56 ` [dpdk-dev v2] Crypto/qat: fix build with OpenSSL 1.1 Kai Ji
2022-07-26 10:14 ` [EXT] [dpdk-dev v1] crypto/qat: add in libcrypto version check Akhil Goyal
1 sibling, 1 reply; 6+ messages in thread
From: Kai Ji @ 2022-07-26 10:05 UTC (permalink / raw)
To: dev; +Cc: gakhil, Kai Ji
This patch add in libcrypto version check before enable libipsec-mb for
QAT. The inter-ipsec-mb lib for partial hash and AES calculation should
only be enabled when both OpensSSL 3.0 and IPSec_MB 1.2.0 are installed
on the system.
Signed-off-by: Kai Ji <kai.ji@intel.com>
---
drivers/common/qat/meson.build | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/common/qat/meson.build b/drivers/common/qat/meson.build
index 245c0fbe61..8247bdd017 100644
--- a/drivers/common/qat/meson.build
+++ b/drivers/common/qat/meson.build
@@ -37,7 +37,9 @@ endif
IMB_required_ver = '1.2.0'
libipsecmb = cc.find_library('IPSec_MB', required: false)
-if libipsecmb.found()
+libcrypto_3 = dependency('libcrypto', required: false,
+ method: 'pkg-config', version : '>=3.0.0')
+if libipsecmb.found() and libcrypto_3.found()
# version comes with quotes, so we split based on " and take the middle
imb_ver = cc.get_define('IMB_VERSION_STR',
prefix : '#include<intel-ipsec-mb.h>').split('"')[1]
--
2.17.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-dev v2] Crypto/qat: fix build with OpenSSL 1.1
2022-07-26 10:05 ` [dpdk-dev v2] " Kai Ji
@ 2022-07-26 11:56 ` Kai Ji
2022-08-04 13:09 ` [EXT] " Akhil Goyal
0 siblings, 1 reply; 6+ messages in thread
From: Kai Ji @ 2022-07-26 11:56 UTC (permalink / raw)
To: dev; +Cc: gakhil, Kai Ji
This patch add in libcrypto version check before enable libipsec-mb for
QAT. The inter-ipsec-mb lib for partial hash and AES calculation should
only be enabled when both OpensSSL 3.0 and IPSec_MB 1.2.0 are installed
on the system.
Signed-off-by: Kai Ji <kai.ji@intel.com>
---
drivers/common/qat/meson.build | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/common/qat/meson.build b/drivers/common/qat/meson.build
index 245c0fbe61..8247bdd017 100644
--- a/drivers/common/qat/meson.build
+++ b/drivers/common/qat/meson.build
@@ -37,7 +37,9 @@ endif
IMB_required_ver = '1.2.0'
libipsecmb = cc.find_library('IPSec_MB', required: false)
-if libipsecmb.found()
+libcrypto_3 = dependency('libcrypto', required: false,
+ method: 'pkg-config', version : '>=3.0.0')
+if libipsecmb.found() and libcrypto_3.found()
# version comes with quotes, so we split based on " and take the middle
imb_ver = cc.get_define('IMB_VERSION_STR',
prefix : '#include<intel-ipsec-mb.h>').split('"')[1]
--
2.17.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [EXT] [dpdk-dev v2] Crypto/qat: fix build with OpenSSL 1.1
2022-07-26 11:56 ` [dpdk-dev v2] Crypto/qat: fix build with OpenSSL 1.1 Kai Ji
@ 2022-08-04 13:09 ` Akhil Goyal
0 siblings, 0 replies; 6+ messages in thread
From: Akhil Goyal @ 2022-08-04 13:09 UTC (permalink / raw)
To: Kai Ji, dev; +Cc: stable
> Subject: [EXT] [dpdk-dev v2] Crypto/qat: fix build with OpenSSL 1.1
common/qat: fix build with OpenSSL 1.1
> This patch add in libcrypto version check before enable libipsec-mb for
> QAT. The inter-ipsec-mb lib for partial hash and AES calculation should
> only be enabled when both OpensSSL 3.0 and IPSec_MB 1.2.0 are installed
> on the system.
>
> Signed-off-by: Kai Ji <kai.ji@intel.com>
Fixes: 3227bc7138f5 ("crypto/qat: use intel-ipsec-mb for partial hash and AES")
Cc: stable@dpdk.org
Acked-by: Akhil Goyal <gakhil@marvell.com>
Applied to dpdk-next-crypto
Thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [EXT] [dpdk-dev v1] crypto/qat: add in libcrypto version check
2022-07-26 9:59 [dpdk-dev v1] crypto/qat: add in libcrypto version check Kai Ji
2022-07-26 10:05 ` [dpdk-dev v2] " Kai Ji
@ 2022-07-26 10:14 ` Akhil Goyal
2022-07-26 11:57 ` Ji, Kai
1 sibling, 1 reply; 6+ messages in thread
From: Akhil Goyal @ 2022-07-26 10:14 UTC (permalink / raw)
To: Kai Ji, dev; +Cc: Ashwin Sekhar Thalakalath Kottilveetil, Anoob Joseph
Hi Kai,
This is a fix for compilation when openssl 1.1 is used to build QAT.
Can you update the title to
Crypto/qat: fix build with OpenSSL 1.1
> This patch add in libcrypto version check before enable libipsec-mb for
> QAT. The inter-ipsec-mb lib for partial hash and AES cacluation should
> only be enabled when both OpensSSL 3.0 and IPSec_MB 1.2.0 are installed
> on the system.
Spell check on description.
>
> Signed-off-by: Kai Ji <kai.ji@intel.com>
> ---
> drivers/common/qat/meson.build | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/common/qat/meson.build
> b/drivers/common/qat/meson.build
> index 245c0fbe61..8247bdd017 100644
> --- a/drivers/common/qat/meson.build
> +++ b/drivers/common/qat/meson.build
> @@ -37,7 +37,9 @@ endif
>
> IMB_required_ver = '1.2.0'
> libipsecmb = cc.find_library('IPSec_MB', required: false)
> -if libipsecmb.found()
> +libcrypto_3 = dependency('libcrypto', required: false,
> + method: 'pkg-config', version : '>=3.0.0')
> +if libipsecmb.found() and libcrypto_3.found()
> # version comes with quotes, so we split based on " and take the middle
> imb_ver = cc.get_define('IMB_VERSION_STR',
> prefix : '#include<intel-ipsec-mb.h>').split('"')[1]
> --
> 2.17.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [EXT] [dpdk-dev v1] crypto/qat: add in libcrypto version check
2022-07-26 10:14 ` [EXT] [dpdk-dev v1] crypto/qat: add in libcrypto version check Akhil Goyal
@ 2022-07-26 11:57 ` Ji, Kai
0 siblings, 0 replies; 6+ messages in thread
From: Ji, Kai @ 2022-07-26 11:57 UTC (permalink / raw)
To: Akhil Goyal, dev; +Cc: Ashwin Sekhar Thalakalath Kottilveetil, Anoob Joseph
Thanks akhil,
V2 here: http://patchwork.dpdk.org/project/dpdk/patch/20220726115608.84140-1-kai.ji@intel.com/
Regards
Kai
> -----Original Message-----
> From: Akhil Goyal <gakhil@marvell.com>
> Sent: Tuesday, July 26, 2022 11:14 AM
> To: Ji, Kai <kai.ji@intel.com>; dev@dpdk.org
> Cc: Ashwin Sekhar Thalakalath Kottilveetil <asekhar@marvell.com>; Anoob
> Joseph <anoobj@marvell.com>
> Subject: RE: [EXT] [dpdk-dev v1] crypto/qat: add in libcrypto version check
>
> Hi Kai,
>
> This is a fix for compilation when openssl 1.1 is used to build QAT.
> Can you update the title to
> Crypto/qat: fix build with OpenSSL 1.1
>
> > This patch add in libcrypto version check before enable libipsec-mb
> > for QAT. The inter-ipsec-mb lib for partial hash and AES cacluation
> > should only be enabled when both OpensSSL 3.0 and IPSec_MB 1.2.0 are
> > installed on the system.
>
> Spell check on description.
>
> >
> > Signed-off-by: Kai Ji <kai.ji@intel.com>
> > ---
> > drivers/common/qat/meson.build | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/common/qat/meson.build
> > b/drivers/common/qat/meson.build index 245c0fbe61..8247bdd017 100644
> > --- a/drivers/common/qat/meson.build
> > +++ b/drivers/common/qat/meson.build
> > @@ -37,7 +37,9 @@ endif
> >
> > IMB_required_ver = '1.2.0'
> > libipsecmb = cc.find_library('IPSec_MB', required: false) -if
> > libipsecmb.found()
> > +libcrypto_3 = dependency('libcrypto', required: false,
> > + method: 'pkg-config', version : '>=3.0.0') if libipsecmb.found()
> > +and libcrypto_3.found()
> > # version comes with quotes, so we split based on " and take the middle
> > imb_ver = cc.get_define('IMB_VERSION_STR',
> > prefix : '#include<intel-ipsec-mb.h>').split('"')[1]
> > --
> > 2.17.1
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-08-04 13:09 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-26 9:59 [dpdk-dev v1] crypto/qat: add in libcrypto version check Kai Ji
2022-07-26 10:05 ` [dpdk-dev v2] " Kai Ji
2022-07-26 11:56 ` [dpdk-dev v2] Crypto/qat: fix build with OpenSSL 1.1 Kai Ji
2022-08-04 13:09 ` [EXT] " Akhil Goyal
2022-07-26 10:14 ` [EXT] [dpdk-dev v1] crypto/qat: add in libcrypto version check Akhil Goyal
2022-07-26 11:57 ` Ji, Kai
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).