public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Szabolcs Nagy <szabolcs.nagy@arm.com>
To: libc-alpha@sourceware.org
Subject: [PATCH v7 10/14] aarch64: configure check for pac-ret code generation
Date: Wed,  8 Jul 2020 13:13:19 +0100	[thread overview]
Message-ID: <edd17bfbbbb6a7bfbd4150f71153f4c477a2fcf5.1594209990.git.szabolcs.nagy@arm.com> (raw)
In-Reply-To: <cover.1594209990.git.szabolcs.nagy@arm.com>

Return address signing requires unwinder support, which is
present in libgcc since >=gcc-7, however due to bugs the
support may be broken in <gcc-10 (and similarly there may
be issues in custom unwinders), so pac-ret is not always
safe to use. So in assembly code glibc should only use
pac-ret if the compiler uses it too. Unfortunately there
is no predefined feature macro for it set by the compiler
so pac-ret is inferred from the code generation.

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
---
 config.h.in                  |  3 +++
 sysdeps/aarch64/configure    | 39 ++++++++++++++++++++++++++++++++++++
 sysdeps/aarch64/configure.ac | 21 +++++++++++++++++++
 3 files changed, 63 insertions(+)

diff --git a/config.h.in b/config.h.in
index 67169e5d01..7921917ad2 100644
--- a/config.h.in
+++ b/config.h.in
@@ -112,6 +112,9 @@
 /* AArch64 BTI support enabled.  */
 #define HAVE_AARCH64_BTI 0
 
+/* AArch64 PAC-RET code generation is enabled.  */
+#define HAVE_AARCH64_PAC_RET 0
+
 /* C-SKY ABI version.  */
 #undef CSKYABI
 
diff --git a/sysdeps/aarch64/configure b/sysdeps/aarch64/configure
index c637540436..ac3cf6fd36 100644
--- a/sysdeps/aarch64/configure
+++ b/sysdeps/aarch64/configure
@@ -216,3 +216,42 @@ if test $libc_cv_aarch64_bti = yes; then
   $as_echo "#define HAVE_AARCH64_BTI 1" >>confdefs.h
 
 fi
