diff mbox series

[ovs-dev,v2,2/3] dpif-netdev/avx512: add -fPIC flag to enable shared builds

Message ID 20200729105934.32606-2-harry.van.haaren@intel.com
State Accepted
Delegated to: Ian Stokes
Headers show
Series [ovs-dev,v2,1/3] dpif-netdev/avx512: avoid compiling avx512 code if binutils check fails | expand

Commit Message

Van Haaren, Harry July 29, 2020, 10:59 a.m. UTC
In certain scenarios with OVS built with --enable-shared and
DPDK enabled as shared build too, Position Independant Code
is required to link the avx512.a file into the relocatable .so
that it must be linked into.

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>

---

v3:
- Note this patch fixes a potential build issue that is only exposed
  with future DPDK Meson integration, and not an issue today. Ensuring
  OVS 2.14 release is compatible with DPDK Meson builds will ease the
  future porting to meson, so is worth including now.
---
 lib/automake.mk | 1 +
 1 file changed, 1 insertion(+)
diff mbox series

Patch

diff --git a/lib/automake.mk b/lib/automake.mk
index 218dc7313..380a67228 100644
--- a/lib/automake.mk
+++ b/lib/automake.mk
@@ -34,6 +34,7 @@  lib_libopenvswitchavx512_la_CFLAGS = \
 	-mavx512bw \
 	-mavx512dq \
 	-mbmi2 \
+	-fPIC \
 	$(AM_CFLAGS)
 lib_libopenvswitchavx512_la_SOURCES = \
 	lib/dpif-netdev-lookup-avx512-gather.c