[arch-commits] Commit in compiler-rt/trunk (PKGBUILD glibc-2.27.patch)

Evangelos Foutras foutrelis at archlinux.org
Thu Sep 20 01:35:07 UTC 2018


    Date: Thursday, September 20, 2018 @ 01:35:07
  Author: foutrelis
Revision: 335049

upgpkg: compiler-rt 7.0.0-1

New upstream release.

Modified:
  compiler-rt/trunk/PKGBUILD
Deleted:
  compiler-rt/trunk/glibc-2.27.patch

------------------+
 PKGBUILD         |   17 +---
 glibc-2.27.patch |  195 -----------------------------------------------------
 2 files changed, 6 insertions(+), 206 deletions(-)

Modified: PKGBUILD
===================================================================
--- PKGBUILD	2018-09-20 00:54:09 UTC (rev 335048)
+++ PKGBUILD	2018-09-20 01:35:07 UTC (rev 335049)
@@ -2,8 +2,8 @@
 # Contributor: Jan "heftig" Steffens <jan.steffens at gmail.com>
 
 pkgname=compiler-rt
-pkgver=6.0.1
-pkgrel=2
+pkgver=7.0.0
+pkgrel=1
 pkgdesc="Compiler runtime libraries for clang"
 arch=('x86_64')
 url="https://compiler-rt.llvm.org/"
@@ -13,11 +13,9 @@
 # Build 32-bit compiler-rt libraries on x86_64 (FS#41911)
 makedepends_x86_64=('lib32-gcc-libs')
 options=('staticlibs')
