[v6,2/3] common/octeontx: enable build only on 64-bit Linux

Message ID 20211014195653.625-2-pbhagavatula@marvell.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series [v6,1/3] net/thunderx: enable build only on 64-bit Linux |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Pavan Nikhilesh Bhagavatula Oct. 14, 2021, 7:56 p.m. UTC
  From: Pavan Nikhilesh <pbhagavatula@marvell.com>

Since AARCH32 extension is not implemented on octeontx family, only
enable build for 64bit.
Due to Linux kernel AF(Admin function) driver dependency, only enable
build for 64-bit Linux.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
Acked-by: Jerin Jacob <jerinj@marvell.com>
---
 drivers/common/octeontx/meson.build   | 4 ++--
 drivers/compress/octeontx/meson.build | 6 ++++++
 drivers/crypto/octeontx/meson.build   | 7 +++++++
 drivers/event/octeontx/meson.build    | 6 ++++++
 drivers/mempool/octeontx/meson.build  | 5 +++--
 drivers/net/octeontx/meson.build      | 4 ++--
 6 files changed, 26 insertions(+), 6 deletions(-)
  

Comments

Harman Kalra Oct. 18, 2021, 7:03 a.m. UTC | #1
> -----Original Message-----
> From: pbhagavatula@marvell.com <pbhagavatula@marvell.com>
> Sent: Friday, October 15, 2021 1:27 AM
> To: ferruh.yigit@intel.com; Jerin Jacob Kollanukkaran <jerinj@marvell.com>;
> Harman Kalra <hkalra@marvell.com>; Ashish Gupta
> <ashishg@marvell.com>; Fiona Trahe <fiona.trahe@intel.com>; Anoob
> Joseph <anoobj@marvell.com>
> Cc: dev@dpdk.org; Pavan Nikhilesh Bhagavatula
> <pbhagavatula@marvell.com>
> Subject: [dpdk-dev] [PATCH v6 2/3] common/octeontx: enable build only on
> 64-bit Linux
> 
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> 
> Since AARCH32 extension is not implemented on octeontx family, only
> enable build for 64bit.
> Due to Linux kernel AF(Admin function) driver dependency, only enable build
> for 64-bit Linux.
> 
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> Acked-by: Jerin Jacob <jerinj@marvell.com>
> ---

Acked-by: Harman Kalra <hkalra@marvell.com>


Thanks
Harman

