Implement os.setsid

Author: msimacek

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

@@ -673,6 +673,22 @@ long getSid(VirtualFrame frame, long pid,
         }
     }
 
+    @Builtin(name = "setsid")
+    @GenerateNodeFactory
+    public abstract static class SetSidNode extends PythonBuiltinNode {
+
+        @Specialization
+        Object setsid(VirtualFrame frame,
+                        @CachedLibrary("getPosixSupport()") PosixSupportLibrary posixLib) {
+            try {
+                posixLib.setsid(getPosixSupport());
+            } catch (PosixException e) {
+                throw raiseOSErrorFromPosixException(frame, e);
+            }
+            return PNone.NONE;
+        }
+    }
+
     @Builtin(name = "open", minNumOfPositionalArgs = 2, parameterNames = {"path", "flags", "mode"}, keywordOnlyNames = {"dir_fd"})
     @ArgumentClinic(name = "path", conversionClass = PathConversionNode.class, args = {"false", "false"})
     @ArgumentClinic(name = "flags", conversion = ClinicConversion.Int)

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/EmulatedPosixSupport.java

@@ -1997,6 +1997,12 @@ public long getsid(long pid) {
         throw new UnsupportedPosixFeatureException("Emulated getsid not supported");
     }
 
+    @ExportMessage
+    @SuppressWarnings("static-method")
+    public long setsid() {
+        throw new UnsupportedPosixFeatureException("Emulated getsid not supported");
+    }
+
     @ExportMessage
     @SuppressWarnings("static-method")
     public TruffleString ctermid() {

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/ImageBuildtimePosixSupport.java

@@ -674,6 +674,13 @@ final long getsid(long pid,
         return nativeLib.getsid(nativePosixSupport, pid);
     }
 
+    @ExportMessage
+    final long setsid(
+                    @CachedLibrary("this.nativePosixSupport") PosixSupportLibrary nativeLib) throws PosixException {
+        checkNotInImageBuildtime();
+        return nativeLib.setsid(nativePosixSupport);
+    }
+
     @ExportMessage
     final TruffleString ctermid(@CachedLibrary("this.nativePosixSupport") PosixSupportLibrary nativeLib) throws PosixException {
         checkNotInImageBuildtime();

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/LoggingPosixSupport.java

@@ -866,6 +866,17 @@ final long getsid(long pid,
         }
     }
 
+    @ExportMessage
+    final long setsid(
+                    @CachedLibrary("this.delegate") PosixSupportLibrary lib) throws PosixException {
+        logEnter("setsid", "");
+        try {
+            return logExit("getsid", "%d", lib.setsid(delegate));
+        } catch (PosixException e) {
+            throw logException("setsid", e);
+        }
+    }
+
     @ExportMessage
     public int mmapReadBytes(Object mmap, long index, byte[] bytes, int length,
                     @CachedLibrary("this.delegate") PosixSupportLibrary lib) throws PosixException {
@@ -1323,8 +1334,8 @@ private static <T> T logExit(Level level, String msg, String argFmt, T retVal) {
         return retVal;
     }
 
-    private static <T> T logExit(String msg, String argFtm, T retVal) {
-        return logExit(DEFAULT_LEVEL, msg, argFtm, retVal);
+    private static <T> T logExit(String msg, String argFmt, T retVal) {
+        return logExit(DEFAULT_LEVEL, msg, argFmt, retVal);
     }
 
     @TruffleBoundary

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/NFIPosixSupport.java

@@ -215,6 +215,7 @@ private enum PosixNativeFunction {
         call_setpgid("(sint64,sint64):sint32"),
         call_getpgrp("():sint64"),
         call_getsid("(sint64):sint64"),
+        call_setsid("():sint64"),
         call_ctermid("([sint8]):sint32"),
         call_setenv("([sint8], [sint8], sint32):sint32"),
         call_unsetenv("([sint8]):sint32"),
@@ -1156,6 +1157,16 @@ public long getsid(long pid,
         return res;
     }
 
+    @ExportMessage
+    public long setsid(
+                    @Shared("invoke") @Cached InvokeNativeFunction invokeNode) throws PosixException {
+        long res = invokeNode.callLong(this, PosixNativeFunction.call_setsid);
+        if (res < 0) {
+            throw getErrnoAndThrowPosixException(invokeNode);
+        }
+        return res;
+    }
+
     @ExportMessage
     public TruffleString ctermid(@Shared("invoke") @Cached InvokeNativeFunction invokeNode,
                     @Shared("tsFromBytes") @Cached TruffleString.FromByteArrayNode fromByteArrayNode,

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/PosixSupportLibrary.java

@@ -286,6 +286,8 @@ public abstract class PosixSupportLibrary extends Library {
 
     public abstract long getsid(Object receiver, long pid) throws PosixException;
 
+    public abstract long setsid(Object receiver) throws PosixException;
+
     public abstract TruffleString ctermid(Object receiver) throws PosixException;
 
     // note: this leaks memory in nfi backend and is not synchronized

graalpython/python-libposix/src/posix.c

@@ -577,6 +577,10 @@ int64_t call_getsid(int64_t pid) {
     return getsid(pid);
 }
 
+int64_t call_setsid() {
+    return setsid();
+}
+
 int32_t call_ctermid(char *buf) {
     return ctermid(buf) == NULL ? -1 : 0;
 }