opt GIL: GraalHPyContext

Author: cosminbasca

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/hpy/GraalHPyContext.java

@@ -38,6 +38,7 @@
  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
  * SOFTWARE.
  */
+// skip GIL
 package com.oracle.graal.python.builtins.objects.cext.hpy;
 
 import static com.oracle.graal.python.builtins.PythonBuiltinClassType.MemoryError;
@@ -132,7 +133,6 @@
 import com.oracle.graal.python.nodes.expression.UnaryArithmetic;
 import com.oracle.graal.python.runtime.AsyncHandler;
 import com.oracle.graal.python.runtime.ExecutionContext.CalleeContext;
-import com.oracle.graal.python.runtime.GilNode;
 import com.oracle.graal.python.runtime.PythonContext;
 import com.oracle.graal.python.runtime.exception.PException;
 import com.oracle.graal.python.util.OverflowException;
@@ -144,7 +144,6 @@
 import com.oracle.truffle.api.TruffleLanguage.ContextReference;
 import com.oracle.truffle.api.TruffleLogger;
 import com.oracle.truffle.api.dsl.Cached;
-import com.oracle.truffle.api.dsl.Cached.Exclusive;
 import com.oracle.truffle.api.dsl.GenerateUncached;
 import com.oracle.truffle.api.dsl.ImportStatic;
 import com.oracle.truffle.api.dsl.Specialization;
@@ -541,41 +540,24 @@ public PException getCurrentException() {
     }
 
     @ExportMessage
-    boolean isPointer(@Exclusive @Cached GilNode gil) {
-        boolean mustRelease = gil.acquire();
-        try {
-            return nativePointer != null;
-        } finally {
-            gil.release(mustRelease);
-        }
+    boolean isPointer() {
+        return nativePointer != null;
     }
 
     @ExportMessage(limit = "1")
     @SuppressWarnings("static-method")
-    long asPointer(
-                    @CachedLibrary("this.nativePointer") InteropLibrary interopLibrary, @Exclusive @Cached GilNode gil) throws UnsupportedMessageException {
-        boolean mustRelease = gil.acquire();
-        try {
-            if (isPointer(gil)) {
-                return interopLibrary.asPointer(nativePointer);
-            }
-            CompilerDirectives.transferToInterpreterAndInvalidate();
-            throw UnsupportedMessageException.create();
-        } finally {
-            gil.release(mustRelease);
+    long asPointer(@CachedLibrary("this.nativePointer") InteropLibrary interopLibrary) throws UnsupportedMessageException {
+        if (isPointer()) {
+            return interopLibrary.asPointer(nativePointer);
         }
+        CompilerDirectives.transferToInterpreterAndInvalidate();
+        throw UnsupportedMessageException.create();
     }
 
     @ExportMessage
-    void toNative(
-                    @Cached PCallHPyFunction callContextToNativeNode, @Exclusive @Cached GilNode gil) {
-        boolean mustRelease = gil.acquire();
-        try {
-            if (!isPointer(gil)) {
-                nativePointer = callContextToNativeNode.call(this, GRAAL_HPY_CONTEXT_TO_NATIVE, this);
-            }
-        } finally {
-            gil.release(mustRelease);
+    void toNative(@Cached PCallHPyFunction callContextToNativeNode) {
+        if (!isPointer()) {
+            nativePointer = callContextToNativeNode.call(this, GRAAL_HPY_CONTEXT_TO_NATIVE, this);
         }
     }
 
@@ -598,24 +580,14 @@ Object getMembers(@SuppressWarnings("unused") boolean includeInternal) {
 
     @ExportMessage
     @SuppressWarnings("static-method")
-    boolean isMemberReadable(String key, @Exclusive @Cached GilNode gil) {
-        boolean mustRelease = gil.acquire();
-        try {
-            return HPyContextMembers.getByName(key) != null;
-        } finally {
-            gil.release(mustRelease);
-        }
+    boolean isMemberReadable(String key) {
+        return HPyContextMembers.getByName(key) != null;
     }
 
     @ExportMessage
     Object readMember(String key,
-                    @Cached GraalHPyReadMemberNode readMemberNode, @Exclusive @Cached GilNode gil) {
-        boolean mustRelease = gil.acquire();
-        try {
-            return readMemberNode.execute(this, key);
-        } finally {
-            gil.release(mustRelease);
-        }
+                    @Cached GraalHPyReadMemberNode readMemberNode) {
+        return readMemberNode.execute(this, key);
     }
 
     @ExportMessage