By introducing explicit -lrte_pmd_ixgbe link request in testpmd Makefile,"-Wl,-lrte_pmd_ixgbe" provided twice, and linker removes the duplication by keeping only first occurrence. This moves "-Wl,-lrte_pmd_ixgbe" out of "-Wl,--whole-archive" flag and makes symbol generation totally different than previous version in case of static build. This patch fixes the static build linking order by introducing -lrte_pmd_ixgbe under the shared library config (CONFIG_RTE_BUILD_SHARED_LIB).
Fixes: 425781ff5afe ("app/testpmd: add ixgbe VF management") CC: sta...@dpdk.org Signed-off-by: Jerin Jacob <jerin.ja...@caviumnetworks.com> --- app/test-pmd/Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/app/test-pmd/Makefile b/app/test-pmd/Makefile index 5988c3e..050663a 100644 --- a/app/test-pmd/Makefile +++ b/app/test-pmd/Makefile @@ -59,7 +59,9 @@ SRCS-y += csumonly.c SRCS-y += icmpecho.c SRCS-$(CONFIG_RTE_LIBRTE_IEEE1588) += ieee1588fwd.c +ifeq ($(CONFIG_RTE_BUILD_SHARED_LIB),y) _LDLIBS-$(CONFIG_RTE_LIBRTE_IXGBE_PMD) += -lrte_pmd_ixgbe +endif CFLAGS_cmdline.o := -D_GNU_SOURCE -- 2.5.5