[dpdk-stable] patch 'net/mlx: fix overlinking with meson and glue dlopen' has been queued to stable release 19.11.1

Luca Boccassi luca.boccassi at gmail.com
Mon Feb 17 18:50:32 CET 2020


This one required a few extra backports, you can see the end result at:

https://github.com/bluca/dpdk-stable/blob/19.11/drivers/net/mlx4/meson.build
https://github.com/bluca/dpdk-stable/blob/19.11/drivers/net/mlx5/meson.build

I can drop the series of changes to mlx's meson.build alternatively if
you prefer.

On Mon, 2020-02-17 at 17:45 +0000, luca.boccassi at gmail.com wrote:
> Hi,
> 
> FYI, your patch has been queued to stable release 19.11.1
> 
> Note it hasn't been pushed to 
> http://dpdk.org/browse/dpdk-stable
>  yet.
> It will be pushed if I get no objections before 02/19/20. So please
> shout if anyone has objections.
> 
> Also note that after the patch there's a diff of the upstream commit
> vs the
> patch applied to the branch. This will indicate if there was any
> rebasing
> needed to apply to the stable branch. If there were code changes for
> rebasing
> (ie: not only metadata diffs), please double check that the rebase
> was
> correctly done.
> 
> Thanks.
> 
> Luca Boccassi
> 
> ---
> From 3253622dc4630e11207629aba50293049938ec20 Mon Sep 17 00:00:00
> 2001
> From: Thomas Monjalon <
> thomas at monjalon.net
> >
> Date: Wed, 12 Feb 2020 23:07:08 +0100
> Subject: [PATCH] net/mlx: fix overlinking with meson and glue dlopen
> 
> [ upstream commit e21492a5175af31ae568fa5fabef1f4b22706962 ]
> 
> If ibverbs_link is dlopen, the PMD and application should not
> be linked with ibverbs, but the glue library is.
> Unfortunately the ibverbs dependency was exported in the
> variable ext_deps, so there were overlinking.
> 
> It is fixed by not exporting the dependency in ext_deps,
> and recreating a limited dependency object for cflags only.
> 
> Fixes: 1dd7c7e38c19 ("net/mlx4: support meson build")
> Fixes: 96d7c62a70c7 ("net/mlx5: support meson build")
> Cc: 
> stable at dpdk.org
> 
> 
> Signed-off-by: Thomas Monjalon <
> thomas at monjalon.net
> >
> Acked-by: Bruce Richardson <
> bruce.richardson at intel.com
> >
> ---
>  drivers/net/mlx4/meson.build | 6 ++++--
>  drivers/net/mlx5/meson.build | 4 +++-
>  2 files changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/net/mlx4/meson.build
> b/drivers/net/mlx4/meson.build
> index 17711f154b..ff7386aee2 100644
> --- a/drivers/net/mlx4/meson.build
> +++ b/drivers/net/mlx4/meson.build
> @@ -31,7 +31,7 @@ foreach libname:libnames
>  	endif
>  	if lib.found()
>  		libs += lib
> -		if not static_ibverbs
> +		if not static_ibverbs and not dlopen_ibverbs
>  			ext_deps += lib
>  		endif
>  	else
> @@ -41,10 +41,12 @@ foreach libname:libnames
>  endforeach
>  
>  if build
> -	if static_ibverbs
> +	if static_ibverbs or dlopen_ibverbs
>  		# Build without adding shared libs to Requires.private
>  		ibv_cflags = run_command(pkgconf, '--cflags',
> 'libibverbs').stdout()
>  		ext_deps += declare_dependency(compile_args:
> ibv_cflags.split())
> +	endif
> +	if static_ibverbs
>  		# Add static deps ldflags to internal apps and
> Libs.private
>  		ibv_ldflags = run_command(ldflags_ibverbs_static,
> check:true).stdout()
>  		ext_deps +=
> declare_dependency(link_args:ibv_ldflags.split())
> diff --git a/drivers/net/mlx5/meson.build
> b/drivers/net/mlx5/meson.build
> index e21f3d8dc8..dab1b83fd3 100644
> --- a/drivers/net/mlx5/meson.build
> +++ b/drivers/net/mlx5/meson.build
> @@ -41,10 +41,12 @@ foreach libname:libnames
>  endforeach
>  
>  if build
> -	if static_ibverbs
> +	if static_ibverbs or dlopen_ibverbs
>  		# Build without adding shared libs to Requires.private
>  		ibv_cflags = run_command(pkgconf, '--cflags',
> 'libibverbs').stdout()
>  		ext_deps += declare_dependency(compile_args:
> ibv_cflags.split())
> +	endif
> +	if static_ibverbs
>  		# Add static deps ldflags to internal apps and
> Libs.private
>  		ibv_ldflags = run_command(ldflags_ibverbs_static,
> check:true).stdout()
>  		ext_deps +=
> declare_dependency(link_args:ibv_ldflags.split())
> 
-- 
Kind regards,
Luca Boccassi



More information about the stable mailing list