From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 509FFA00C2; Tue, 8 Mar 2022 15:24:39 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6043F410E1; Tue, 8 Mar 2022 15:24:35 +0100 (CET) Received: from wout2-smtp.messagingengine.com (wout2-smtp.messagingengine.com [64.147.123.25]) by mails.dpdk.org (Postfix) with ESMTP id EE9364068B; Tue, 8 Mar 2022 15:24:33 +0100 (CET) Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.west.internal (Postfix) with ESMTP id 75DB13201F73; Tue, 8 Mar 2022 09:24:32 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute5.internal (MEProxy); Tue, 08 Mar 2022 09:24:33 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:cc:content-transfer-encoding:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm1; bh=9kXPYyDZCOvkoLSXMsJXgSAEo79H8f tZ2BNte58MhIg=; b=hkP/eBDwp6fQhC/6rjro5e8xrZqOHjNB0GUcLcL/KG+m32 cO93UQwQTNSZhf9y3tL3nZCkqdYNA8+t8ecqVKkTHfhVfBcuaYxH3C9BUNy/he4D Mzz8g4VH4rJ/K51doZ44WSQ3+X9X4udToEfdRABC0NFDfHxDNZ6MvVgam8xjtfY+ OPHABL8t5jQ3nHKFNmRkJpzh8n6/35d2x1hCUom4JVrBFb/OWtUpuTlOQxfsoVx1 sfNbbiEhquxybwsbT7Cun2ZS7g677JC0XHxFsIru8qmMI5RVU4HGLaSGAXHd3h8Z vyEp0v3g0OfdFaPQ7XggrX7D3UWj7D7wbRvO1OEQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding:date:date :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm2; bh=9kXPYy DZCOvkoLSXMsJXgSAEo79H8ftZ2BNte58MhIg=; b=QAxDVNPMmmiWWkW8qdvD2n TZ8MqwWTxltmIXHcFjsytN2XvG2vs4ipZjcPrHbv+nKOqMnv/vKdSd1efTMWfxl7 iY/RcVFfxDtqyHAXqBXcbh35rB1UZ5CUwORarBJxA2jNWCmvKyFyUdvBWrjmrglG MSlP41YoZuTFY0zbaDt96SEjs/EQMgYfq+ed7r/OpuwTEFHWeAUoPMAFC9dBjhsB +xZap9xA0pxrUQndA+XyZTHzhMuudLFHVjq8FSdGNFlKvWOtQ15vb7iv9dtvYrAS rA73B852TjueLnUd9CUOsDH0B9bEJOHQfNoQD2juuPm2cuwpHZcXbDTg5KbElJ9g == X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvvddrudduiedgiedvucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvufffkffojghfggfgsedtkeertdertddtnecuhfhrohhmpefvhhhomhgr shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg ftrfgrthhtvghrnhepvdehgfeivdejgedtveehfefhteelfefgieevgfffveefjeegtdfg uedthedtgeevnecuvehluhhsthgvrhfuihiivgepudenucfrrghrrghmpehmrghilhhfrh homhepthhhohhmrghssehmohhnjhgrlhhonhdrnhgvth X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Tue, 8 Mar 2022 09:24:30 -0500 (EST) From: Thomas Monjalon To: dev@dpdk.org Cc: david.marchand@redhat.com, stable@dpdk.org, Ray Kinsella , Parav Pandit , Xueming Li , Elena Agostini , Ori Kam , Andrew Rybchenko Subject: [PATCH v2 2/2] build: hide local symbols in shared libraries Date: Tue, 8 Mar 2022 15:24:12 +0100 Message-Id: <20220308142412.2069408-3-thomas@monjalon.net> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220308142412.2069408-1-thomas@monjalon.net> References: <20220306092023.2461466-1-thomas@monjalon.net> <20220308142412.2069408-1-thomas@monjalon.net> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org The symbols which are not listed in the version script are exported by default. Adding a local section with a wildcard make non-listed functions and variables as hidden, as it should be in all version.map files. These are the changes done in the shared libraries: - DF .text Base auxiliary_add_device - DF .text Base auxiliary_dev_exists - DF .text Base auxiliary_dev_iterate - DF .text Base auxiliary_insert_device - DF .text Base auxiliary_is_ignored_device - DF .text Base auxiliary_match - DF .text Base auxiliary_on_scan - DF .text Base auxiliary_scan - DO .bss Base auxiliary_bus_logtype - DO .data Base auxiliary_bus - DO .bss Base gpu_logtype There is no impact on regexdev library. Because these local symbols were exported as non-internal in DPDK 21.11, any change in these functions would break the ABI. Exception rules are added for these experimental libraries, so the ABI check will skip them until the next ABI version. A check is added to avoid such miss in future. Fixes: 1afce3086cf4 ("bus/auxiliary: introduce auxiliary bus") Fixes: 8b8036a66e3d ("gpudev: introduce GPU device class library") Cc: stable@dpdk.org Signed-off-by: Thomas Monjalon --- devtools/check-symbol-maps.sh | 7 +++++++ devtools/libabigail.abignore | 8 ++++++++ drivers/bus/auxiliary/version.map | 2 ++ lib/gpudev/version.map | 2 ++ lib/regexdev/version.map | 2 ++ 5 files changed, 21 insertions(+) diff --git a/devtools/check-symbol-maps.sh b/devtools/check-symbol-maps.sh index 5bd290ac97..8266fdf9ea 100755 --- a/devtools/check-symbol-maps.sh +++ b/devtools/check-symbol-maps.sh @@ -53,4 +53,11 @@ if [ -n "$duplicate_symbols" ] ; then ret=1 fi +local_miss_maps=$(grep -L 'local: \*;' $@) +if [ -n "$local_miss_maps" ] ; then + echo "Found maps without local catch-all:" + echo "$local_miss_maps" + ret=1 +fi + exit $ret diff --git a/devtools/libabigail.abignore b/devtools/libabigail.abignore index 18c11c80c6..c618f20032 100644 --- a/devtools/libabigail.abignore +++ b/devtools/libabigail.abignore @@ -32,3 +32,11 @@ ; Ignore changes in common mlx5 driver, should be all internal [suppress_file] soname_regexp = ^librte_common_mlx5\. + +; Ignore visibility fix of local functions in experimental auxiliary driver +[suppress_file] + soname_regexp = ^librte_bus_auxiliary\. + +; Ignore visibility fix of local functions in experimental gpudev library +[suppress_file] + soname_regexp = ^librte_gpudev\. diff --git a/drivers/bus/auxiliary/version.map b/drivers/bus/auxiliary/version.map index a52260657c..dc993e84ff 100644 --- a/drivers/bus/auxiliary/version.map +++ b/drivers/bus/auxiliary/version.map @@ -4,4 +4,6 @@ EXPERIMENTAL { # added in 21.08 rte_auxiliary_register; rte_auxiliary_unregister; + + local: *; }; diff --git a/lib/gpudev/version.map b/lib/gpudev/version.map index b23e3fd6eb..a2c8ce5759 100644 --- a/lib/gpudev/version.map +++ b/lib/gpudev/version.map @@ -39,4 +39,6 @@ INTERNAL { rte_gpu_get_by_name; rte_gpu_notify; rte_gpu_release; + + local: *; }; diff --git a/lib/regexdev/version.map b/lib/regexdev/version.map index 988b909638..3c6e9fffa1 100644 --- a/lib/regexdev/version.map +++ b/lib/regexdev/version.map @@ -26,6 +26,8 @@ EXPERIMENTAL { rte_regexdev_xstats_get; rte_regexdev_xstats_names_get; rte_regexdev_xstats_reset; + + local: *; }; INTERNAL { -- 2.34.1