On Thu, Feb 28, 2019 at 9:45 AM Xu, Rosen <[email protected]> wrote:

> Hi,
>
>
>
> *From:* David Marchand [mailto:[email protected]]
> *Sent:* Thursday, February 28, 2019 16:37
> *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 8:16 AM Rosen Xu <[email protected]> wrote:
>
> Add IPN3KE support for testpmd
>
> Signed-off-by: Rosen Xu <[email protected]>
> Signed-off-by: Andy Pei <[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 ?


-- 
David Marchand

Reply via email to