Release GIL when closing a file descriptor

Author: fangerer

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/PosixModuleBuiltins.java

@@ -523,13 +523,19 @@ protected ArgumentClinicProvider getArgumentClinic() {
 
         @Specialization
         PNone close(VirtualFrame frame, int fd,
-                        @CachedLibrary("getPosixSupport()") PosixSupportLibrary posixLib) {
+                        @CachedLibrary("getPosixSupport()") PosixSupportLibrary posixLib,
+                        @Cached GilNode gil) {
             try {
                 PythonContext ctx = getContext();
                 if (ctx.getSharedMultiprocessingData().decrementFDRefCount(fd)) {
                     return PNone.NONE;
                 }
-                posixLib.close(getPosixSupport(), fd);
+                gil.release(true);
+                try {
+                    posixLib.close(getPosixSupport(), fd);
+                } finally {
+                    gil.acquire();
+                }
                 return PNone.NONE;
             } catch (PosixException e) {
                 throw raiseOSErrorFromPosixException(frame, e);

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/io/PFileIO.java

@@ -42,6 +42,7 @@
 
 import com.oracle.graal.python.builtins.objects.object.PythonBuiltinObject;
 import com.oracle.graal.python.runtime.AsyncHandler;
+import com.oracle.graal.python.runtime.GilNode;
 import com.oracle.graal.python.runtime.PosixSupportLibrary;
 import com.oracle.graal.python.runtime.PythonContext;
 import com.oracle.truffle.api.CompilerDirectives;
@@ -213,9 +214,10 @@ public OwnFD(Object referent, int fd, PythonContext context) {
         this.context = context;
     }
 
+    @SuppressWarnings("try")
     protected void doRelease() {
-        try {
-            markReleased();
+        markReleased();
+        try (GilNode.UncachedRelease gil = GilNode.uncachedRelease()) {
             PosixSupportLibrary.getUncached().close(context.getPosixSupport(), (int) getReference());
         } catch (PosixSupportLibrary.PosixException e) {
             // ignore