+
+# Check if glibc is built with return address signing, i.e.
+# if -mbranch-protection=pac-ret is on. We need this because
+# pac-ret relies on unwinder support so it's not safe to use
+# it in assembly code unconditionally, but there is no
+# feature test macro for it in gcc.
+{ $as_echo "$as_me:${as_lineno-$LINENO}: checking if pac-ret is enabled" >&5
+$as_echo_n "checking if pac-ret is enabled... " >&6; }
+if ${libc_cv_aarch64_pac_ret+:} false; then :
+  $as_echo_n "(cached) " >&6
+else
+    cat > conftest.c <<EOF
+int bar (void);
+int foo (void) { return bar () + 1; }
+EOF
+  libc_cv_aarch64_pac_ret=no
+  if { ac_try='${CC-cc} $CFLAGS $CPPFLAGS -S -o conftest.s conftest.c'
+  { { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5
+  (eval $ac_try) 2>&5
+  ac_status=$?
+  $as_echo "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
+  test $ac_status = 0; }; } \
+     && { ac_try='grep -q -E '\''(hint( |	)+25|paciasp)'\'' conftest.s'
+  { { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5
+  (eval $ac_try) 2>&5
+  ac_status=$?
+  $as_echo "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
+  test $ac_status = 0; }; }
+  then
+    libc_cv_aarch64_pac_ret=yes
+  fi
+  rm -rf conftest.*
+fi
+{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $libc_cv_aarch64_pac_ret" >&5
+$as_echo "$libc_cv_aarch64_pac_ret" >&6; }
+if test $libc_cv_aarch64_pac_ret = yes; then
+  $as_echo "#define HAVE_AARCH64_PAC_RET 1" >>confdefs.h
+
+fi
diff --git a/sysdeps/aarch64/configure.ac b/sysdeps/aarch64/configure.ac
index 2c2817514d..8b042d6d05 100644
--- a/sysdeps/aarch64/configure.ac
+++ b/sysdeps/aarch64/configure.ac
@@ -40,3 +40,24 @@ LIBC_CONFIG_VAR([aarch64-bti], [$libc_cv_aarch64_bti])
 if test $libc_cv_aarch64_bti = yes; then
   AC_DEFINE(HAVE_AARCH64_BTI)
 fi
+
+# Check if glibc is built with return address signing, i.e.
+# if -mbranch-protection=pac-ret is on. We need this because
+# pac-ret relies on unwinder support so it's not safe to use
+# it in assembly code unconditionally, but there is no
+# feature test macro for it in gcc.
+AC_CACHE_CHECK([if pac-ret is enabled], [libc_cv_aarch64_pac_ret], [dnl
+  cat > conftest.c <<EOF
+int bar (void);
+int foo (void) { return bar () + 1; }
+EOF
+  libc_cv_aarch64_pac_ret=no
+  if AC_TRY_COMMAND([${CC-cc} $CFLAGS $CPPFLAGS -S -o conftest.s conftest.c]) \
+     && AC_TRY_COMMAND([grep -q -E '\''(hint( |	)+25|paciasp)'\'' conftest.s])
+  then
+    libc_cv_aarch64_pac_ret=yes
+  fi
+  rm -rf conftest.*])
+if test $libc_cv_aarch64_pac_ret = yes; then
+  AC_DEFINE(HAVE_AARCH64_PAC_RET)
+fi
-- 
2.17.1


  parent reply	other threads:[~2020-07-08 12:13 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-08 12:10 [PATCH v7 00/14] aarch64: branch protection support Szabolcs Nagy
2020-07-08 12:10 ` [PATCH v7 01/14] Rewrite abi-note.S in C Szabolcs Nagy
2020-07-08 12:15   ` Florian Weimer
2020-07-08 12:10 ` [PATCH v7 02/14] aarch64: configure test for BTI support Szabolcs Nagy
2020-07-08 12:11 ` [PATCH v7 03/14] aarch64: Rename place holder .S files to .c Szabolcs Nagy
2020-07-08 12:11 ` [PATCH v7 04/14] aarch64: Add BTI support to assembly files Szabolcs Nagy
2020-07-08 12:11 ` [PATCH v7 05/14] aarch64: fix swapcontext for BTI Szabolcs Nagy
2020-07-08 12:12 ` [PATCH v7 06/14] aarch64: fix RTLD_START " Szabolcs Nagy
2020-07-08 12:12 ` [PATCH v7 07/14] rtld: Clean up PT_NOTE and add PT_GNU_PROPERTY handling Szabolcs Nagy
2020-07-08 13:23   ` H.J. Lu
2020-07-08 13:29     ` Szabolcs Nagy
2020-07-08 12:12 ` [PATCH v7 08/14] aarch64: enable BTI at runtime Szabolcs Nagy
2020-07-08 12:13 ` [PATCH v7 09/14] aarch64: ensure objects are BTI compatible Szabolcs Nagy
2021-09-20  8:58   ` Andreas Schwab
2021-09-20  9:43     ` Andreas Schwab
2021-09-20 10:49       ` Szabolcs Nagy
2021-09-20 10:58         ` Florian Weimer
2021-09-20 11:06         ` Florian Weimer
2021-09-20 11:34           ` Andreas Schwab
2021-09-20 11:40             ` Florian Weimer
2021-09-20 11:55               ` Andreas Schwab
2021-09-20 12:41                 ` Szabolcs Nagy
2020-07-08 12:13 ` Szabolcs Nagy [this message]
2020-07-08 12:13 ` [PATCH v7 11/14] aarch64: Add pac-ret support to assembly files Szabolcs Nagy
2020-07-08 12:13 ` [PATCH v7 12/14] aarch64: fix pac-ret support in _mcount Szabolcs Nagy
2020-07-08 12:14 ` [PATCH v7 13/14] aarch64: redefine RETURN_ADDRESS to strip PAC Szabolcs Nagy
2020-07-08 12:14 ` [PATCH v7 14/14] aarch64: add NEWS entry about branch protection support Szabolcs Nagy
2020-07-08 13:48   ` Adhemerval Zanella
2020-07-08 14:03     ` Szabolcs Nagy
2020-07-24  7:19   ` Florian Weimer
2020-07-24  7:50     ` Szabolcs Nagy
2020-07-24  8:06       ` Florian Weimer

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

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

  git send-email \
    --in-reply-to=edd17bfbbbb6a7bfbd4150f71153f4c477a2fcf5.1594209990.git.szabolcs.nagy@arm.com \
    --to=szabolcs.nagy@arm.com \
    --cc=libc-alpha@sourceware.org \
    /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).