From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] mk: fix misnamed variable in rte.app.mk Date: Fri, 23 May 2014 18:10:49 +0200 Message-ID: <1633454.Txn6nOcm6F@xps13> References: <1400855964-20482-1-git-send-email-bruce.richardson@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Bruce Richardson Return-path: In-Reply-To: <1400855964-20482-1-git-send-email-bruce.richardson-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 2014-05-23 15:39, Bruce Richardson: > The variable CONFIG_RTE_BUILD_SHARED_LIB was in rte.app.mk as > "RTE_BUILD_SHARED_LIB", which meant that none of the example apps linked > in the PMDs and just didn't work with any eth ports in any static builds > > Link for l2fwd before patch: > "... -Wl,--whole-archive -Wl,-lrte_kni -Wl,-lrte_timer -Wl,-lrte_hash > -Wl,-lrte_lpm -Wl,-lrte_power -Wl,-lrte_meter -Wl,-lrte_sched -Wl,-lm > -Wl,-lrt -Wl,--start-group -Wl,-lrte_kvargs -Wl,-lrte_mbuf -Wl,-lethdev > -Wl,-lrte_malloc -Wl,-lrte_mempool -Wl,-lrte_ring -Wl,-lrte_eal > -Wl,-lrte_cmdline -Wl,-lrt -Wl,-lm -Wl,-ldl -Wl,--end-group > -Wl,--no-whole-archive" > > Link for l2fwd after patch: > "... -Wl,--whole-archive -Wl,-lrte_kni -Wl,-lrte_timer -Wl,-lrte_hash > -Wl,-lrte_lpm -Wl,-lrte_power -Wl,-lrte_meter -Wl,-lrte_sched -Wl,-lm > -Wl,-lrt -Wl,--start-group -Wl,-lrte_kvargs -Wl,-lrte_mbuf -Wl,-lethdev > -Wl,-lrte_malloc -Wl,-lrte_mempool -Wl,-lrte_ring -Wl,-lrte_eal > -Wl,-lrte_cmdline -Wl,-lrte_pmd_vmxnet3_uio -Wl,-lrte_pmd_virtio_uio > -Wl,-lrte_pmd_ixgbe -Wl,-lrte_pmd_e1000 -Wl,-lrte_pmd_ring -Wl,-lrt > -Wl,-lm -Wl,-ldl -Wl,--end-group -Wl,--no-whole-archive" > > Signed-off-by: Bruce Richardson This bug has been introduced in commit 3660cdf990: pcap: convert to use of PMD_REGISTER_DRIVER and fix linking Note: bug fixes should include reference to root cause in the log. Acked-by: Thomas Monjalon Applied with this title: mk: fix linking drivers in static apps -- Thomas