[dpdk-dev] [PATCH] net/mlx5: Avoid stripping the glue library

Christian Ehrhardt christian.ehrhardt at canonical.com
Tue Jul 31 15:33:13 CEST 2018


On Tue, Jul 31, 2018 at 3:27 PM Timothy Redaelli <tredaelli at redhat.com>
wrote:

> Stripping binaries at build time is usually a bad thing since it makes
> impossible to generate (split) debug symbols and this can lead to a more
> difficult debugging.
>
> CC: Adrien Mazarguil <adrien.mazarguil at 6wind.com>
> Fixes: 59b91bec12c6 ("net/mlx5: spawn rdma-core dependency plug-in")
>
> Signed-off-by: Timothy Redaelli <tredaelli at redhat.com>
> ---
>  drivers/net/mlx5/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/mlx5/Makefile b/drivers/net/mlx5/Makefile
> index 75bee4640..2e70dec5b 100644
> --- a/drivers/net/mlx5/Makefile
> +++ b/drivers/net/mlx5/Makefile
> @@ -409,7 +409,7 @@ endif
>  $(LIB_GLUE): mlx5_glue.o
>         $Q $(LD) $(GLUE_LDFLAGS) $(EXTRA_LDFLAGS) \
>                 -Wl,-h,$(LIB_GLUE) \
> -               -s -shared -o $@ $< -libverbs -lmlx5
> +               -shared -o $@ $< -libverbs -lmlx5
>
>  mlx5_glue.o: mlx5_autoconf.h
>

After IRC discussion I completely agree that this shouldn't be stripped,
thanks for the fix.

Acked-by: Christian Ehrhardt <christian.ehrhardt at canonical.com>



>
> --
> 2.17.1
>
>

-- 
Christian Ehrhardt
Software Engineer, Ubuntu Server
Canonical Ltd


More information about the dev mailing list