Re: [dpdk-dev] [PATCH v2 0/4] net/mlx: make rdma-core optional at run-time

["Nélio Laranjeiro" <nelio.laranjeiro@6wind.com>]

On Fri, Jan 26, 2018 at 03:18:55PM +0100, Adrien Mazarguil wrote:
> A problem encountered with Mellanox PMDs and frequently reported by DPDK
> application developers and Linux distribution package maintainers is their
> dependency on rdma-core components, namely libibverbs, libmlx4, and libmlx5.
> 
> For best performance in applications, DPDK is normally built as a collection
> of library archives (.a files), whose external dependencies are inherited
> through rte.app.mk during link.
> 
> When these PMDs are built-in, any binary DPDK package, whether DPDK itself
> or derived applications, always have to pull rdma-core. This dependency is
> not obvious and may be missed during the packaging of any intermediate layer
> between DPDK itself and the end application.
> 
> While still required during compilation, this series trades this hard
> dependency for an optional one, dynamically loaded at run-time through
> dlopen().
> 
> It supersedes Shachar's previous work on the same topic [1] using a
> different approach in order to preserve symbol versioning and address
> the remaining issues.
> 
> [1] http://dpdk.org/ml/archives/dev/2017-December/085090.html
> 
> v2 changes:
> 
> - Rebased on dpdk-next-net-mlx.
> - Rely on CONFIG_RTE_LIBRTE_MLX[45]_DLOPEN_DEPS instead of
>   CONFIG_RTE_BUILD_SHARED_LIB to enable this mode, so it can be overridden
>   if necessary.
> - Fixed -lmlx5 -libibverbs leftovers in rte.app.mk.
> - Minor fixes for indentation and unnecessary includes in mlx5.
> 
> Adrien Mazarguil (3):
>   net/mlx4: move rdma-core calls to separate file
>   net/mlx4: spawn rdma-core dependency plug-in
>   net/mlx5: spawn rdma-core dependency plug-in
> 
> Nelio Laranjeiro (1):
>   net/mlx5: move rdma-core calls to separate file
> 
>  config/common_base             |   2 +
>  doc/guides/nics/mlx4.rst       |  13 ++
>  doc/guides/nics/mlx5.rst       |  13 ++
>  drivers/net/mlx4/Makefile      |  42 +++++
>  drivers/net/mlx4/mlx4.c        | 115 +++++++++++--
>  drivers/net/mlx4/mlx4_ethdev.c |   3 +-
>  drivers/net/mlx4/mlx4_flow.c   |  32 ++--
>  drivers/net/mlx4/mlx4_glue.c   | 275 ++++++++++++++++++++++++++++++
>  drivers/net/mlx4/mlx4_glue.h   |  80 +++++++++
>  drivers/net/mlx4/mlx4_intr.c   |  10 +-
>  drivers/net/mlx4/mlx4_mr.c     |   7 +-
>  drivers/net/mlx4/mlx4_rxq.c    |  53 +++---
>  drivers/net/mlx4/mlx4_txq.c    |  17 +-
>  drivers/net/mlx5/Makefile      |  42 +++++
>  drivers/net/mlx5/mlx5.c        | 123 +++++++++++---
>  drivers/net/mlx5/mlx5_ethdev.c |   7 +-
>  drivers/net/mlx5/mlx5_flow.c   |  92 +++++------
>  drivers/net/mlx5/mlx5_glue.c   | 321 ++++++++++++++++++++++++++++++++++++
>  drivers/net/mlx5/mlx5_glue.h   |  99 +++++++++++
>  drivers/net/mlx5/mlx5_mr.c     |   7 +-
>  drivers/net/mlx5/mlx5_rxq.c    |  54 +++---
>  drivers/net/mlx5/mlx5_txq.c    |  22 +--
>  drivers/net/mlx5/mlx5_vlan.c   |  13 +-
>  mk/rte.app.mk                  |   8 +
>  24 files changed, 1265 insertions(+), 185 deletions(-)
>  create mode 100644 drivers/net/mlx4/mlx4_glue.c
>  create mode 100644 drivers/net/mlx4/mlx4_glue.h
>  create mode 100644 drivers/net/mlx5/mlx5_glue.c
>  create mode 100644 drivers/net/mlx5/mlx5_glue.h
> 
> -- 
> 2.11.0

Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>

-- 
Nélio Laranjeiro
6WIND