From patchwork Thu Jun 25 14:35:48 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Neil Horman X-Patchwork-Id: 5792 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 83DA1C68C; Thu, 25 Jun 2015 16:36:16 +0200 (CEST) Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id 8F4C4C604 for ; Thu, 25 Jun 2015 16:36:15 +0200 (CEST) Received: from hmsreliant.think-freely.org ([2001:470:8:a08:7aac:c0ff:fec2:933b] helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63) (envelope-from ) id 1Z88GB-0001gy-31; Thu, 25 Jun 2015 10:36:14 -0400 From: Neil Horman To: dev@dpdk.org Date: Thu, 25 Jun 2015 10:35:48 -0400 Message-Id: <1435242949-31520-2-git-send-email-nhorman@tuxdriver.com> X-Mailer: git-send-email 2.1.0 In-Reply-To: <1435242949-31520-1-git-send-email-nhorman@tuxdriver.com> References: <1435088014-18973-1-git-send-email-nhorman@tuxdriver.com> <1435242949-31520-1-git-send-email-nhorman@tuxdriver.com> X-Spam-Score: -2.9 (--) X-Spam-Status: No Subject: [dpdk-dev] [PATCHv3 2/3] rte_compat: Add MAP_STATIC_SYMBOL macro 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" It was pointed out in my examples that doing shared library symbol versioning by partitioning symbols to version specific functions (as opposed to leaving the latest symol version at the base symbol name), neglects to take into account static builds. Add a macro to handle that. If you choose a versioning approach that uniquely names every version of the symbol, then this macro lets you map your symbol choice to the base name when building a static library Also, while I'm at it, since we're documenting this in the guide, take the abbreviated example out of the header Signed-off-by: Neil Horman CC: thomas.monjalon@6wind.com Acked-by: Maciej Gajdzica --- lib/librte_compat/rte_compat.h | 35 ++++++++++++++++++----------------- 1 file changed, 18 insertions(+), 17 deletions(-) diff --git a/lib/librte_compat/rte_compat.h b/lib/librte_compat/rte_compat.h index 75920a1..d7768d5 100644 --- a/lib/librte_compat/rte_compat.h +++ b/lib/librte_compat/rte_compat.h @@ -49,22 +49,8 @@ * Assumptions: DPDK 2.(X) contains a function int foo(char *string) * DPDK 2.(X+1) needs to change foo to be int foo(int index) * - * To accomplish this: - * 1) Edit lib//library_version.map to add a DPDK_2.(X+1) node, in which - * foo is exported as a global symbol. - * - * 2) rename the existing function int foo(char *string) to - * int foo_v20(char *string) - * - * 3) Add this macro immediately below the function - * VERSION_SYMBOL(foo, _v20, 2.0); - * - * 4) Implement a new version of foo. - * char foo(int value, int otherval) { ...} - * - * 5) Mark the newest version as the default version - * BIND_DEFAULT_SYMBOL(foo, _v21, 2.1); - * + * Refer to the guidelines document in the docs subdirectory for details on the + * use of these macros */ /* @@ -72,6 +58,8 @@ * b - function base name * e - function version extension, to be concatenated with base name * n - function symbol version string to be applied + * f - function prototype + * p - full function symbol name */ /* @@ -96,6 +84,19 @@ #define BIND_DEFAULT_SYMBOL(b, e, n) __asm__(".symver " RTE_STR(b) RTE_STR(e) ", " RTE_STR(b) "@@DPDK_" RTE_STR(n)) #define __vsym __attribute__((used)) +/* + * MAP_STATIC_SYMBOL + * If a function has been bifurcated into multiple versions, none of which + * are defined as the exported symbol name in the map file, this macro can be + * used to alias a specific version of the symbol to its exported name. For + * example, if you have 2 versions of a function foo_v1 and foo_v2, where the + * former is mapped to foo@DPDK_1 and the latter is mapped to foo@DPDK_2 when + * building a shared library, this macro can be used to map either foo_v1 or + * foo_v2 to the symbol foo when building a static library, e.g.: + * MAP_STATIC_SYMBOL(void foo(), foo_v2); + */ +#define MAP_STATIC_SYMBOL(f, p) + #else /* * No symbol versioning in use @@ -104,7 +105,7 @@ #define __vsym #define BASE_SYMBOL(b, n) #define BIND_DEFAULT_SYMBOL(b, e, n) - +#define MAP_STATIC_SYMBOL(f, p) f __attribute__((alias( RTE_STR(p)))) /* * RTE_BUILD_SHARED_LIB=n */