Hi Ferruh,
> -----Original Message----- > From: Yigit, Ferruh > Sent: Tuesday, December 20, 2016 1:10 AM > To: dev@dpdk.org > Cc: Wu, Jingjing; Zhang, Helin; Lu, Wenzhuo; Chen, Jing D; Iremonger, Bernard > Subject: Re: [PATCH v5 18/29] app/testpmd: use VFD APIs on i40e > > On 12/16/2016 7:02 PM, Ferruh Yigit wrote: > > From: Wenzhuo Lu <wenzhuo...@intel.com> > > > > The new VF Daemon (VFD) APIs is implemented on i40e. Change testpmd > > code to use them, including VF MAC anti-spoofing, VF VLAN > > anti-spoofing, TX loopback, VF VLAN strip, VF VLAN insert. > > > > Signed-off-by: Wenzhuo Lu <wenzhuo...@intel.com> > > Signed-off-by: Chen Jing D(Mark) <jing.d.c...@intel.com> > > Signed-off-by: Bernard Iremonger <bernard.iremon...@intel.com> > > --- > > app/test-pmd/Makefile | 2 + > > app/test-pmd/cmdline.c | 150 > > +++++++++++++++++++++++++++++++++++++++---------- > > 2 files changed, 121 insertions(+), 31 deletions(-) > > > > diff --git a/app/test-pmd/Makefile b/app/test-pmd/Makefile index > > 891b85a..a0c3366 100644 > > --- a/app/test-pmd/Makefile > > +++ b/app/test-pmd/Makefile > > @@ -58,7 +58,9 @@ SRCS-y += csumonly.c SRCS-y += icmpecho.c > > SRCS-$(CONFIG_RTE_LIBRTE_IEEE1588) += ieee1588fwd.c > > > > +_LDLIBS-y += --whole-archive > > Hi Wenzhuo, > > Following lines are required for shared library, but I guess above line > required > because they cause problem with static library. > So, instead of adding above line, what do you think wrapping below lines with > ifeq ($(CONFIG_RTE_BUILD_SHARED_LIB),y) ? Thanks for your suggestion. It's also a choice. I'll change it. > > > _LDLIBS-$(CONFIG_RTE_LIBRTE_IXGBE_PMD) += -lrte_pmd_ixgbe > > +_LDLIBS-$(CONFIG_RTE_LIBRTE_I40E_PMD) += -lrte_pmd_i40e > > > > CFLAGS_cmdline.o := -D_GNU_SOURCE > > > <...>