[GR-74148] Relax multiprocessing wait timeout upper bound

timfel · timfel · commit dd462f172fea · 2026-03-19T11:57:05.000Z
PullRequest: graalpython/4333
diff --git a/graalpython/com.oracle.graal.python.test/src/tests/test_multiprocessing_graalpy.py b/graalpython/com.oracle.graal.python.test/src/tests/test_multiprocessing_graalpy.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2020, 2024, Oracle and/or its affiliates. All rights reserved.
+# Copyright (c) 2020, 2026, Oracle and/or its affiliates. All rights reserved.
 # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
 #
 # The Universal Permissive License (UPL), Version 1.0
@@ -80,7 +80,9 @@ def test_wait_timeout():
             res = wait(fds, timeout)
             delta = time.monotonic() - start
             assert not res
-            assert delta < timeout * 2
+            # The GraalPy multiprocessing wait path actively polls fake file descriptors and may
+            # overshoot under scheduling contention.
+            assert delta < timeout * 8
             assert delta > timeout / 2
 
 
diff --git a/graalpython/lib-python/3/multiprocessing/context.py b/graalpython/lib-python/3/multiprocessing/context.py
@@ -23,6 +23,24 @@ class TimeoutError(ProcessError):
 class AuthenticationError(ProcessError):
     pass
 
+
+# Begin Truffle change
+_graalpy_backend_futurewarned = False
+
+
+def _warn_graalpy_backend_futurewarning_once(stacklevel):
+    import warnings
+    global _graalpy_backend_futurewarned
+    if not _graalpy_backend_futurewarned:
+        warnings.warn(
+            "The 'graalpy' multiprocessing backend is deprecated and will be removed in a future "
+            "GraalPy release. Use the 'spawn' start method where available.",
+            FutureWarning,
+            stacklevel=stacklevel,
+        )
+        _graalpy_backend_futurewarned = True
+# End Truffle change
+
 #
 # Base type for contexts. Bound methods of an instance of this type are included in __all__ of __init__.py
 #
@@ -215,7 +233,10 @@ def _check_available(self):
 
     # Begin Truffle change
     def _is_graalpy(self):
-        return isinstance(self.get_context(), GraalPyContext)
+        is_graalpy = isinstance(self.get_context(), GraalPyContext)
+        if is_graalpy:
+            _warn_graalpy_backend_futurewarning_once(stacklevel=3)
+        return is_graalpy
 
     def _get_id(self):
         if self._is_graalpy():
@@ -321,6 +342,7 @@ class GraalPyProcess(process.BaseProcess):
     _start_method = 'graalpy'
     @staticmethod
     def _Popen(process_obj):
+        _warn_graalpy_backend_futurewarning_once(stacklevel=3)
         from multiprocessing.popen_truffleprocess import Popen
         return Popen(process_obj)
 
