[Python-checkins] cpython (merge 3.5 -> default): Merge 3.5 (test_faulthandler)

[victor.stinner]

https://hg.python.org/cpython/rev/de5f5648be2d
changeset:   100547:de5f5648be2d
parent:      100544:0f251e1b877d
parent:      100546:c298c6d8b324
user:        Victor Stinner <victor.stinner at gmail.com>
date:        Tue Mar 15 17:24:13 2016 +0100
summary:
  Merge 3.5 (test_faulthandler)

files:
  Lib/test/test_faulthandler.py |   8 ++++++++
  Modules/faulthandler.c        |  12 ++++++++++--
  2 files changed, 18 insertions(+), 2 deletions(-)


diff --git a/Lib/test/test_faulthandler.py b/Lib/test/test_faulthandler.py
--- a/Lib/test/test_faulthandler.py
+++ b/Lib/test/test_faulthandler.py
@@ -185,6 +185,14 @@
             2,
             'xyz')
 
+    def test_fatal_error_without_gil(self):
+        self.check_fatal_error("""
+            import faulthandler
+            faulthandler._fatal_error(b'xyz', True)
+            """,
+            2,
+            'xyz')
+
     @unittest.skipIf(sys.platform.startswith('openbsd') and HAVE_THREADS,
                      "Issue #12868: sigaltstack() doesn't work on "
                      "OpenBSD if Python is compiled with pthread")
diff --git a/Modules/faulthandler.c b/Modules/faulthandler.c
--- a/Modules/faulthandler.c
+++ b/Modules/faulthandler.c
@@ -935,10 +935,18 @@
 faulthandler_fatal_error_py(PyObject *self, PyObject *args)
 {
     char *message;
-    if (!PyArg_ParseTuple(args, "y:fatal_error", &message))
+    int release_gil = 0;
+    if (!PyArg_ParseTuple(args, "y|i:fatal_error", &message, &release_gil))
         return NULL;
     faulthandler_suppress_crash_report();
-    Py_FatalError(message);
+    if (release_gil) {
+        Py_BEGIN_ALLOW_THREADS
+        Py_FatalError(message);
+        Py_END_ALLOW_THREADS
+    }
+    else {
+        Py_FatalError(message);
+    }
     Py_RETURN_NONE;
 }
 

-- 
Repository URL: https://hg.python.org/cpython