From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2] mk: fix acl library static linking Date: Thu, 30 Jun 2016 18:10:20 +0200 Message-ID: <2826363.SrA66T9uvx@xps13> References: <1467285021-103920-1-git-send-email-sergio.gonzalez.monroy@intel.com> <1467302516-106285-1-git-send-email-sergio.gonzalez.monroy@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, konstantin.ananyev@intel.com To: Sergio Gonzalez Monroy Return-path: Received: from mail-wm0-f42.google.com (mail-wm0-f42.google.com [74.125.82.42]) by dpdk.org (Postfix) with ESMTP id 4509B568F for ; Thu, 30 Jun 2016 18:10:54 +0200 (CEST) Received: by mail-wm0-f42.google.com with SMTP id v199so227860711wmv.0 for ; Thu, 30 Jun 2016 09:10:54 -0700 (PDT) In-Reply-To: <1467302516-106285-1-git-send-email-sergio.gonzalez.monroy@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2016-06-30 17:01, Sergio Gonzalez Monroy: > --- a/mk/rte.app.mk > +++ b/mk/rte.app.mk > @@ -76,12 +76,13 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_IP_FRAG) += -lrte_ip_frag > _LDLIBS-$(CONFIG_RTE_LIBRTE_METER) += -lrte_meter > _LDLIBS-$(CONFIG_RTE_LIBRTE_SCHED) += -lrte_sched > _LDLIBS-$(CONFIG_RTE_LIBRTE_LPM) += -lrte_lpm > -_LDLIBS-$(CONFIG_RTE_LIBRTE_ACL) += -lrte_acl > _LDLIBS-$(CONFIG_RTE_LIBRTE_JOBSTATS) += -lrte_jobstats > _LDLIBS-$(CONFIG_RTE_LIBRTE_POWER) += -lrte_power > > _LDLIBS-y += --whole-archive > > +# librte_acl needs --whole-archive because of weak functions > +_LDLIBS-$(CONFIG_RTE_LIBRTE_ACL) += -lrte_acl > _LDLIBS-$(CONFIG_RTE_LIBRTE_TIMER) += -lrte_timer > _LDLIBS-$(CONFIG_RTE_LIBRTE_HASH) += -lrte_hash > _LDLIBS-$(CONFIG_RTE_LIBRTE_VHOST) += -lrte_vhost I was suggesting to keep -lrte_acl at the same place in the group of algorithms libraries, in order to keep an order satisfying this comment: # Order is important: from higher level to lower level But I have not tested if --whole-archive -lrte_acl --no-whole-archive works.