[Lldb-commits] [lldb] [lldb][debugserver] Synchronize interrupt and resume signals (PR #131073)

[Jonas Devlieghere via lldb-commits]

================
@@ -1573,6 +1576,8 @@ static uint64_t bits(uint64_t value, uint32_t msbit, uint32_t lsbit) {
 }
 
 bool MachProcess::Interrupt() {
+  PTHREAD_MUTEX_LOCKER(locker, m_exception_and_signal_mutex);
+
----------------
JDevlieghere wrote:

I was worried about lock order inversion between the exception mutex and the state mutex. That said, how we're protecting GetState is already questionable (i.e. nothing guarantees the state hasn't changed between the function returned you read the value) so I can tighten the scope and we can fix the issue if it comes up when we try to fix the state locking. 

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