examples: fix linking against specific drivers
Checks
Commit Message
Some example apps rely on driver-specific functionality and link explicitly
against those drivers. These apps need their makefiles updated to take
account of the renaming of the driver libs.
Fixes: a20b2c01a7a1 ("build: standardize component names and defines")
Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
examples/bond/Makefile | 2 +-
examples/ethtool/lib/Makefile | 2 +-
examples/vm_power_manager/Makefile | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
Comments
On Tue, Oct 27, 2020 at 6:20 PM Bruce Richardson
<bruce.richardson@intel.com> wrote:
>
> Some example apps rely on driver-specific functionality and link explicitly
> against those drivers. These apps need their makefiles updated to take
> account of the renaming of the driver libs.
>
> Fixes: a20b2c01a7a1 ("build: standardize component names and defines")
>
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
Reviewed-by: David Marchand <david.marchand@redhat.com>
Applied, thanks Bruce.
I think the documentation would also need some fixes as I can see a
lot of references to librte_pmd_.
> -----Original Message-----
> From: David Marchand <david.marchand@redhat.com>
> Sent: Friday, October 30, 2020 1:36 PM
> To: Richardson, Bruce <bruce.richardson@intel.com>
> Cc: dev <dev@dpdk.org>; Thomas Monjalon <thomas@monjalon.net>
> Subject: Re: [dpdk-dev] [PATCH] examples: fix linking against specific
> drivers
>
> On Tue, Oct 27, 2020 at 6:20 PM Bruce Richardson
> <bruce.richardson@intel.com> wrote:
> >
> > Some example apps rely on driver-specific functionality and link
> explicitly
> > against those drivers. These apps need their makefiles updated to take
> > account of the renaming of the driver libs.
> >
> > Fixes: a20b2c01a7a1 ("build: standardize component names and defines")
> >
> > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> Reviewed-by: David Marchand <david.marchand@redhat.com>
>
> Applied, thanks Bruce.
>
>
> I think the documentation would also need some fixes as I can see a
> lot of references to librte_pmd_.
>
First draft patchset sent, based off grepping the docs for "librte_pmd_".
http://patches.dpdk.org/project/dpdk/list/?series=13578
/Bruce
@@ -19,7 +19,7 @@ shared: build/$(APP)-shared
static: build/$(APP)-static
ln -sf $(APP)-static build/$(APP)
-LDFLAGS += -lrte_pmd_bond
+LDFLAGS += -lrte_net_bond
PKGCONF ?= pkg-config
@@ -27,7 +27,7 @@ LDFLAGS += -Wl,--no-undefined $(LDFLAGS_SHARED)
# check for ixgbe by grepping pre-processor output
ifneq ($(shell $(CC) $(CFLAGS) -dM -E - < /dev/null | grep IXGBE),)
-LDFLAGS += -lrte_pmd_ixgbe
+LDFLAGS += -lrte_net_ixgbe
endif
.PHONY: all clean static shared
@@ -44,7 +44,7 @@ CFLAGS += -DUSE_JANSSON
endif
# for shared library builds, we need to explicitly link these PMDs
-LDFLAGS_SHARED += -lrte_pmd_ixgbe -lrte_pmd_i40e -lrte_pmd_bnxt
+LDFLAGS_SHARED += -lrte_net_ixgbe -lrte_net_i40e -lrte_net_bnxt
build/$(APP)-shared: $(SRCS-y) Makefile $(PC_FILE) | build
$(CC) $(CFLAGS) $(SRCS-y) -o $@ $(LDFLAGS) $(LDFLAGS_SHARED)