unofficial mirror of libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Szabolcs Nagy via Libc-alpha <libc-alpha@sourceware.org>
To: libc-alpha@sourceware.org
Subject: [PATCH v4 10/10] Make libc symbols hidden in static PIE
Date: Mon, 18 Jan 2021 16:26:07 +0000	[thread overview]
Message-ID: <4b939181155fc70ccb708530c1244eb0c2fafbef.1610986541.git.szabolcs.nagy@arm.com> (raw)
In-Reply-To: <cover.1610986541.git.szabolcs.nagy@arm.com>

Hidden matters with static PIE: extern symbol access in position
independent code usually involves GOT indirections which needs
RELATIVE relocs in a static linked PIE. Hidden visibility avoids
indirections and RELATIVE relocs on targets that can access symbols
pc-relative.

The check should use IS_IN_LIB instead of IS_IN(libc) since all
static libraries can use hidden visibility to avoid indirections,
however the test system links objects from libcrypt.a into dynamic
linked test binaries so hidden does not work there.  I think mixing
static and shared libc components in the same binary should not be
supported usage, but to be safe only use hidden in libc.a.

This is an optimization. But on some targets (i386) it cannot be
applied because hidden visibility PIE ifunc functions don't work,
so it is gated by NO_HIDDEN_EXTERN_FUNC_IN_PIE.

From -static-pie linked 'int main(){}' this shaves off 71 relative
relocs on aarch64 and reduces code size by about 2k.
---
 include/libc-symbols.h | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/include/libc-symbols.h b/include/libc-symbols.h
index ea126ae70c..f4dd735555 100644
--- a/include/libc-symbols.h
+++ b/include/libc-symbols.h
@@ -434,13 +434,18 @@ for linking")
   strong_alias(real, name)
 #endif
 
-#if defined SHARED || defined LIBC_NONSHARED \
-  || (BUILD_PIE_DEFAULT && IS_IN (libc))
+#if defined SHARED || defined LIBC_NONSHARED
 # define attribute_hidden __attribute__ ((visibility ("hidden")))
 #else
 # define attribute_hidden
 #endif
 
+/* Mark all symbols hidden in static PIE libc to avoid GOT indirections.  */
+#if BUILD_PIE_DEFAULT && !defined NO_HIDDEN_EXTERN_FUNC_IN_PIE \
+    && IS_IN (libc) && !defined LIBC_NONSHARED
+# pragma GCC visibility push(hidden)
+#endif
+
 #define attribute_tls_model_ie __attribute__ ((tls_model ("initial-exec")))
 
 #define attribute_relro __attribute__ ((section (".data.rel.ro")))
-- 
2.17.1


  parent reply	other threads:[~2021-01-18 16:26 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-18 16:22 [PATCH v4 00/10] fix ifunc with static pie [BZ #27072] Szabolcs Nagy via Libc-alpha
2021-01-18 16:23 ` [PATCH v4 01/10] configure: Require PI_STATIC_AND_HIDDEN for static pie Szabolcs Nagy via Libc-alpha
2021-01-18 16:23 ` [PATCH v4 02/10] libmvec: Add extra-test-objs to test-extras Szabolcs Nagy via Libc-alpha
2021-01-18 20:04   ` Adhemerval Zanella via Libc-alpha
2021-01-18 16:23 ` [PATCH v4 03/10] elf: Make the tunable struct definition internal only Szabolcs Nagy via Libc-alpha
2021-01-18 16:24 ` [PATCH v4 04/10] elf: Avoid RELATIVE relocs in __tunables_init Szabolcs Nagy via Libc-alpha
2021-01-18 16:24 ` [PATCH v4 05/10] Use hidden visibility for early static PIE code Szabolcs Nagy via Libc-alpha
2021-01-18 21:49   ` Adhemerval Zanella via Libc-alpha
2021-01-18 16:24 ` [PATCH v4 06/10] elf: Avoid RELATIVE relocation for _dl_sysinfo Szabolcs Nagy via Libc-alpha
2021-01-19 13:51   ` Adhemerval Zanella via Libc-alpha
2021-01-19 14:25     ` V2 " H.J. Lu via Libc-alpha
2021-01-19 14:35       ` Adhemerval Zanella via Libc-alpha
2021-01-18 16:25 ` [PATCH v4 07/10] Use <startup.h> in __libc_init_secure Szabolcs Nagy via Libc-alpha
2021-01-19 13:56   ` Adhemerval Zanella via Libc-alpha
2021-01-18 16:25 ` [PATCH v4 08/10] csu: Move static pie self relocation later [BZ #27072] Szabolcs Nagy via Libc-alpha
2021-01-19 14:07   ` Adhemerval Zanella via Libc-alpha
2021-01-19 14:35     ` Szabolcs Nagy via Libc-alpha
2021-01-19 14:36       ` Adhemerval Zanella via Libc-alpha
2021-01-19 14:48         ` H.J. Lu via Libc-alpha
2021-01-19 15:24           ` Szabolcs Nagy via Libc-alpha
2021-01-19 15:32             ` H.J. Lu via Libc-alpha
2021-01-19 16:47               ` H.J. Lu via Libc-alpha
2021-01-19 17:03                 ` Szabolcs Nagy via Libc-alpha
2021-01-19 17:10                   ` H.J. Lu via Libc-alpha
2021-01-19 17:25                     ` Fāng-ruì Sòng via Libc-alpha
2021-01-19 17:33                       ` H.J. Lu via Libc-alpha
2021-01-19 17:38                         ` Fāng-ruì Sòng via Libc-alpha
2021-01-19 17:38                     ` Szabolcs Nagy via Libc-alpha
2021-01-19 17:42                       ` H.J. Lu via Libc-alpha
2021-01-19 17:47                         ` Szabolcs Nagy via Libc-alpha
2021-01-19 17:53                           ` H.J. Lu via Libc-alpha
2021-01-19 17:59                             ` H.J. Lu via Libc-alpha
2021-01-18 16:25 ` [PATCH v4 09/10] x86: Check ifunc resolver with CPU_FEATURE_USABLE " Szabolcs Nagy via Libc-alpha
2021-01-19 14:11   ` Adhemerval Zanella via Libc-alpha
2021-01-19 14:37     ` V2 " H.J. Lu via Libc-alpha
2021-01-21 16:33       ` H.J. Lu via Libc-alpha
2021-01-18 16:26 ` Szabolcs Nagy via Libc-alpha [this message]
2021-01-18 21:37 ` [PATCH v4 00/10] fix ifunc with static pie " Adhemerval Zanella via Libc-alpha
2021-01-19 18:25   ` Szabolcs Nagy via Libc-alpha
2021-01-19 19:41     ` H.J. Lu via Libc-alpha
2021-01-19 20:16       ` Adhemerval Zanella via Libc-alpha
2021-01-19 21:38         ` H.J. Lu via Libc-alpha
2021-01-20 11:29           ` Adhemerval Zanella via Libc-alpha
2021-01-20 12:38             ` Szabolcs Nagy via Libc-alpha
2021-01-20 12:49               ` H.J. Lu via Libc-alpha

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/libc/involved.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4b939181155fc70ccb708530c1244eb0c2fafbef.1610986541.git.szabolcs.nagy@arm.com \
    --to=libc-alpha@sourceware.org \
    --cc=szabolcs.nagy@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).