From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] app/testpmd: fix static build link ordering Date: Fri, 13 Jan 2017 17:31:58 +0100 Message-ID: <5998641.OcUmbfPz3r@xps13> References: <1484207214-13638-1-git-send-email-jerin.jacob@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, ferruh.yigit@intel.com, stable@dpdk.org To: Jerin Jacob Return-path: Received: from mail-wm0-f41.google.com (mail-wm0-f41.google.com [74.125.82.41]) by dpdk.org (Postfix) with ESMTP id DEFE96CD4 for ; Fri, 13 Jan 2017 17:32:00 +0100 (CET) Received: by mail-wm0-f41.google.com with SMTP id n129so13237283wmn.0 for ; Fri, 13 Jan 2017 08:32:00 -0800 (PST) In-Reply-To: <1484207214-13638-1-git-send-email-jerin.jacob@caviumnetworks.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2017-01-12 13:16, Jerin Jacob: > By introducing explicit -lrte_pmd_ixgbe link request in > testpmd Makefile,"-Wl,-lrte_pmd_ixgbe" provided twice, and linker > removes the duplication by keeping only first occurrence. > This moves "-Wl,-lrte_pmd_ixgbe" out of "-Wl,--whole-archive" flag > and makes symbol generation totally different than previous version > in case of static build. > This patch fixes the static build linking order by introducing > -lrte_pmd_ixgbe under the shared library config > (CONFIG_RTE_BUILD_SHARED_LIB). > > Fixes: 425781ff5afe ("app/testpmd: add ixgbe VF management") > > CC: stable@dpdk.org > Signed-off-by: Jerin Jacob Applied, thanks