net/mlx4: fix build with -fno-common
authorThomas Monjalon <thomas@monjalon.net>
Wed, 8 Apr 2020 00:09:00 +0000 (02:09 +0200)
committerFerruh Yigit <ferruh.yigit@intel.com>
Tue, 21 Apr 2020 11:57:07 +0000 (13:57 +0200)
commita5e5af7f20b4106b68eeedf83daada0371b83918
treec9e02b6ba40a640e7def6b84fdd9a046e88cac66
parent2889d38256906a5cbff717e7e524ec645c89c794
net/mlx4: fix build with -fno-common

The variable storages of the same name are merged together
if compiled with -fcommon. This is the default.
This default behaviour allows to declare a variable in a header file and
share the variable in every .o binaries thanks to merge at link-time.

In the case of dlopen linking of the glue library, the pointer mlx4_glue
is referencing the glue functions struct and is set after calling
dlopen.

If compiling with -fno-common (default in GCC 10), the variables must be
declared as extern to avoid multiple re-definitions.
In case the glue layer is split in glue library, the variable mlx4_glue
needs to have its own storage for the rest of the PMD.

Cc: stable@dpdk.org
Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
Acked-by: Matan Azrad <matan@mellanox.com>
drivers/net/mlx4/mlx4.c
drivers/net/mlx4/mlx4_glue.h
drivers/net/mlx4/mlx4_rxtx.h