From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f180.google.com (mail-wi0-f180.google.com [209.85.212.180]) by dpdk.org (Postfix) with ESMTP id B265958EF for ; Wed, 18 Mar 2015 13:59:46 +0100 (CET) Received: by wixw10 with SMTP id w10so39133311wix.0 for ; Wed, 18 Mar 2015 05:59:46 -0700 (PDT) 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:content-type; bh=9d/aYs11QrXL2CyzPeI7gAQHdayHw0qM6TaGEshmmRw=; b=QChj1gjAqFeD5+sSk88LkMu1DC2m3fjKDRaNjNdg52l/4yxF9OMvQOKFlByPxyDmOn a47Cpjng0SWFBXtCl3TZ53MybdM6sb6dm0EVf6buzS28Ir0eQKMfOQUHiVt9mFbXD9t4 IorjqlWXHft0fA9uDrW48kRPltEopNTJ2ShLTpKHLloPPpXRP2KBshIT9jKNkn+H0Ze2 GH/CaB/aLOg/HlGG6sbem22UZO6fE9balgY9XN/Pb04XVCSThiCRNr26c0lVRJRJqTpM GFWs8loIbU5kLWrIP06IipnDuBu+c0ACx7SzSmCHrTKQTcenpm4J966CVEvA25TZyXIg c/WQ== X-Gm-Message-State: ALoCoQmBf5h/IzIMGptOzwtWbKDNio8jd2vpGCYUCdPpoHHG/FgZzIz6VLgkHI34eUsn3FbaKKvo X-Received: by 10.180.24.162 with SMTP id v2mr6507276wif.80.1426683586595; Wed, 18 Mar 2015 05:59:46 -0700 (PDT) Received: from xps13.localnet (136-92-190-109.dsl.ovh.fr. [109.190.92.136]) by mx.google.com with ESMTPSA id k6sm3115868wia.6.2015.03.18.05.59.44 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 18 Mar 2015 05:59:45 -0700 (PDT) From: Thomas Monjalon To: "Gonzalez Monroy, Sergio" Date: Wed, 18 Mar 2015 13:59:10 +0100 Message-ID: <40763037.Qv1hPgBHv0@xps13> Organization: 6WIND User-Agent: KMail/4.14.4 (Linux/3.18.4-1-ARCH; KDE/4.14.4; x86_64; ; ) In-Reply-To: <55096B86.7040303@intel.com> References: <1422544811-26385-1-git-send-email-sergio.gonzalez.monroy@intel.com> <1426177681-16931-2-git-send-email-sergio.gonzalez.monroy@intel.com> <55096B86.7040303@intel.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 1/4] mk: Remove combined library and related options 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, 18 Mar 2015 12:59:46 -0000 Hi Sergio, Thank you for explaining the situation. 2015-03-18 12:11, Gonzalez Monroy, Sergio: > Given that the patch to remove combined libraries is not welcome, I'll > try to explain the current situation so we can agree on the way forward. > > Currently we have build config option for shared libraries and combined > libraries. Thus, this results in four possible combinations when > building dpdk: > - not combined static > - not combined shared > - combined static > - combined shared > > The makefile rules/targets for combined are different than for not > combined. Thus, we currently have two different files for > archive/linking (rte.lib.mk and rte.sharelib.mk). > > Since having versioning, combined shared libraries build will be broken > the moment we add a versioned API, as we do not have a global version > map that we use when linking such library. > Also in my opinion, we would want to prevent users linking against a > combined libdpdk.so that may have different features built-in, with the > corresponding debugging difficulties when users > report different problems/errors. I think this would defeat many of the > advantages of using shared libraries. > > By removing the combined library build option, we would simplify the > build system with only two possible choices: > - static > - shared +1 I believe that simplification is the way go. > This would allow us to remove one file (rte.sharelib.mk) and have a > single file with archive/linking rules. > > For the convenience of linking against a single library instead of the > multiple dpdk libraries, there are a few ways to go around it: > - for combined static lib, we can either have a script to re-archive > all libraries into a single/combined library (ie. extract all archives > into one directory, the re-archive all objects into a combined library), > or use a linker script (ie. GROUP ( -lrte_eal -lrte_malloc ... ) ). > - for combined shared lib, we can use a linker script (ie. INPUT ( > -lrte_eal -lrte_malloc ... AS_NEEDED -lrte_hash ...) ) or we could use a > global version map (either somehow merging all independent version maps > or maintaining a global version map). > > My preference would be to remove the combined libs as a build config > option, then either add scripts to create those linker scripts or > document it so users know how to create their own linker scripts. > This would simplify the build process and still be able to provide the > convenience of the combined library by using a linker script. > > Comments? You're right about the word convenience. There are many ways to provide such convenience. The first one is to simply use the DPDK makefiles which abstract linking problems. If using DPDK framework is not an option, we can add new conveniences like scripts or pkgconfig support.