On 6/11/2016 7:34 AM, Thomas Monjalon wrote: > Hi Ferruh, > > 2016-06-10 19:32, Ferruh Yigit: >> --- a/mk/rte.app.mk >> +++ b/mk/rte.app.mk >> @@ -50,6 +50,14 @@ ifeq ($(NO_LDSCRIPT),) >> LDSCRIPT = $(RTE_LDSCRIPT) >> endif >> >> +ifeq ($(CONFIG_RTE_BUILD_SHARED_LIB),y) >> +# Workaround to eal <-> mempool cyclic dependency >> +_LDLIBS-$(CONFIG_RTE_LIBRTE_MEMPOOL) += -lrte_mempool >> +endif >> + >> +# Link only the libraries used in the application >> +_LDLIBS-y += --as-needed > > I think we do not need this workaround. > The dependency of EAL logs on mempool is now removed. > We only have the dependency of EAL ivshmem on librte_ivshmem header > (which needs mempool and rings headers). >
I missed that dependency removed, this is good, I will send a new version of the patch. Thanks, ferruh
