[PATCH] D54664: [tsan] Add __cxa_guard_acquire hooks to support cooperative scheduling

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka created this revision.
Herald added subscribers: llvm-commits, kubamracek.
vitalybuka added a reviewer: dvyukov.

Repository:
  rL LLVM

https://reviews.llvm.org/D54664

Files:
  compiler-rt/lib/tsan/rtl/tsan_interceptors.cc
  compiler-rt/test/tsan/Linux/cxa_guard_acquire.cc


Index: compiler-rt/test/tsan/Linux/cxa_guard_acquire.cc
===================================================================
--- /dev/null
+++ compiler-rt/test/tsan/Linux/cxa_guard_acquire.cc
@@ -0,0 +1,23 @@
+// RUN: %clangxx_tsan -O1 %s -o %t && %run %t 2>&1 | FileCheck %s
+
+#include <stdio.h>
+
+namespace __tsan {
+
+void OnCxaGuardAcquireEnter() { printf("Enter __cxa_guard_acquire\n"); }
+
+void OnCxaGuardAcquireExit() { printf("Exit __cxa_guard_acquire\n"); }
+
+} // namespace __tsan
+
+int main(int argc, char **argv) {
+  // CHECK: Enter main
+  printf("Enter main\n");
+  // CHECK-NEXT: Enter __cxa_guard_acquire
+  // CHECK-NEXT: Exit __cxa_guard_acquire
+  static int s = argc;
+  (void)s;
+  // CHECK-NEXT: Exit main
+  printf("Exit main\n");
+  return 0;
+}
Index: compiler-rt/lib/tsan/rtl/tsan_interceptors.cc
===================================================================
--- compiler-rt/lib/tsan/rtl/tsan_interceptors.cc
+++ compiler-rt/lib/tsan/rtl/tsan_interceptors.cc
@@ -228,6 +228,16 @@
   libignore()->OnLibraryLoaded(0);
 }
 
+// The following two hooks can be used by for cooperative scheduling when
+// locking.
+#ifdef TSAN_EXTERNAL_HOOKS
+void OnCxaGuardAcquireEnter();
+void OnCxaGuardAcquireExit();
+#else
+SANITIZER_WEAK_CXX_DEFAULT_IMPL void OnCxaGuardAcquireEnter() {}
+SANITIZER_WEAK_CXX_DEFAULT_IMPL void OnCxaGuardAcquireExit() {}
+#endif
+
 }  // namespace __tsan
 
 static ThreadSignalContext *SigCtx(ThreadState *thr) {
@@ -866,6 +876,8 @@
 // Used in thread-safe function static initialization.
 STDCXX_INTERCEPTOR(int, __cxa_guard_acquire, atomic_uint32_t *g) {
   SCOPED_INTERCEPTOR_RAW(__cxa_guard_acquire, g);
+  OnCxaGuardAcquireEnter();
+  auto on_exit = at_scope_exit(&OnCxaGuardAcquireExit);
   for (;;) {
     u32 cmp = atomic_load(g, memory_order_acquire);
     if (cmp == 0) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54664.174500.patch
Type: text/x-patch
Size: 1836 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181117/fd835601/attachment.bin>