[dpdk-dev] app/testpmd: Remove xenvirt again
Checks
Commit Message
Fixes: 5b590fbe09b6 ("app/testpmd: add traffic management forwarding mode")
Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>
---
app/test-pmd/Makefile | 4 ----
1 file changed, 4 deletions(-)
Comments
On Fri, 2017-11-17 at 15:40 +0100, Timothy Redaelli wrote:
> Fixes: 5b590fbe09b6 ("app/testpmd: add traffic management forwarding
> mode")
>
> Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>
> ---
> app/test-pmd/Makefile | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/app/test-pmd/Makefile b/app/test-pmd/Makefile
> index d21308fcd..4993c9184 100644
> --- a/app/test-pmd/Makefile
> +++ b/app/test-pmd/Makefile
> @@ -83,10 +83,6 @@ ifeq ($(CONFIG_RTE_LIBRTE_BNXT_PMD),y)
> LDLIBS += -lrte_pmd_bnxt
> endif
>
> -ifeq ($(CONFIG_RTE_LIBRTE_PMD_XENVIRT),y)
> -LDLIBS += -lrte_pmd_xenvirt
> -endif
> -
> ifeq ($(CONFIG_RTE_LIBRTE_PMD_SOFTNIC),y)
> LDLIBS += -lrte_pmd_softnic
> endif
Acked-by: <bluca@debian.org>
On 17/11/17 15:40, Timothy Redaelli wrote:
> Fixes: 5b590fbe09b6 ("app/testpmd: add traffic management forwarding mode")
>
> Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>
> ---
> app/test-pmd/Makefile | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/app/test-pmd/Makefile b/app/test-pmd/Makefile
> index d21308fcd..4993c9184 100644
> --- a/app/test-pmd/Makefile
> +++ b/app/test-pmd/Makefile
> @@ -83,10 +83,6 @@ ifeq ($(CONFIG_RTE_LIBRTE_BNXT_PMD),y)
> LDLIBS += -lrte_pmd_bnxt
> endif
>
> -ifeq ($(CONFIG_RTE_LIBRTE_PMD_XENVIRT),y)
> -LDLIBS += -lrte_pmd_xenvirt
> -endif
> -
> ifeq ($(CONFIG_RTE_LIBRTE_PMD_SOFTNIC),y)
> LDLIBS += -lrte_pmd_softnic
> endif
Acked-by: Eelco Chaudron <echaudro@redhat.com>
17/11/2017 17:44, Luca Boccassi:
> On Fri, 2017-11-17 at 15:40 +0100, Timothy Redaelli wrote:
> > Fixes: 5b590fbe09b6 ("app/testpmd: add traffic management forwarding
> > mode")
> >
> > Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>
>
> Acked-by: <bluca@debian.org>
Applied with this explanation:
The xenvirt PMD was removed from 17.11
while this link parameter was added.
@@ -83,10 +83,6 @@ ifeq ($(CONFIG_RTE_LIBRTE_BNXT_PMD),y)
LDLIBS += -lrte_pmd_bnxt
endif
-ifeq ($(CONFIG_RTE_LIBRTE_PMD_XENVIRT),y)
-LDLIBS += -lrte_pmd_xenvirt
-endif
-
ifeq ($(CONFIG_RTE_LIBRTE_PMD_SOFTNIC),y)
LDLIBS += -lrte_pmd_softnic
endif