Skip to content

gh-76785: Improved Subinterpreters Compatibility with 3.12 #115424

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ericsnowcurrently merged 20 commits into from
Feb 13, 2024
Merged

gh-76785: Improved Subinterpreters Compatibility with 3.12 #115424

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
ba23dd7
Add accessor macros for cross-interpreter data.
ericsnowcurrently Dec 19, 2023
db3f7de
Hide newer PyCodeObject fields behind macros.
ericsnowcurrently Dec 19, 2023
ee80f64
Add _PyThreadState_SetWhence().
ericsnowcurrently Dec 19, 2023
ef8dea0
Fix crossinterp.c.
ericsnowcurrently Jan 4, 2024
71a6a9a
Use PyInterpreterState_GetID().
ericsnowcurrently Jan 5, 2024
00b80ff
Add _PyRuntimeState_GetXIState() and _PyInterpreterState_GetXIState().
ericsnowcurrently Jan 5, 2024
e5fcc75
Factor out _interpreters_common.h.
ericsnowcurrently Jan 5, 2024
f50017e
Factor out crossinterp_data_lookup.h.
ericsnowcurrently Jan 5, 2024
7bd8f7f
Use PyInterpreterState_Get().
ericsnowcurrently Jan 5, 2024
f68737b
Add _Py_EMPTY_STR.
ericsnowcurrently Jan 5, 2024
c99fd01
Use PyErr_SetString().
ericsnowcurrently Jan 5, 2024
e2f9cf0
Use MODULE_NAME for the module init func name.
ericsnowcurrently Jan 6, 2024
dedc0e8
Factor out crossinterp_exceptionsw.h.
ericsnowcurrently Jan 8, 2024
64605ed
Fix exception docstrings.
ericsnowcurrently Jan 8, 2024
5c008de
Move NotShareableError to crossinterp_exceptions.h.
ericsnowcurrently Jan 8, 2024
f17b29b
Work around a ref leak.
ericsnowcurrently Jan 9, 2024
18ede77
Fix build dependencies.
ericsnowcurrently Jan 10, 2024
842a990
Fix the Exception names.
ericsnowcurrently Jan 22, 2024
8448f9d
Use PyInterpreterState_GetID().
ericsnowcurrently Jan 22, 2024
455460c
Update the C analyzer.
ericsnowcurrently Feb 13, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use PyInterpreterState_Get().
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed Feb 13, 2024
commit 7bd8f7f8b5d0d877c7a2b4ebc208a951467c1750
17 changes: 6 additions & 11 deletions Python/crossinterp.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
#include "pycore_initconfig.h" // _PyStatus_OK()
#include "pycore_namespace.h" //_PyNamespace_New()
#include "pycore_pyerrors.h" // _PyErr_Clear()
#include "pycore_pystate.h" // _PyInterpreterState_GET()
#include "pycore_typeobject.h" // _PyType_GetModuleName()
#include "pycore_weakref.h" // _PyWeakref_GET_REF()

Expand Down Expand Up @@ -67,7 +66,7 @@ int
_Py_CallInInterpreter(PyInterpreterState *interp,
_Py_simple_func func, void *arg)
{
if (interp == _PyThreadState_GetCurrent()->interp) {
if (interp == PyInterpreterState_Get()) {
return func(arg);
}
// XXX Emit a warning if this fails?
Expand All @@ -79,7 +78,7 @@ int
_Py_CallInInterpreterAndRawFree(PyInterpreterState *interp,
_Py_simple_func func, void *arg)
{
if (interp == _PyThreadState_GetCurrent()->interp) {
if (interp == PyInterpreterState_Get()) {
int res = func(arg);
PyMem_RawFree(arg);
return res;
Expand Down Expand Up @@ -292,7 +291,7 @@ _set_xid_lookup_failure(PyInterpreterState *interp,
int
_PyObject_CheckCrossInterpreterData(PyObject *obj)
{
PyInterpreterState *interp = _PyInterpreterState_GET();
PyInterpreterState *interp = PyInterpreterState_Get();
crossinterpdatafunc getdata = lookup_getdata(interp, obj);
if (getdata == NULL) {
if (!PyErr_Occurred()) {
Expand All @@ -306,11 +305,7 @@ _PyObject_CheckCrossInterpreterData(PyObject *obj)
int
_PyObject_GetCrossInterpreterData(PyObject *obj, _PyCrossInterpreterData *data)
{
PyThreadState *tstate = _PyThreadState_GetCurrent();
#ifdef Py_DEBUG
// The caller must hold the GIL
_Py_EnsureTstateNotNULL(tstate);
#endif
PyThreadState *tstate = PyThreadState_Get();
PyInterpreterState *interp = tstate->interp;

// Reset data before re-populating.
Expand Down Expand Up @@ -1340,7 +1335,7 @@ _PyXI_FreeNamespace(_PyXI_namespace *ns)
return;
}

if (interpid == PyInterpreterState_GetID(_PyInterpreterState_GET())) {
if (interpid == PyInterpreterState_GetID(PyInterpreterState_Get())) {
_sharedns_free(ns);
}
else {
Expand Down Expand Up @@ -1539,7 +1534,7 @@ _propagate_not_shareable_error(_PyXI_session *session)
if (session == NULL) {
return;
}
PyInterpreterState *interp = _PyInterpreterState_GET();
PyInterpreterState *interp = PyInterpreterState_Get();
if (PyErr_ExceptionMatches(_get_not_shareable_error_type(interp))) {
// We want to propagate the exception directly.
session->_error_override = _PyXI_ERR_NOT_SHAREABLE;
Expand Down