(2014/09/30 20:23), Neil Horman wrote: > On Tue, Sep 30, 2014 at 06:56:10PM +0900, mukawa at igel.co.jp wrote: >> From: Tetsuya Mukawa <mukawa at igel.co.jp> >> >> Signed-off-by: Tetsuya Mukawa <mukawa at igel.co.jp> >> --- >> mk/rte.app.mk | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/mk/rte.app.mk b/mk/rte.app.mk >> index 34dff2a..f059290 100644 >> --- a/mk/rte.app.mk >> +++ b/mk/rte.app.mk >> @@ -179,6 +179,10 @@ LDLIBS += -lrte_pmd_xenvirt >> LDLIBS += -lxenstore >> endif >> >> +ifeq ($(CONFIG_RTE_LIBRTE_PMD_NULL),y) >> +LDLIBS += -lrte_pmd_null >> +endif >> + > You don't need to add this, as the pmd can be loaded dynamically via the > dlopen > call executed via the -d option on the test app command line. The only pmds > that need explicit linking are those that offer additional API calls to an > appilcation. Thanks, I will change above.
Regards, Tetsuya > Neil > >> ifeq ($(CONFIG_RTE_BUILD_SHARED_LIB),n) >> # plugins (link only if static libraries) >> >> -- >> 1.9.1 >> >>