gh-123275: Support -Xgil=1 and PYTHON_GIL=1 on non-free-threaded builds (gh-123276)

ZeroIntensity · web-flow · commit 84ad264ce602 · 2024-09-06T08:53:47.000+09:00
diff --git a/Doc/using/cmdline.rst b/Doc/using/cmdline.rst
@@ -622,7 +622,7 @@ Miscellaneous options
      .. versionadded:: 3.13
 
    * :samp:`-X gil={0,1}` forces the GIL to be disabled or enabled,
-     respectively. Only available in builds configured with
+     respectively. Setting to ``0`` is only available in builds configured with
      :option:`--disable-gil`. See also :envvar:`PYTHON_GIL` and
      :ref:`whatsnew313-free-threaded-cpython`.
 
@@ -1221,13 +1221,12 @@ conflict.
 .. envvar:: PYTHON_GIL
 
    If this variable is set to ``1``, the global interpreter lock (GIL) will be
-   forced on. Setting it to ``0`` forces the GIL off.
+   forced on. Setting it to ``0`` forces the GIL off (needs Python configured with
+   the :option:`--disable-gil` build option).
 
    See also the :option:`-X gil <-X>` command-line option, which takes
    precedence over this variable, and :ref:`whatsnew313-free-threaded-cpython`.
 
-   Needs Python configured with the :option:`--disable-gil` build option.
-
    .. versionadded:: 3.13
 
 Debug-mode variables
diff --git a/Misc/NEWS.d/next/Core_and_Builtins/2024-08-23-18-31-10.gh-issue-123275.DprIrj.rst b/Misc/NEWS.d/next/Core_and_Builtins/2024-08-23-18-31-10.gh-issue-123275.DprIrj.rst
@@ -0,0 +1 @@
+Support :option:`-X gil=1 <-X>` and :envvar:`PYTHON_GIL=1 <PYTHON_GIL>` on non-free-threaded builds.
diff --git a/Python/initconfig.c b/Python/initconfig.c
@@ -1714,20 +1714,24 @@ config_wstr_to_int(const wchar_t *wstr, int *result)
 static PyStatus
 config_read_gil(PyConfig *config, size_t len, wchar_t first_char)
 {
-#ifdef Py_GIL_DISABLED
     if (len == 1 && first_char == L'0') {
+#ifdef Py_GIL_DISABLED
         config->enable_gil = _PyConfig_GIL_DISABLE;
+#else
+        return _PyStatus_ERR("Disabling the GIL is not supported by this build");
+#endif
     }
     else if (len == 1 && first_char == L'1') {
+#ifdef Py_GIL_DISABLED
         config->enable_gil = _PyConfig_GIL_ENABLE;
+#else
+        return _PyStatus_OK();
+#endif
     }
     else {
         return _PyStatus_ERR("PYTHON_GIL / -X gil must be \"0\" or \"1\"");
     }
     return _PyStatus_OK();
-#else
-    return _PyStatus_ERR("PYTHON_GIL / -X gil are not supported by this build");
-#endif
 }
 
 static PyStatus

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+Support :option:`-X gil=1 <-X>` and :envvar:`PYTHON_GIL=1 <PYTHON_GIL>` on non-free-threaded builds.
Original file line number	Diff line number	Diff line change
`@@ -1714,20 +1714,24 @@ config_wstr_to_int(const wchar_t wstr, int result)`
`1714`	`1714`	`static PyStatus`
`1715`	`1715`	`config_read_gil(PyConfig *config, size_t len, wchar_t first_char)`
`1716`	`1716`	`{`
`1717`		`-#ifdef Py_GIL_DISABLED`
`1718`	`1717`	`if (len == 1 && first_char == L'0') {`
	`1718`	`+#ifdef Py_GIL_DISABLED`
`1719`	`1719`	`config->enable_gil = _PyConfig_GIL_DISABLE;`
	`1720`	`+#else`
	`1721`	`+ return _PyStatus_ERR("Disabling the GIL is not supported by this build");`
	`1722`	`+#endif`
`1720`	`1723`	`}`
`1721`	`1724`	`else if (len == 1 && first_char == L'1') {`
	`1725`	`+#ifdef Py_GIL_DISABLED`
`1722`	`1726`	`config->enable_gil = _PyConfig_GIL_ENABLE;`
	`1727`	`+#else`
	`1728`	`+ return _PyStatus_OK();`
	`1729`	`+#endif`
`1723`	`1730`	`}`
`1724`	`1731`	`else {`
`1725`	`1732`	`return _PyStatus_ERR("PYTHON_GIL / -X gil must be \"0\" or \"1\"");`
`1726`	`1733`	`}`
`1727`	`1734`	`return _PyStatus_OK();`
`1728`		`-#else`
`1729`		`- return _PyStatus_ERR("PYTHON_GIL / -X gil are not supported by this build");`
`1730`		`-#endif`
`1731`	`1735`	`}`
`1732`	`1736`
`1733`	`1737`	`static PyStatus`