From mboxrd@z Thu Jan 1 00:00:00 1970 From: Panu Matilainen Subject: Re: [PATCH] mk: fix the combined library problems by replacing it with a linker script Date: Thu, 26 Nov 2015 10:05:19 +0200 Message-ID: <5656BD3F.2020507@redhat.com> References: <079fa1cfc3550c8147ea8b137fa1bc0f34d051dc.1448375477.git.pmatilai@redhat.com> <20151124144638.0b659e7d@xeon-e3> <56557398.3020600@redhat.com> <20151125080837.68e24a2d@xeon-e3> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Stephen Hemminger Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id F3EF69A8E for ; Thu, 26 Nov 2015 09:05:21 +0100 (CET) In-Reply-To: <20151125080837.68e24a2d@xeon-e3> 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" On 11/25/2015 06:08 PM, Stephen Hemminger wrote: > On Wed, 25 Nov 2015 10:38:48 +0200 > Panu Matilainen wrote: > >> On 11/25/2015 12:46 AM, Stephen Hemminger wrote: >>> On Tue, 24 Nov 2015 16:31:17 +0200 >>> Panu Matilainen wrote: >>> >>>> 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 >>> >>> But it now means distros have to ship 20 libraries which seems like >>> a step back. >> >> That's how Fedora and RHEL are shipping it already and nobody has so >> much as noticed anything strange, much less complained about it. 20 >> libraries is but a drop in the ocean on a average distro. But more to >> the point, distros will prefer 50 working libraries over one that doesn't. >> >> The combined library as it is simply is no longer a viable option. >> Besides just being broken (witness the strange hacks people are coming >> up with to work around issues in it) its ugly because it basically gives >> the middle finger to all the effort going into version compatibility, >> and its also big. Few projects will use every library in DPDK, but with >> the combined library they're forced to lug the 800 pound gorilla along >> needlessly. >> >> - Panu - >> > > Fixing the combined library took less than an hour for us. > An hour (and many more in times to come) that I bet you could've used on something more interesting than fighting that abomination. - Panu -