DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers
@ 2019-05-01 15:53 Bruce Richardson
  2019-05-01 15:53 ` Bruce Richardson
                   ` (4 more replies)
  0 siblings, 5 replies; 10+ messages in thread
From: Bruce Richardson @ 2019-05-01 15:53 UTC (permalink / raw)
  To: pablo.de.lara.guarch; +Cc: dev, Bruce Richardson

Some fixes for the dependency checking of the kasumi and zuc PMDs, and an
addition of meson support for snow3g which seems to have been overlooked
previously.

[Apologies for the late submission, but hopefully these can make it into 19.05]

Bruce Richardson (3):
  crypto/snow3g: add to meson build
  crypto/kasumi: fix dependency check
  crypto/zuc: fix dependency check

 drivers/crypto/kasumi/meson.build |  8 ++++----
 drivers/crypto/meson.build        |  2 +-
 drivers/crypto/snow3g/meson.build | 13 +++++++++++++
 drivers/crypto/zuc/meson.build    |  8 ++++----
 4 files changed, 22 insertions(+), 9 deletions(-)
 create mode 100644 drivers/crypto/snow3g/meson.build

-- 
2.21.0

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

* [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers
  2019-05-01 15:53 [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers Bruce Richardson
@ 2019-05-01 15:53 ` Bruce Richardson
  2019-05-01 15:53 ` [dpdk-dev] [PATCH 1/3] crypto/snow3g: add to meson build Bruce Richardson
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 10+ messages in thread
From: Bruce Richardson @ 2019-05-01 15:53 UTC (permalink / raw)
  To: pablo.de.lara.guarch; +Cc: dev, Bruce Richardson

Some fixes for the dependency checking of the kasumi and zuc PMDs, and an
addition of meson support for snow3g which seems to have been overlooked
previously.

[Apologies for the late submission, but hopefully these can make it into 19.05]

Bruce Richardson (3):
  crypto/snow3g: add to meson build
  crypto/kasumi: fix dependency check
  crypto/zuc: fix dependency check

 drivers/crypto/kasumi/meson.build |  8 ++++----
 drivers/crypto/meson.build        |  2 +-
 drivers/crypto/snow3g/meson.build | 13 +++++++++++++
 drivers/crypto/zuc/meson.build    |  8 ++++----
 4 files changed, 22 insertions(+), 9 deletions(-)
 create mode 100644 drivers/crypto/snow3g/meson.build

-- 
2.21.0


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

* [dpdk-dev] [PATCH 1/3] crypto/snow3g: add to meson build
  2019-05-01 15:53 [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers Bruce Richardson
  2019-05-01 15:53 ` Bruce Richardson
@ 2019-05-01 15:53 ` Bruce Richardson
  2019-05-01 15:53   ` Bruce Richardson
  2019-05-01 15:53 ` [dpdk-dev] [PATCH 2/3] crypto/kasumi: fix dependency check Bruce Richardson
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 10+ messages in thread
From: Bruce Richardson @ 2019-05-01 15:53 UTC (permalink / raw)
  To: pablo.de.lara.guarch; +Cc: dev, Bruce Richardson

Snow3G driver is missing support for building with meson. Add a new
meson.build file so it can be included in the builds.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/crypto/meson.build        |  2 +-
 drivers/crypto/snow3g/meson.build | 13 +++++++++++++
 2 files changed, 14 insertions(+), 1 deletion(-)
 create mode 100644 drivers/crypto/snow3g/meson.build

diff --git a/drivers/crypto/meson.build b/drivers/crypto/meson.build
index bf1bd9286..83e78860e 100644
--- a/drivers/crypto/meson.build
+++ b/drivers/crypto/meson.build
@@ -3,7 +3,7 @@
 
 drivers = ['aesni_gcm', 'aesni_mb', 'caam_jr', 'ccp', 'dpaa_sec', 'dpaa2_sec',
 	'kasumi', 'mvsam', 'null', 'octeontx', 'openssl', 'qat', 'scheduler',
-	'virtio', 'zuc']
+	'snow3g', 'virtio', 'zuc']
 
 std_deps = ['cryptodev'] # cryptodev pulls in all other needed deps
 config_flag_fmt = 'RTE_LIBRTE_@0@_PMD'
diff --git a/drivers/crypto/snow3g/meson.build b/drivers/crypto/snow3g/meson.build
new file mode 100644
index 000000000..c566a5f67
--- /dev/null
+++ b/drivers/crypto/snow3g/meson.build
@@ -0,0 +1,13 @@
+# SPDX-License-Identifier: BSD-3-Clause
+# Copyright(c) 2019 Intel Corporation
+
+lib = cc.find_library('sso_snow3g', required: false)
+if not lib.found() or not cc.has_header('sso_snow3g.h')
+	build = false
+	subdir_done()
+endif
+
+allow_experimental_apis = true
+ext_deps += lib
+sources = files('rte_snow3g_pmd.c', 'rte_snow3g_pmd_ops.c')
+deps += ['bus_vdev', 'cryptodev']
-- 
2.21.0

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

* [dpdk-dev] [PATCH 1/3] crypto/snow3g: add to meson build
  2019-05-01 15:53 ` [dpdk-dev] [PATCH 1/3] crypto/snow3g: add to meson build Bruce Richardson
@ 2019-05-01 15:53   ` Bruce Richardson
  0 siblings, 0 replies; 10+ messages in thread
From: Bruce Richardson @ 2019-05-01 15:53 UTC (permalink / raw)
  To: pablo.de.lara.guarch; +Cc: dev, Bruce Richardson

Snow3G driver is missing support for building with meson. Add a new
meson.build file so it can be included in the builds.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/crypto/meson.build        |  2 +-
 drivers/crypto/snow3g/meson.build | 13 +++++++++++++
 2 files changed, 14 insertions(+), 1 deletion(-)
 create mode 100644 drivers/crypto/snow3g/meson.build

diff --git a/drivers/crypto/meson.build b/drivers/crypto/meson.build
index bf1bd9286..83e78860e 100644
--- a/drivers/crypto/meson.build
+++ b/drivers/crypto/meson.build
@@ -3,7 +3,7 @@
 
 drivers = ['aesni_gcm', 'aesni_mb', 'caam_jr', 'ccp', 'dpaa_sec', 'dpaa2_sec',
 	'kasumi', 'mvsam', 'null', 'octeontx', 'openssl', 'qat', 'scheduler',
-	'virtio', 'zuc']
+	'snow3g', 'virtio', 'zuc']
 
 std_deps = ['cryptodev'] # cryptodev pulls in all other needed deps
 config_flag_fmt = 'RTE_LIBRTE_@0@_PMD'
diff --git a/drivers/crypto/snow3g/meson.build b/drivers/crypto/snow3g/meson.build
new file mode 100644
index 000000000..c566a5f67
--- /dev/null
+++ b/drivers/crypto/snow3g/meson.build
@@ -0,0 +1,13 @@
+# SPDX-License-Identifier: BSD-3-Clause
+# Copyright(c) 2019 Intel Corporation
+
+lib = cc.find_library('sso_snow3g', required: false)
+if not lib.found() or not cc.has_header('sso_snow3g.h')
+	build = false
+	subdir_done()
+endif
+
+allow_experimental_apis = true
+ext_deps += lib
+sources = files('rte_snow3g_pmd.c', 'rte_snow3g_pmd_ops.c')
+deps += ['bus_vdev', 'cryptodev']
-- 
2.21.0


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

* [dpdk-dev] [PATCH 2/3] crypto/kasumi: fix dependency check
  2019-05-01 15:53 [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers Bruce Richardson
  2019-05-01 15:53 ` Bruce Richardson
  2019-05-01 15:53 ` [dpdk-dev] [PATCH 1/3] crypto/snow3g: add to meson build Bruce Richardson
@ 2019-05-01 15:53 ` Bruce Richardson
  2019-05-01 15:53   ` Bruce Richardson
  2019-05-01 15:53 ` [dpdk-dev] [PATCH 3/3] crypto/zuc: " Bruce Richardson
  2019-05-02  9:08 ` [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers Thomas Monjalon
  4 siblings, 1 reply; 10+ messages in thread
From: Bruce Richardson @ 2019-05-01 15:53 UTC (permalink / raw)
  To: pablo.de.lara.guarch; +Cc: dev, Bruce Richardson

The check for the kasumi library dependency did not check for the include
file needed for compilation. It's also recommended when looking for a
library to use the name without the lib prefix.

Fixes: 2ec2d46c6255 ("crypto/kasumi: enable meson build")

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/crypto/kasumi/meson.build | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/crypto/kasumi/meson.build b/drivers/crypto/kasumi/meson.build
index 80f13cd24..0fa301740 100644
--- a/drivers/crypto/kasumi/meson.build
+++ b/drivers/crypto/kasumi/meson.build
@@ -1,13 +1,13 @@
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2018 Intel Corporation
 
-lib = cc.find_library('libsso_kasumi', required: false)
-if not lib.found()
+lib = cc.find_library('sso_kasumi', required: false)
+if not lib.found() or not cc.has_header('sso_kasumi.h')
 	build = false
-else
-	ext_deps += lib
+	subdir_done()
 endif
 
 allow_experimental_apis = true
+ext_deps += lib
 sources = files('rte_kasumi_pmd.c', 'rte_kasumi_pmd_ops.c')
 deps += ['bus_vdev']
-- 
2.21.0

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

* [dpdk-dev] [PATCH 2/3] crypto/kasumi: fix dependency check
  2019-05-01 15:53 ` [dpdk-dev] [PATCH 2/3] crypto/kasumi: fix dependency check Bruce Richardson
@ 2019-05-01 15:53   ` Bruce Richardson
  0 siblings, 0 replies; 10+ messages in thread
From: Bruce Richardson @ 2019-05-01 15:53 UTC (permalink / raw)
  To: pablo.de.lara.guarch; +Cc: dev, Bruce Richardson

The check for the kasumi library dependency did not check for the include
file needed for compilation. It's also recommended when looking for a
library to use the name without the lib prefix.

Fixes: 2ec2d46c6255 ("crypto/kasumi: enable meson build")

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/crypto/kasumi/meson.build | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/crypto/kasumi/meson.build b/drivers/crypto/kasumi/meson.build
index 80f13cd24..0fa301740 100644
--- a/drivers/crypto/kasumi/meson.build
+++ b/drivers/crypto/kasumi/meson.build
@@ -1,13 +1,13 @@
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2018 Intel Corporation
 
-lib = cc.find_library('libsso_kasumi', required: false)
-if not lib.found()
+lib = cc.find_library('sso_kasumi', required: false)
+if not lib.found() or not cc.has_header('sso_kasumi.h')
 	build = false
-else
-	ext_deps += lib
+	subdir_done()
 endif
 
 allow_experimental_apis = true
+ext_deps += lib
 sources = files('rte_kasumi_pmd.c', 'rte_kasumi_pmd_ops.c')
 deps += ['bus_vdev']
-- 
2.21.0


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

* [dpdk-dev] [PATCH 3/3] crypto/zuc: fix dependency check
  2019-05-01 15:53 [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers Bruce Richardson
                   ` (2 preceding siblings ...)
  2019-05-01 15:53 ` [dpdk-dev] [PATCH 2/3] crypto/kasumi: fix dependency check Bruce Richardson
@ 2019-05-01 15:53 ` Bruce Richardson
  2019-05-01 15:53   ` Bruce Richardson
  2019-05-02  9:08 ` [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers Thomas Monjalon
  4 siblings, 1 reply; 10+ messages in thread
From: Bruce Richardson @ 2019-05-01 15:53 UTC (permalink / raw)
  To: pablo.de.lara.guarch; +Cc: dev, Bruce Richardson

The check for the zuc library dependency did not check for the include
file needed for compilation. It's also recommended when looking for a
library to use the name without the lib prefix.

Fixes: df1740a8db5f ("crypto/zuc: enable meson build")

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/crypto/zuc/meson.build | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/crypto/zuc/meson.build b/drivers/crypto/zuc/meson.build
index 63f2a5298..fc2900244 100644
--- a/drivers/crypto/zuc/meson.build
+++ b/drivers/crypto/zuc/meson.build
@@ -1,13 +1,13 @@
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2018 Intel Corporation
 
-lib = cc.find_library('libsso_zuc', required: false)
-if not lib.found()
+lib = cc.find_library('sso_zuc', required: false)
+if not lib.found() or not cc.has_header('sso_zuc.h')
 	build = false
-else
-	ext_deps += lib
+	subdir_done()
 endif
 
 allow_experimental_apis = true
+ext_deps += lib
 sources = files('rte_zuc_pmd.c', 'rte_zuc_pmd_ops.c')
 deps += ['bus_vdev']
-- 
2.21.0

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

* [dpdk-dev] [PATCH 3/3] crypto/zuc: fix dependency check
  2019-05-01 15:53 ` [dpdk-dev] [PATCH 3/3] crypto/zuc: " Bruce Richardson
@ 2019-05-01 15:53   ` Bruce Richardson
  0 siblings, 0 replies; 10+ messages in thread
From: Bruce Richardson @ 2019-05-01 15:53 UTC (permalink / raw)
  To: pablo.de.lara.guarch; +Cc: dev, Bruce Richardson

The check for the zuc library dependency did not check for the include
file needed for compilation. It's also recommended when looking for a
library to use the name without the lib prefix.

Fixes: df1740a8db5f ("crypto/zuc: enable meson build")

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/crypto/zuc/meson.build | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/crypto/zuc/meson.build b/drivers/crypto/zuc/meson.build
index 63f2a5298..fc2900244 100644
--- a/drivers/crypto/zuc/meson.build
+++ b/drivers/crypto/zuc/meson.build
@@ -1,13 +1,13 @@
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2018 Intel Corporation
 
-lib = cc.find_library('libsso_zuc', required: false)
-if not lib.found()
+lib = cc.find_library('sso_zuc', required: false)
+if not lib.found() or not cc.has_header('sso_zuc.h')
 	build = false
-else
-	ext_deps += lib
+	subdir_done()
 endif
 
 allow_experimental_apis = true
+ext_deps += lib
 sources = files('rte_zuc_pmd.c', 'rte_zuc_pmd_ops.c')
 deps += ['bus_vdev']
-- 
2.21.0


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

* Re: [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers
  2019-05-01 15:53 [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers Bruce Richardson
                   ` (3 preceding siblings ...)
  2019-05-01 15:53 ` [dpdk-dev] [PATCH 3/3] crypto/zuc: " Bruce Richardson
@ 2019-05-02  9:08 ` Thomas Monjalon
  2019-05-02  9:08   ` Thomas Monjalon
  4 siblings, 1 reply; 10+ messages in thread
From: Thomas Monjalon @ 2019-05-02  9:08 UTC (permalink / raw)
  To: Bruce Richardson; +Cc: dev, pablo.de.lara.guarch

01/05/2019 17:53, Bruce Richardson:
> Some fixes for the dependency checking of the kasumi and zuc PMDs, and an
> addition of meson support for snow3g which seems to have been overlooked
> previously.
> 
> [Apologies for the late submission, but hopefully these can make it into 19.05]
> 
> Bruce Richardson (3):
>   crypto/snow3g: add to meson build
>   crypto/kasumi: fix dependency check
>   crypto/zuc: fix dependency check

Cc: stable@dpdk.org

Applied, thanks

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

* Re: [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers
  2019-05-02  9:08 ` [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers Thomas Monjalon
@ 2019-05-02  9:08   ` Thomas Monjalon
  0 siblings, 0 replies; 10+ messages in thread
From: Thomas Monjalon @ 2019-05-02  9:08 UTC (permalink / raw)
  To: Bruce Richardson; +Cc: dev, pablo.de.lara.guarch

01/05/2019 17:53, Bruce Richardson:
> Some fixes for the dependency checking of the kasumi and zuc PMDs, and an
> addition of meson support for snow3g which seems to have been overlooked
> previously.
> 
> [Apologies for the late submission, but hopefully these can make it into 19.05]
> 
> Bruce Richardson (3):
>   crypto/snow3g: add to meson build
>   crypto/kasumi: fix dependency check
>   crypto/zuc: fix dependency check

Cc: stable@dpdk.org

Applied, thanks



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

end of thread, other threads:[~2019-05-02  9:08 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-01 15:53 [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers Bruce Richardson
2019-05-01 15:53 ` Bruce Richardson
2019-05-01 15:53 ` [dpdk-dev] [PATCH 1/3] crypto/snow3g: add to meson build Bruce Richardson
2019-05-01 15:53   ` Bruce Richardson
2019-05-01 15:53 ` [dpdk-dev] [PATCH 2/3] crypto/kasumi: fix dependency check Bruce Richardson
2019-05-01 15:53   ` Bruce Richardson
2019-05-01 15:53 ` [dpdk-dev] [PATCH 3/3] crypto/zuc: " Bruce Richardson
2019-05-01 15:53   ` Bruce Richardson
2019-05-02  9:08 ` [dpdk-dev] [PATCH 0/3] small fixes for crypto drivers Thomas Monjalon
2019-05-02  9:08   ` 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).