Re: [dpdk-dev] [PATCH v6 2/3] common/octeontx: enable build only on 64-bit Linux

[Harman Kalra <hkalra@marvell.com>]

> -----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