From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f54.google.com (mail-wm0-f54.google.com [74.125.82.54]) by dpdk.org (Postfix) with ESMTP id 9E8DA95C8 for ; Wed, 2 Mar 2016 13:42:34 +0100 (CET) Received: by mail-wm0-f54.google.com with SMTP id p65so76026901wmp.0 for ; Wed, 02 Mar 2016 04:42:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:organization:user-agent :in-reply-to:references:mime-version:content-transfer-encoding; bh=JUpMJDFglbanqLENKqLu1ayXhsZE+37UkARicGtM8+Q=; b=Tl00NJ/UlgPR4K61WdkSxxIM6EbLuBXzlJr+hUT4RMKHUTiSPNfaDGNgiETBfb11BH XoslYJeGRfD8xFHMRHN9JBAtzpivqiohiG0BrZy9co3kYPFdcmppoblbFKxQVerLKjIW ZRXVd+F5+GfA2mBYiCp/nPVlokdFC6Pv/gBv8O4s5hH7rpywDOeLYgNfg/kVYP6sc1be ksqMUU2fiZyuFXrY793cXUKWJzwEmD6x1SMJ5ORZE4YBzeo/Aah7ujdBMrfuRlrXOTC7 2YjI4UZrIai6FFej3J2RiH8VXoFIv9KzE+1ArQ2G+fqP78HeE5tVqu6KPdFJPUjxMnSJ wCPQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:organization :user-agent:in-reply-to:references:mime-version :content-transfer-encoding; bh=JUpMJDFglbanqLENKqLu1ayXhsZE+37UkARicGtM8+Q=; b=BklWJ+AGWtKjAg/yr6DGPZBnINoXbncatBn8Ai8u/CUEkbme99SkRwywpKiLTgekyC lP3ung7TpEYCSgEZGLSvQBXYzcbskoMksToNzd+uP8sKPMX4xySF4vFdwzWgmQPtQ1NR wqWit9gmslfKzFECuMuzuLnrJIv6ANu7IeDnUpZyuBlYl+1o3SBPkJ4hv1mjk2edVJDu TIQMZcmLpHNvMHiz0AhJRvRvi6cK2/1fzKAWXvO5yVNAWfEYvkMUIOG1Vt2SBsyE1sqV Bxrdjen29izQv+Hu4/BsArmswArvneP5PddFruvTHQfvXvgcbmxUZt+XL2bAwCc/U4+P i6CA== X-Gm-Message-State: AD7BkJIYP+7gaGc3CRUdBMxSYB7GrBw6k9Ns5BhLpKEiODWHms0pR1o+MbXb/CwEsAaAbc27 X-Received: by 10.194.143.82 with SMTP id sc18mr25219944wjb.14.1456922554430; Wed, 02 Mar 2016 04:42:34 -0800 (PST) Received: from xps13.localnet (guy78-3-82-239-227-177.fbx.proxad.net. [82.239.227.177]) by smtp.gmail.com with ESMTPSA id gb9sm35772764wjb.26.2016.03.02.04.42.33 (version=TLSv1/SSLv3 cipher=OTHER); Wed, 02 Mar 2016 04:42:33 -0800 (PST) From: Thomas Monjalon To: Panu Matilainen Date: Wed, 02 Mar 2016 13:40:59 +0100 Message-ID: <11773022.MfDspfNx6j@xps13> Organization: 6WIND User-Agent: KMail/4.14.10 (Linux/4.1.6-1-ARCH; KDE/4.14.11; x86_64; ; ) In-Reply-To: <56D6DCE0.3010501@redhat.com> References: <079fa1cfc3550c8147ea8b137fa1bc0f34d051dc.1448375477.git.pmatilai@redhat.com> <56D5ABB4.3010609@redhat.com> <56D6DCE0.3010501@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v2] mk: replace the combined library with a linker script X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Mar 2016 12:42:34 -0000 2016-03-02 14:30, Panu Matilainen: > On 03/01/2016 04:48 PM, Panu Matilainen wrote: > > On 03/01/2016 03:40 PM, Thomas Monjalon wrote: > >> ping > >> I would like to be sure nothing is forgotten in this new revision. > > > > Sorry, didn't realize you were waiting for input from me, it feels a bit > > strange to comment on something supposedly coming from myself :) > > > >> 2016-02-23 23:20, Thomas Monjalon: > >>> From: Panu Matilainen > >>> > >>> The physically linked-together combined library has been an increasing > >>> source of problems, as was predicted when library and symbol versioning > >>> was introduced. Replace the complex and fragile construction with a > >>> simple linker script which achieves the same without all the problems, > >>> remove the related kludges from eg mlx drivers. > >>> > >>> Since creating the linker script is practically zero cost, remove the > >>> config option and just create it always. > >>> > >>> Based on a patch by Sergio Gonzales Monroy, linker script approach > >>> initially suggested by Neil Horman. > >>> > >>> Suggested-by: Sergio Gonzalez Monroy > >>> Suggested-by: Neil Horman > >>> Signed-off-by: Panu Matilainen > >>> Signed-off-by: Thomas Monjalon > >>> --- > >>> v2: > >>> - move RTE_LIBNAME assignment rte.vars.mk to rte.combinedlib.mk > >>> - update crypto > >>> - update doc > >>> - update rte.app.mk > >>> - update test-build.sh > >> > > > > Briefly tested, gets generated and installed as it should etc - looks > > good to me. > > Forgot to note that the patch doesn't apply anymore because of > scripts/test-build.sh changes, so it needs a rebase. Want me to send a > v3 or will you handle it when committing? > > On a related note, if this is about to go in then I'd rather have it > sooner than later because it also conflicts with the LDLIBS fixing > that's been slowly going on for months and months but been on hold > lately, partly because of this hangup. Applied, thanks