drivers: add dependencies for some classes
Checks
Commit Message
A few classes meson.build were not expressing dependencies to the
associated device library. Define std_deps for baseband, gpu and regex
drivers.
Signed-off-by: David Marchand <david.marchand@redhat.com>
---
drivers/baseband/acc/meson.build | 2 +-
drivers/baseband/fpga_5gnr_fec/meson.build | 2 +-
drivers/baseband/fpga_lte_fec/meson.build | 2 +-
drivers/baseband/la12xx/meson.build | 2 +-
drivers/baseband/meson.build | 2 ++
drivers/baseband/null/meson.build | 2 +-
drivers/baseband/turbo_sw/meson.build | 2 +-
drivers/gpu/cuda/meson.build | 2 +-
drivers/gpu/meson.build | 2 ++
drivers/regex/cn9k/meson.build | 2 +-
drivers/regex/meson.build | 2 +-
drivers/regex/mlx5/meson.build | 2 +-
12 files changed, 14 insertions(+), 10 deletions(-)
Comments
On Fri, Aug 25, 2023 at 07:02:36PM +0200, David Marchand wrote:
> A few classes meson.build were not expressing dependencies to the
> associated device library. Define std_deps for baseband, gpu and regex
> drivers.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
> drivers/baseband/acc/meson.build | 2 +-
> drivers/baseband/fpga_5gnr_fec/meson.build | 2 +-
> drivers/baseband/fpga_lte_fec/meson.build | 2 +-
> drivers/baseband/la12xx/meson.build | 2 +-
> drivers/baseband/meson.build | 2 ++
> drivers/baseband/null/meson.build | 2 +-
> drivers/baseband/turbo_sw/meson.build | 2 +-
> drivers/gpu/cuda/meson.build | 2 +-
> drivers/gpu/meson.build | 2 ++
> drivers/regex/cn9k/meson.build | 2 +-
> drivers/regex/meson.build | 2 +-
> drivers/regex/mlx5/meson.build | 2 +-
> 12 files changed, 14 insertions(+), 10 deletions(-)
>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> From: David Marchand [mailto:david.marchand@redhat.com]
> Sent: Friday, 25 August 2023 19.03
>
> A few classes meson.build were not expressing dependencies to the
> associated device library. Define std_deps for baseband, gpu and regex
> drivers.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
On the surface, it looks like you are also removing a lot of (superfluous) dependencies. If not just a side effect of the added std_deps, perhaps the patch description should mention this too.
Anyway,
Acked-by: Morten Brørup <mb@smartsharesystems.com>
On Fri, Aug 25, 2023 at 8:18 PM Morten Brørup <mb@smartsharesystems.com> wrote:
>
> > From: David Marchand [mailto:david.marchand@redhat.com]
> > Sent: Friday, 25 August 2023 19.03
> >
> > A few classes meson.build were not expressing dependencies to the
> > associated device library. Define std_deps for baseband, gpu and regex
> > drivers.
> >
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > ---
>
> On the surface, it looks like you are also removing a lot of (superfluous) dependencies. If not just a side effect of the added std_deps, perhaps the patch description should mention this too.
The dependency to the device library in a driver become superfluous as
a result of adding it to std_deps:
- std_deps is passed to drivers as the default deps:
https://git.dpdk.org/dpdk/tree/drivers/meson.build#n125
- none of those updated drivers meson.build was overriding/resetting
deps (grep -w deps $patch).
I'll add a note in the commitlog to make this explicit.
>
> Anyway,
>
> Acked-by: Morten Brørup <mb@smartsharesystems.com>
>
Thanks.
On 8/25/23 19:02, David Marchand wrote:
> A few classes meson.build were not expressing dependencies to the
> associated device library. Define std_deps for baseband, gpu and regex
> drivers.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
> drivers/baseband/acc/meson.build | 2 +-
> drivers/baseband/fpga_5gnr_fec/meson.build | 2 +-
> drivers/baseband/fpga_lte_fec/meson.build | 2 +-
> drivers/baseband/la12xx/meson.build | 2 +-
> drivers/baseband/meson.build | 2 ++
> drivers/baseband/null/meson.build | 2 +-
> drivers/baseband/turbo_sw/meson.build | 2 +-
> drivers/gpu/cuda/meson.build | 2 +-
> drivers/gpu/meson.build | 2 ++
> drivers/regex/cn9k/meson.build | 2 +-
> drivers/regex/meson.build | 2 +-
> drivers/regex/mlx5/meson.build | 2 +-
> 12 files changed, 14 insertions(+), 10 deletions(-)
>
Acked-by: Maxime Coquelin <maxime.coquelin@redhat.com>
On Fri, Aug 25, 2023 at 7:03 PM David Marchand
<david.marchand@redhat.com> wrote:
>
> A few classes meson.build were not expressing dependencies to the
> associated device library. Define std_deps for baseband, gpu and regex
> drivers.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
Applied, with updated commitlog.
Thanks.
@@ -1,7 +1,7 @@
# SPDX-License-Identifier: BSD-3-Clause
# Copyright(c) 2020 Intel Corporation
-deps += ['bbdev', 'bus_pci']
+deps += ['bus_pci']
sources = files('rte_acc100_pmd.c', 'rte_vrb_pmd.c')
@@ -1,7 +1,7 @@
# SPDX-License-Identifier: BSD-3-Clause
# Copyright(c) 2020 Intel Corporation
-deps += ['bbdev', 'bus_vdev', 'ring', 'pci', 'bus_pci']
+deps += ['bus_vdev', 'ring', 'pci', 'bus_pci']
sources = files('rte_fpga_5gnr_fec.c')
@@ -1,5 +1,5 @@
# SPDX-License-Identifier: BSD-3-Clause
# Copyright(c) 2019 Intel Corporation
-deps += ['bbdev', 'bus_vdev', 'ring', 'pci', 'bus_pci']
+deps += ['bus_vdev', 'ring', 'pci', 'bus_pci']
sources = files('fpga_lte_fec.c')
@@ -1,6 +1,6 @@
# SPDX-License-Identifier: BSD-3-Clause
# Copyright 2020-2021 NXP
-deps += ['bbdev', 'bus_vdev', 'ring']
+deps += ['bus_vdev', 'ring']
sources = files('bbdev_la12xx.c')
@@ -14,4 +14,6 @@ drivers = [
'turbo_sw',
]
+std_deps = [ 'bbdev' ]
+
log_prefix = 'pmd.bb'
@@ -1,5 +1,5 @@
# SPDX-License-Identifier: BSD-3-Clause
# Copyright(c) 2018 Luca Boccassi <bluca@debian.org>
-deps += ['bbdev', 'bus_vdev', 'ring']
+deps += ['bus_vdev', 'ring']
sources = files('bbdev_null.c')
@@ -26,5 +26,5 @@ if dep_dec5g.found()
cflags += ['-DRTE_BBDEV_SDK_AVX512']
endif
-deps += ['bbdev', 'bus_vdev', 'ring']
+deps += ['bus_vdev', 'ring']
sources = files('bbdev_turbo_software.c')
@@ -21,5 +21,5 @@ if cc.has_header('gdrapi.h')
dpdk_conf.set('DRIVERS_GPU_CUDA_GDRCOPY_H', 1)
endif
-deps += ['gpudev', 'pci', 'bus_pci']
+deps += ['pci', 'bus_pci']
sources = files('cuda.c', 'gdrcopy.c')
@@ -2,3 +2,5 @@
# Copyright (c) 2021 NVIDIA Corporation & Affiliates
drivers = [ 'cuda' ]
+
+std_deps = [ 'gpudev' ]
@@ -12,5 +12,5 @@ sources = files(
'cn9k_regexdev.c',
)
-deps += ['bus_pci', 'regexdev']
+deps += ['bus_pci']
deps += ['common_cnxk', 'mempool_cnxk']
@@ -5,4 +5,4 @@ drivers = [
'mlx5',
'cn9k',
]
-std_deps = ['ethdev', 'kvargs'] # 'ethdev' also pulls in mbuf, net, eal etc
+std_deps = ['ethdev', 'kvargs', 'regexdev'] # 'ethdev' also pulls in mbuf, net, eal etc
@@ -7,7 +7,7 @@ if not is_linux
subdir_done()
endif
-deps += ['common_mlx5', 'eal', 'regexdev']
+deps += ['common_mlx5', 'eal']
if not dpdk_conf.has('RTE_COMMON_MLX5')
# avoid referencing undefined variables from common/mlx5
subdir_done()