Hi, From: David Marchand [mailto:[email protected]] Sent: Thursday, February 28, 2019 16:48 To: Xu, Rosen <[email protected]> Cc: [email protected]; Yigit, Ferruh <[email protected]>; Zhang, Tianfei <[email protected]>; Wei, Dan <[email protected]>; Pei, Andy <[email protected]>; Yang, Qiming <[email protected]>; Wang, Haiyue <[email protected]>; Chen, Santos <[email protected]>; Zhang, Zhang <[email protected]> Subject: Re: [dpdk-dev] [PATCH v1 08/11] app/test-pmd: add IPN3KE support for testpmd
On Thu, Feb 28, 2019 at 9:45 AM Xu, Rosen <[email protected]<mailto:[email protected]>> wrote: Hi, From: David Marchand [mailto:[email protected]<mailto:[email protected]>] Sent: Thursday, February 28, 2019 16:37 To: Xu, Rosen <[email protected]<mailto:[email protected]>> Cc: [email protected]<mailto:[email protected]>; Yigit, Ferruh <[email protected]<mailto:[email protected]>>; Zhang, Tianfei <[email protected]<mailto:[email protected]>>; Wei, Dan <[email protected]<mailto:[email protected]>>; Pei, Andy <[email protected]<mailto:[email protected]>>; Yang, Qiming <[email protected]<mailto:[email protected]>>; Wang, Haiyue <[email protected]<mailto:[email protected]>>; Chen, Santos <[email protected]<mailto:[email protected]>>; Zhang, Zhang <[email protected]<mailto:[email protected]>> Subject: Re: [dpdk-dev] [PATCH v1 08/11] app/test-pmd: add IPN3KE support for testpmd On Thu, Feb 28, 2019 at 8:16 AM Rosen Xu <[email protected]<mailto:[email protected]>> wrote: Add IPN3KE support for testpmd Signed-off-by: Rosen Xu <[email protected]<mailto:[email protected]>> Signed-off-by: Andy Pei <[email protected]<mailto:[email protected]>> --- app/test-pmd/Makefile | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/app/test-pmd/Makefile b/app/test-pmd/Makefile index d5258ea..a6b6f6f 100644 --- a/app/test-pmd/Makefile +++ b/app/test-pmd/Makefile @@ -62,6 +62,10 @@ ifeq ($(CONFIG_RTE_LIBRTE_I40E_PMD),y) LDLIBS += -lrte_pmd_i40e endif +ifeq ($(CONFIG_RTE_LIBRTE_IPN3KE_PMD),y) +LDLIBS += -lrte_pmd_ipn3ke +endif + ifeq ($(CONFIG_RTE_LIBRTE_BNXT_PMD),y) LDLIBS += -lrte_pmd_bnxt endif I am a bit lost. Why is it needed ? Involve IPN3KE in testpmd, for testpmd is widely used in NIC. The drivers declared in this makefile are there because testpmd directly calls private apis of those drivers. Bis repetita, why is linking to ipn3ke needed here ? It will be fixed in v2 patch. Thanks. -- David Marchand
