From patchwork Wed Jan 25 15:29:53 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ferruh Yigit X-Patchwork-Id: 20004 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 0DFD1108F; Wed, 25 Jan 2017 16:29:59 +0100 (CET) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id 5953D1023 for ; Wed, 25 Jan 2017 16:29:57 +0100 (CET) Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga105.jf.intel.com with ESMTP; 25 Jan 2017 07:29:56 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,284,1477983600"; d="scan'208";a="57113296" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.38]) ([10.237.220.38]) by orsmga005.jf.intel.com with ESMTP; 25 Jan 2017 07:29:54 -0800 To: Thomas Monjalon , Hemant Agrawal References: <1484832240-2048-1-git-send-email-hemant.agrawal@nxp.com> <228ff5e7-2fa8-7731-681d-e4759bff93cb@nxp.com> <20101825.1zhD9dk20U@xps13> Cc: Shreyansh Jain , dev@dpdk.org, bruce.richardson@intel.com, john.mcnamara@intel.com, jerin.jacob@caviumnetworks.com From: Ferruh Yigit Message-ID: <29605f35-7cb2-e526-382c-175df14abcf8@intel.com> Date: Wed, 25 Jan 2017 15:29:53 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <20101825.1zhD9dk20U@xps13> Subject: Re: [dpdk-dev] [PATCHv6 16/33] drivers/pool/dpaa2: adding hw offloaded mempool X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 1/24/2017 5:28 PM, Thomas Monjalon wrote: > 2017-01-24 20:07, Hemant Agrawal: >> On 1/24/2017 4:19 PM, Ferruh Yigit wrote: >>> On 1/24/2017 9:12 AM, Shreyansh Jain wrote: >>>> On Monday 23 January 2017 11:04 PM, Ferruh Yigit wrote: >>>>> On 1/23/2017 11:59 AM, Hemant Agrawal wrote: >>>>>> +# library dependencies >>>>>> +DEPDIRS-$(CONFIG_RTE_LIBRTE_DPAA2_POOL) += lib/librte_eal >>>>>> +DEPDIRS-$(CONFIG_RTE_LIBRTE_DPAA2_POOL) += lib/librte_mempool >>>>>> +DEPDIRS-$(CONFIG_RTE_LIBRTE_DPAA2_POOL) += lib/librte_common_dpaa2_qbman >>>>> >>>>> This dependeny doesn not looks correct, there is no folder like that. >>>> >>>> This is something even I need to understand. From the DEPDIRS what I >>>> understood was that though it refers to a directory, it essentially >>>> links libraries in build/lib/*. >>>> >>>> Further, somehow the development is deploying drivers/bus, >>>> drivers/common and drivers/pool in lib/* under the name specified as >>>> LIB in Makefile. My understanding was that it is expected behavior and >>>> not special because of drivers folder. >>>> >>>> Thus, above line only links lib/librte_common_dpaa2_qbman generated by >>>> drivers/common/dpaa2/qbman code. >>>> >>>> In fact, I think, this might also one of the issues why a parallel >>>> shared build fails for DPAA2 PMD (added in Cover letter). >>>> The dependency graph cannot create a graph for drivers/common >>>> as dependency for drivers/net or drivers/bus and hence parallel build >>>> fails because of missing libraries which are being parallely compiled. >>> >>> DEPDIRS-y is mainly to resolve dependencies for compilation order, and >>> should point to the folder, >>> >>> Following line will cause "librte_eal" to be compiled before driver: >>> DEPDIRS-$(CONFIG_RTE_LIBRTE_DPAA2_POOL) += lib/librte_eal >>> >>> So "lib/librte_common_dpaa2_qbman" does not makes more sense, since >>> there is no folder like that. >>> >>> >>> Somewhere in the history, with following commit, DEPDIRS-y gained a side >>> effect, it has been used to set dynamic linking dependencies, to fix >>> underlinking issue: >>> bf5a46fa5972 ("mk: generate internal library dependencies") >>> >>> I guess you are having that line to benefit from this side effect, but >>> this can be done with following more properly: >>> LDLIBS += lib/librte_common_dpaa2_qbman >>> >>> >>> To resolve the drivers/net to drivers/common dependency, following line >>> in this Makefile should work: >>> DEPDIRS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD) += drivers/common/dpaa2 >>> >>> This adds following, which will cause "drivers/common" compiled before >>> any "drivers/net": >>> LOCAL_DEPDIRS-drivers/net += drivers/common >> >> Thanks for your suggestion. This is one thing, I am not yet able to fix. >> >> Based on your suggestions: >> e.g. >> LDLIBS += -lrte_common_dpaa2_qbman >> DEPDIRS-$(CONFIG_RTE_LIBRTE_FSLMC_BUS) += drivers/common/dpaa2 >> >> It does add entry in the ".depdirs" >> ./arm64-dpaa2-linuxapp-gcc/.depdirs:168:LOCAL_DEPDIRS-drivers/bus += >> drivers/common >> ./arm64-dpaa2-linuxapp-gcc/.depdirs:170:LOCAL_DEPDIRS-drivers += lib >> ./arm64-dpaa2-linuxapp-gcc/.depdirs:172:LOCAL_DEPDIRS-drivers += lib >> ./arm64-dpaa2-linuxapp-gcc/.depdirs:174:LOCAL_DEPDIRS-drivers/pool += >> drivers/common >> >> However, we keep on getting: >> LD librte_bus_fslmc.so.1.1 >> aarch64-linux-gnu-gcc: error: drivers/common/dpaa2: No such file or >> directory >> make[6]: *** [librte_bus_fslmc.so.1.1] Error 1 > > Probably because of: > > # Translate DEPDIRS-y into LDLIBS > # Ignore (sub)directory dependencies which do not provide an actual library > _IGNORE_DIRS = lib/librte_eal/% lib/librte_compat > _DEPDIRS = $(filter-out $(_IGNORE_DIRS),$(DEPDIRS-y)) > _LDDIRS = $(subst librte_ether,librte_ethdev,$(_DEPDIRS)) > LDLIBS += $(subst lib/lib,-l,$(_LDDIRS)) > > It shows one important thing: qbman is not a driver, it is a lib. > So drivers/common/dpaa2 should be handled differently. > > Solution 1: tweak mk/rte.lib.mk for directories in drivers/common/ > Solution 2: host your bus libs outside of DPDK > For solution 1, following [1] seems working, "drivers/%" preferred against "drivers/common/%" because "drivers/bus/%" has same issue. And as far as I can see dpaa2 is the only driver dependency in lib folder. [1] diff --git a/mk/rte.lib.mk b/mk/rte.lib.mk index 33a5f5a..ac4df9a 100644 --- a/mk/rte.lib.mk +++ b/mk/rte.lib.mk @@ -79,7 +79,7 @@ endif # Translate DEPDIRS-y into LDLIBS # Ignore (sub)directory dependencies which do not provide an actual library -_IGNORE_DIRS = lib/librte_eal/% lib/librte_compat +_IGNORE_DIRS = lib/librte_eal/% lib/librte_compat drivers/% _DEPDIRS = $(filter-out $(_IGNORE_DIRS),$(DEPDIRS-y)) _LDDIRS = $(subst librte_ether,librte_ethdev,$(_DEPDIRS)) LDLIBS += $(subst lib/lib,-l,$(_LDDIRS))