[compiler-rt] [scudo] Add two missing locks to enable/disable. (PR #79670)

Evgenii Stepanov via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 29 13:26:39 PST 2024


================
@@ -136,6 +136,14 @@ class StackDepot {
   u64 operator[](uptr RingPos) const {
     return atomic_load_relaxed(&Ring[RingPos & RingMask]);
   }
+
+  void disable() NO_THREAD_SAFETY_ANALYSIS {
+    RingEndMu.lock();
+  }
+
+  void enable() NO_THREAD_SAFETY_ANALYSIS {
+    RingEndMu.unlock();
+  }
----------------
eugenis wrote:

Great point about the BufferPool lock, removed it from the pull request.
Added a comment in StackDepot.

https://github.com/llvm/llvm-project/pull/79670


More information about the llvm-commits mailing list