From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f169.google.com (mail-wi0-f169.google.com [209.85.212.169]) by dpdk.org (Postfix) with ESMTP id 71AD9C536 for ; Sun, 28 Jun 2015 22:25:51 +0200 (CEST) Received: by wiwl6 with SMTP id l6so83048577wiw.0 for ; Sun, 28 Jun 2015 13:25:51 -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=VQ2XNiSbirRNvwj8WKY2n9vk+eixUBeLpZYmTG1WvwY=; b=VhHHoJG4QNX3GmgnW/CvBBum+HIoNKn4hjWtBgO5IMx/XCJyFEnTWICqGxAkucDrOg ldmzgpyQfo+RiZHQ1WVaE/Kp8WCpTr9HLKeQ4K68OQCIzcV2HxOgp1nSOm6B0mv/rxcs XVz/cfYSz9xqqg1TPJcZvQi+luLUyxl4LRthdWh2GNV9oIzxR7zYJY0iDVeDNmrglj9Q ALQwhwG+prkWVNS55zDXxPQRVbhLvDbhajEt+WGng0su2qs/DDthTsAjG2S67IlERMC+ iCIQ000l75V8lY63OPwHnsuoqbnK8bPAzsQ71u0NGWISA4yELhlah6PLwz+ezTwoio4h x67g== X-Gm-Message-State: ALoCoQmPcKln6ezckbh90QjW+M0acSR/ee4b/Ef/6uAcDrSFCr3SP97Q/oYiOSWBSygcx/iNxnbr X-Received: by 10.194.6.37 with SMTP id x5mr24287035wjx.73.1435523151165; Sun, 28 Jun 2015 13:25:51 -0700 (PDT) Received: from xps13.localnet (136-92-190-109.dsl.ovh.fr. [109.190.92.136]) by mx.google.com with ESMTPSA id di7sm8818867wib.23.2015.06.28.13.25.49 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 28 Jun 2015 13:25:50 -0700 (PDT) From: Thomas Monjalon To: Neil Horman Date: Sun, 28 Jun 2015 22:24:42 +0200 Message-ID: <1543730.acTPx5xhcA@xps13> Organization: 6WIND User-Agent: KMail/4.14.8 (Linux/4.0.4-2-ARCH; KDE/4.14.8; x86_64; ; ) In-Reply-To: <20150626145434.GB27458@hmsreliant.think-freely.org> References: <1435088014-18973-1-git-send-email-nhorman@tuxdriver.com> <2425730.LWzQFRnlUA@xps13> <20150626145434.GB27458@hmsreliant.think-freely.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCHv3 3/3] ABI: Add some documentation 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: Sun, 28 Jun 2015 20:25:51 -0000 2015-06-26 10:54, Neil Horman: > On Fri, Jun 26, 2015 at 03:00:17PM +0200, Thomas Monjalon wrote: > > 2015-06-25 10:35, Neil Horman: > > > +* ``VERSION_SYMBOL(b, e, n)``: Creates a symbol version table entry binding > > > + unversioned symbol ``b`` to the internal function ``b_e``. > > > > Should a versioned symbol @DPDK_ > > > Sure. When fixed, this series can be applied. > > > +* ``BASE_SYMBOL(b, e)``: Creates a symbol version table entry binding > > > + unversioned symbol ``b`` to the internal function ``b_e``. > > > > Please give a use case of BASE_SYMBOL. > > > No, I'd rather remove it if you really insist. As noted before the way we set > up the version map files means we currently have no need for this particular > directive. I only included it for completeness. I think an example use in > light of that fact would only confuse people. If you're going to draw a line in > the sand around it, I'll just remove it. No line in the sand, but it seems better to remove it to avoid confusing people. ABI compat is already enough difficult to understand ;) > > [...] > > > +The addition of the new block tells the linker that a new version node is > > > +available (DPDK_2.1), which contains the symbol rte_acl_create, and inherits the > > > +symbols from the DPDK_2.0 node. > > > > which contains the old version of the symbol rte_acl_create > > > I don't understand, is it not obvious that the DPDK_2.0 node contains the 2.0 > version of the symbol and the DPDK_2.1 node contains the 2.1 version of the > symbol? Yes it is. I thought it was needed to insist on the existence of a new symbol while the old one still exists. Just trying to be didactic. Hope this documentation will help to unlock the patch series currently breaking the ABI.