-source=(https://releases.llvm.org/$pkgver/$pkgname-$pkgver.src.tar.xz{,.sig}
-        glibc-2.27.patch)
-sha256sums=('f4cd1e15e7d5cb708f9931d4844524e4904867240c306b06a4287b22ac1c99b9'
-            'SKIP'
-            'acfba6a38cf3b40dc4f2edca2f1d487e1b287db71d6a6ca9f27657c5f90f876b')
+source=(https://releases.llvm.org/$pkgver/$pkgname-$pkgver.src.tar.xz{,.sig})
+sha256sums=('bdec7fe3cf2c85f55656c07dfb0bd93ae46f2b3dd8f33ff3ad6e7586f4c670d6'
+            'SKIP')
 validpgpkeys+=('B6C8F98282B944E3B0D5C2530FC3042E345AD05D') # Hans Wennborg <hans at chromium.org>
 validpgpkeys+=('474E22316ABF4785A88C6E8EA2C794A986419D8A') # Tom Stellard <tstellar at redhat.com>
 
@@ -24,9 +22,6 @@
 prepare() {
   cd "$srcdir/$pkgname-$pkgver.src"
   mkdir build
-
-  # Fixes for glibc 2.27
-  patch -Np1 -i ../glibc-2.27.patch
 }
 
 build() {
@@ -46,7 +41,7 @@
 
   mkdir -p "$pkgdir/usr/lib/clang/$pkgver/lib"
   mv "$pkgdir"/usr/lib/{linux,clang/$pkgver/lib/}
-  mv "$pkgdir"/usr/{*.txt,lib/clang/$pkgver/}
+  mv "$pkgdir"/usr/{share/*.txt,lib/clang/$pkgver/}
 }
 
 # vim:set ts=2 sw=2 et:

Deleted: glibc-2.27.patch
===================================================================
--- glibc-2.27.patch	2018-09-20 00:54:09 UTC (rev 335048)
+++ glibc-2.27.patch	2018-09-20 01:35:07 UTC (rev 335049)
@@ -1,195 +0,0 @@
-From 80dc25a213216b31f981b80f74c94a757eb3b8c1 Mon Sep 17 00:00:00 2001
-From: Peter Wu <peter at lekensteyn.nl>
-Date: Sun, 10 Jun 2018 11:17:47 +0000
-Subject: [PATCH 1/2] [ASAN] Fix crash on i?86-linux (32-bit) against glibc
- 2.27 and later
-
-Summary:
-Running sanitized 32-bit x86 programs on glibc 2.27 crashes at startup, with:
-
-    ERROR: AddressSanitizer: SEGV on unknown address 0xf7a8a250 (pc 0xf7f807f4 bp 0xff969fc8 sp 0xff969f7c T16777215)
-    The signal is caused by a WRITE memory access.
-    #0 0xf7f807f3 in _dl_get_tls_static_info (/lib/ld-linux.so.2+0x127f3)
-    #1 0xf7a92599  (/lib/libasan.so.5+0x112599)
-    #2 0xf7a80737  (/lib/libasan.so.5+0x100737)
-    #3 0xf7f7e14f in _dl_init (/lib/ld-linux.so.2+0x1014f)
-    #4 0xf7f6eb49  (/lib/ld-linux.so.2+0xb49)
-
-The problem is that glibc changed the calling convention for the GLIBC_PRIVATE
-symbol that sanitizer uses (even when it should not, GLIBC_PRIVATE is exactly
-for symbols that can change at any time, be removed etc.), see
-https://sourceware.org/ml/libc-alpha/2017-08/msg00497.html
-
-Fixes https://github.com/google/sanitizers/issues/954
-
-Patch By: Jakub Jelinek
-
-Reviewed By: vitalybuka, Lekensteyn
-
-Differential Revison: https://reviews.llvm.org/D44623
-
-git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@334363 91177308-0d34-0410-b5e6-96231b3b80d8
----
- .../sanitizer_linux_libcdep.cc                | 45 ++++++++++++++++---
- 1 file changed, 38 insertions(+), 7 deletions(-)
-
-diff --git a/lib/sanitizer_common/sanitizer_linux_libcdep.cc b/lib/sanitizer_common/sanitizer_linux_libcdep.cc
-index 56fdfc870..5d543622f 100644
---- a/lib/sanitizer_common/sanitizer_linux_libcdep.cc
-+++ b/lib/sanitizer_common/sanitizer_linux_libcdep.cc
-@@ -178,24 +178,55 @@ bool SanitizerGetThreadName(char *name, int max_len) {
- static uptr g_tls_size;
- 
- #ifdef __i386__
-+# ifndef __GLIBC_PREREQ
-+#  define CHECK_GET_TLS_STATIC_INFO_VERSION 1
-+# else
-+#  define CHECK_GET_TLS_STATIC_INFO_VERSION (!__GLIBC_PREREQ(2, 27))
-+# endif
-+#else
-+# define CHECK_GET_TLS_STATIC_INFO_VERSION 0
-+#endif
-+
-+#if CHECK_GET_TLS_STATIC_INFO_VERSION
- # define DL_INTERNAL_FUNCTION __attribute__((regparm(3), stdcall))
- #else
- # define DL_INTERNAL_FUNCTION
- #endif
- 
-+namespace {
-+struct GetTlsStaticInfoCall {
-+  typedef void (*get_tls_func)(size_t*, size_t*);
-+};
-+struct GetTlsStaticInfoRegparmCall {
-+  typedef void (*get_tls_func)(size_t*, size_t*) DL_INTERNAL_FUNCTION;
-+};
-+
-+template <typename T>
-+void CallGetTls(void* ptr, size_t* size, size_t* align) {
-+  typename T::get_tls_func get_tls;
-+  CHECK_EQ(sizeof(get_tls), sizeof(ptr));
-+  internal_memcpy(&get_tls, &ptr, sizeof(ptr));
-+  CHECK_NE(get_tls, 0);
-+  get_tls(size, align);
-+}
-+}  // namespace
-+
- void InitTlsSize() {
- // all current supported platforms have 16 bytes stack alignment
-   const size_t kStackAlign = 16;
--  typedef void (*get_tls_func)(size_t*, size_t*) DL_INTERNAL_FUNCTION;
--  get_tls_func get_tls;
-   void *get_tls_static_info_ptr = dlsym(RTLD_NEXT, "_dl_get_tls_static_info");
--  CHECK_EQ(sizeof(get_tls), sizeof(get_tls_static_info_ptr));
--  internal_memcpy(&get_tls, &get_tls_static_info_ptr,
--                  sizeof(get_tls_static_info_ptr));
--  CHECK_NE(get_tls, 0);
-   size_t tls_size = 0;
-   size_t tls_align = 0;
--  get_tls(&tls_size, &tls_align);
-+  // On i?86, _dl_get_tls_static_info used to be internal_function, i.e.
-+  // __attribute__((regparm(3), stdcall)) before glibc 2.27 and is normal
-+  // function in 2.27 and later.
-+  if (CHECK_GET_TLS_STATIC_INFO_VERSION &&
-+      !dlvsym(RTLD_NEXT, "glob", "GLIBC_2.27"))
-+    CallGetTls<GetTlsStaticInfoRegparmCall>(get_tls_static_info_ptr,
-+                                            &tls_size, &tls_align);
-+  else
-+    CallGetTls<GetTlsStaticInfoCall>(get_tls_static_info_ptr,
-+                                     &tls_size, &tls_align);
-   if (tls_align < kStackAlign)
-     tls_align = kStackAlign;
-   g_tls_size = RoundUpTo(tls_size, tls_align);
-
-From 05ecff3cb7a0898de0d709a495f229ab611f3ca4 Mon Sep 17 00:00:00 2001
-From: Peter Wu <peter at lekensteyn.nl>
-Date: Thu, 14 Jun 2018 10:42:54 +0000
-Subject: [PATCH 2/2] [ASAN] fix startup crash in dlsym for long paths since
- glibc 2.27
-
-Summary:
-Error messages for dlsym used to be stored on the stack, but since
-commit 2449ae7b ("ld.so: Introduce struct dl_exception") in glibc 2.27
-these are now stored on the heap (and thus use the dlsym alloc pool).
-
-Messages look like "undefined symbol: __isoc99_printf\0/path/to/a.out".
-With many missing library functions and long object paths, the pool is
-quickly exhausted. Implement a simple mechanism to return freed memory
-to the pool (clear it in case it is used for calloc).
-
-Fixes https://github.com/google/sanitizers/issues/957
-
-Reviewed By: vitalybuka
-
-Differential Revision: https://reviews.llvm.org/D47995
-
-git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@334703 91177308-0d34-0410-b5e6-96231b3b80d8
----
- lib/asan/asan_malloc_linux.cc           | 19 ++++++++++++++++++-
- test/asan/TestCases/long-object-path.cc |  7 +++++++
- 2 files changed, 25 insertions(+), 1 deletion(-)
- create mode 100644 test/asan/TestCases/long-object-path.cc
-
-diff --git a/lib/asan/asan_malloc_linux.cc b/lib/asan/asan_malloc_linux.cc
-index 6697ff876..af05af9d6 100644
---- a/lib/asan/asan_malloc_linux.cc
-+++ b/lib/asan/asan_malloc_linux.cc
-@@ -28,6 +28,7 @@
- using namespace __asan;  // NOLINT
- 
- static uptr allocated_for_dlsym;
-+static uptr last_dlsym_alloc_size_in_words;
- static const uptr kDlsymAllocPoolSize = 1024;
- static uptr alloc_memory_for_dlsym[kDlsymAllocPoolSize];
- 
-@@ -39,11 +40,25 @@ static INLINE bool IsInDlsymAllocPool(const void *ptr) {
- static void *AllocateFromLocalPool(uptr size_in_bytes) {
-   uptr size_in_words = RoundUpTo(size_in_bytes, kWordSize) / kWordSize;
-   void *mem = (void*)&alloc_memory_for_dlsym[allocated_for_dlsym];
-+  last_dlsym_alloc_size_in_words = size_in_words;
-   allocated_for_dlsym += size_in_words;
-   CHECK_LT(allocated_for_dlsym, kDlsymAllocPoolSize);
-   return mem;
- }
- 
-+static void DeallocateFromLocalPool(const void *ptr) {
-+  // Hack: since glibc 2.27, dlsym longer use stack-allocated memory to store
-+  // error messages and instead use malloc followed by free. To avoid pool
-+  // exhaustion due to long object filenames, handle that special case here.
-+  uptr prev_offset = allocated_for_dlsym - last_dlsym_alloc_size_in_words;
-+  void *prev_mem = (void*)&alloc_memory_for_dlsym[prev_offset];
-+  if (prev_mem == ptr) {
-+    REAL(memset)(prev_mem, 0, last_dlsym_alloc_size_in_words * kWordSize);
-+    allocated_for_dlsym = prev_offset;
-+    last_dlsym_alloc_size_in_words = 0;
-+  }
-+}
-+
- static INLINE bool MaybeInDlsym() {
-   // Fuchsia doesn't use dlsym-based interceptors.
-   return !SANITIZER_FUCHSIA && asan_init_is_running;
-@@ -66,8 +81,10 @@ static void *ReallocFromLocalPool(void *ptr, uptr size) {
- 
- INTERCEPTOR(void, free, void *ptr) {
-   GET_STACK_TRACE_FREE;
--  if (UNLIKELY(IsInDlsymAllocPool(ptr)))
-+  if (UNLIKELY(IsInDlsymAllocPool(ptr))) {
-+    DeallocateFromLocalPool(ptr);
-     return;
-+  }
-   asan_free(ptr, &stack, FROM_MALLOC);
- }
- 
-diff --git a/test/asan/TestCases/long-object-path.cc b/test/asan/TestCases/long-object-path.cc
-new file mode 100644
-index 000000000..592b0abb0
---- /dev/null
-+++ b/test/asan/TestCases/long-object-path.cc
-@@ -0,0 +1,7 @@
-+// RUN: mkdir -p %T/a-long-directory-name-to-test-allocations-for-exceptions-in-_dl_lookup_symbol_x-since-glibc-2.27
-+// RUN: %clangxx_asan -g %s -o %T/long-object-path
-+// RUN: %run %T/a-*/../a-*/../a-*/../a-*/../a-*/../a-*/../a-*/../a-*/../long-object-path
-+
-+int main(void) {
-+    return 0;
-+}



More information about the arch-commits mailing list