no PYJL_LOCK

Author: Dani Pinyol

src/JlWrap/C.jl

@@ -26,7 +26,7 @@ const PYJLVALUES = []
 # unused indices in PYJLVALUES
 const PYJLFREEVALUES = Int[]
 # lock protecting PYJLVALUES and PYJLFREEVALUES from concurrent modification
-const PYJL_LOCK = Threads.SpinLock()
+#const PYJL_LOCK = Threads.SpinLock()
 # set to true by an atexit hook so _pyjl_dealloc can skip Julia runtime calls
 # (Base.GC.enable, lock) after jl_atexit_hook has torn down the runtime
 const JL_EXITING = Ref(false)
@@ -49,10 +49,10 @@ function _pyjl_dealloc(o::C.PyPtr)
         # enqueue → Py_DecRef → _pyjl_dealloc → push! on same vector →
         # ConcurrencyViolationError). The lock guards against true multi-thread races.
         prev = Base.GC.enable(false)
-        lock(PYJL_LOCK)
+        # lock(PYJL_LOCK)
         PYJLVALUES[idx] = nothing
         push!(PYJLFREEVALUES, idx)
-        unlock(PYJL_LOCK)
+        # unlock(PYJL_LOCK)
         Base.GC.enable(prev)
     end
     (C.@ft UnsafePtr{PyJuliaValueObject}(o).weaklist[!]) == C.PyNULL || C.PyObject_ClearWeakRefs(o)
@@ -394,15 +394,15 @@ PyJuliaValue_SetValue(_o, @nospecialize(v)) = Base.GC.@preserve _o begin
         # Disable GC to prevent push!/pop! from triggering a GC whose finalizers
         # re-entrantly resize the same vectors (see _pyjl_dealloc comment).
         prev = Base.GC.enable(false)
-        lock(PYJL_LOCK)
+        #lock(PYJL_LOCK)
         if isempty(PYJLFREEVALUES)
             push!(PYJLVALUES, v)
             idx = length(PYJLVALUES)
         else
             idx = pop!(PYJLFREEVALUES)
             PYJLVALUES[idx] = v
         end
-        unlock(PYJL_LOCK)
+        #unlock(PYJL_LOCK)
         Base.GC.enable(prev)
         C.@ft UnsafePtr{PyJuliaValueObject}(o).value[] = idx
     else