>  drivers/common/octeontx/meson.build   | 4 ++--
>  drivers/compress/octeontx/meson.build | 6 ++++++
>  drivers/crypto/octeontx/meson.build   | 7 +++++++
>  drivers/event/octeontx/meson.build    | 6 ++++++
>  drivers/mempool/octeontx/meson.build  | 5 +++--
>  drivers/net/octeontx/meson.build      | 4 ++--
>  6 files changed, 26 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/common/octeontx/meson.build
> b/drivers/common/octeontx/meson.build
> index fcda0f6445..dc060dfea1 100644
> --- a/drivers/common/octeontx/meson.build
> +++ b/drivers/common/octeontx/meson.build
> @@ -2,9 +2,9 @@
>  # Copyright(c) 2018 Cavium, Inc
>  #
> 
> -if is_windows
> +if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
>      build = false
> -    reason = 'not supported on Windows'
> +    reason = 'only supported on 64-bit Linux'
>      subdir_done()
>  endif
> 
> diff --git a/drivers/compress/octeontx/meson.build
> b/drivers/compress/octeontx/meson.build
> index e1b7bed422..cd8687fde3 100644
> --- a/drivers/compress/octeontx/meson.build
> +++ b/drivers/compress/octeontx/meson.build
> @@ -1,6 +1,12 @@
>  # SPDX-License-Identifier: BSD-3-Clause  # Copyright(c) 2018 Cavium, Inc
> 
> +if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
> +	build = false
> +	reason = 'only supported on 64-bit Linux'
> +	subdir_done()
> +endif
> +
>  sources = files('otx_zip.c', 'otx_zip_pmd.c')  includes +=
> include_directories('include')  deps += ['mempool_octeontx', 'bus_pci'] diff --
> git a/drivers/crypto/octeontx/meson.build
> b/drivers/crypto/octeontx/meson.build
> index 244b16230e..bc6187e1cf 100644
> --- a/drivers/crypto/octeontx/meson.build
> +++ b/drivers/crypto/octeontx/meson.build
> @@ -1,5 +1,12 @@
>  # SPDX-License-Identifier: BSD-3-Clause  # Copyright(c) 2018 Cavium, Inc
> +#
> +
> +if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
> +    build = false
> +    reason = 'only supported on 64-bit Linux'
> +    subdir_done()
> +endif
> 
>  deps += ['bus_pci']
>  deps += ['bus_vdev']
> diff --git a/drivers/event/octeontx/meson.build
> b/drivers/event/octeontx/meson.build
> index 0d9eec3f2e..eb17e059d8 100644
> --- a/drivers/event/octeontx/meson.build
> +++ b/drivers/event/octeontx/meson.build
> @@ -1,6 +1,12 @@
>  # SPDX-License-Identifier: BSD-3-Clause  # Copyright(c) 2017 Cavium, Inc
> 
> +if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
> +    build = false
> +    reason = 'only supported on 64-bit Linux'
> +    subdir_done()
> +endif
> +
>  sources = files(
>          'ssovf_worker.c',
>          'ssovf_evdev.c',
> diff --git a/drivers/mempool/octeontx/meson.build
> b/drivers/mempool/octeontx/meson.build
> index a57213e2d8..fb05928129 100644
> --- a/drivers/mempool/octeontx/meson.build
> +++ b/drivers/mempool/octeontx/meson.build
> @@ -1,9 +1,10 @@
>  # SPDX-License-Identifier: BSD-3-Clause  # Copyright(c) 2017 Cavium, Inc
> 
> -if is_windows
> +if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
>      build = false
> -    reason = 'not supported on Windows'
> +    reason = 'only supported on 64-bit Linux'
> +    subdir_done()
>  endif
> 
>  sources = files(
> diff --git a/drivers/net/octeontx/meson.build
> b/drivers/net/octeontx/meson.build
> index 84cd4e1142..a5a6c9661c 100644
> --- a/drivers/net/octeontx/meson.build
> +++ b/drivers/net/octeontx/meson.build
> @@ -1,9 +1,9 @@
>  # SPDX-License-Identifier: BSD-3-Clause  # Copyright(c) 2017 Cavium, Inc
> 
> -if is_windows
> +if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
>      build = false
> -    reason = 'not supported on Windows'
> +    reason = 'only supported on 64-bit Linux'
>      subdir_done()
>  endif
> 
> --
> 2.17.1
  

Patch

diff --git a/drivers/common/octeontx/meson.build b/drivers/common/octeontx/meson.build
index fcda0f6445..dc060dfea1 100644
--- a/drivers/common/octeontx/meson.build
+++ b/drivers/common/octeontx/meson.build
@@ -2,9 +2,9 @@ 
 # Copyright(c) 2018 Cavium, Inc
 #
 
-if is_windows
+if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
     build = false
-    reason = 'not supported on Windows'
+    reason = 'only supported on 64-bit Linux'
     subdir_done()
 endif
 
diff --git a/drivers/compress/octeontx/meson.build b/drivers/compress/octeontx/meson.build
index e1b7bed422..cd8687fde3 100644
--- a/drivers/compress/octeontx/meson.build
+++ b/drivers/compress/octeontx/meson.build
@@ -1,6 +1,12 @@ 
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2018 Cavium, Inc
 
+if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
+	build = false
+	reason = 'only supported on 64-bit Linux'
+	subdir_done()
+endif
+
 sources = files('otx_zip.c', 'otx_zip_pmd.c')
 includes += include_directories('include')
 deps += ['mempool_octeontx', 'bus_pci']
diff --git a/drivers/crypto/octeontx/meson.build b/drivers/crypto/octeontx/meson.build
index 244b16230e..bc6187e1cf 100644
--- a/drivers/crypto/octeontx/meson.build
+++ b/drivers/crypto/octeontx/meson.build
@@ -1,5 +1,12 @@ 
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2018 Cavium, Inc
+#
+
+if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
+    build = false
+    reason = 'only supported on 64-bit Linux'
+    subdir_done()
+endif
 
 deps += ['bus_pci']
 deps += ['bus_vdev']
diff --git a/drivers/event/octeontx/meson.build b/drivers/event/octeontx/meson.build
index 0d9eec3f2e..eb17e059d8 100644
--- a/drivers/event/octeontx/meson.build
+++ b/drivers/event/octeontx/meson.build
@@ -1,6 +1,12 @@ 
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2017 Cavium, Inc
 
+if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
+    build = false
+    reason = 'only supported on 64-bit Linux'
+    subdir_done()
+endif
+
 sources = files(
         'ssovf_worker.c',
         'ssovf_evdev.c',
diff --git a/drivers/mempool/octeontx/meson.build b/drivers/mempool/octeontx/meson.build
index a57213e2d8..fb05928129 100644
--- a/drivers/mempool/octeontx/meson.build
+++ b/drivers/mempool/octeontx/meson.build
@@ -1,9 +1,10 @@ 
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2017 Cavium, Inc
 
-if is_windows
+if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
     build = false
-    reason = 'not supported on Windows'
+    reason = 'only supported on 64-bit Linux'
+    subdir_done()
 endif
 
 sources = files(
diff --git a/drivers/net/octeontx/meson.build b/drivers/net/octeontx/meson.build
index 84cd4e1142..a5a6c9661c 100644
--- a/drivers/net/octeontx/meson.build
+++ b/drivers/net/octeontx/meson.build
@@ -1,9 +1,9 @@ 
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2017 Cavium, Inc
 
-if is_windows
+if not is_linux or not dpdk_conf.get('RTE_ARCH_64')
     build = false
-    reason = 'not supported on Windows'
+    reason = 'only supported on 64-bit Linux'
     subdir_done()
 endif