Skip to content

gh-94673: Properly Initialize and Finalize Static Builtin Types for Each Interpreter #104072

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
merged 14 commits into from
May 2, 2023
Merged

gh-94673: Properly Initialize and Finalize Static Builtin Types for Each Interpreter #104072

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
dec8f6d
Fix an old bug.
ericsnowcurrently Apr 28, 2023
98f3ed7
Call _PyIO_FiniTypes() earlier.
ericsnowcurrently May 1, 2023
b313293
Pass interp to _PyStructSequence_InitBuiltin() and _PyStructSequence_…
ericsnowcurrently May 1, 2023
4a78881
Pass interp to _PyStaticType_InitBuiltin() and _PyStaticType_Dealloc().
ericsnowcurrently May 1, 2023
4355c00
Add clear_static_type_objects().
ericsnowcurrently May 1, 2023
7dd7d96
Builtin static types are always immortal.
ericsnowcurrently May 1, 2023
f678350
_PyStaticType_Dealloc() is only called on static builtin types.
ericsnowcurrently May 1, 2023
eeeff52
Set the proper relationship between static builtin types and the subi…
ericsnowcurrently May 1, 2023
ae74c41
Set the static builtin index correctly.
ericsnowcurrently May 1, 2023
f2fb5a4
Add some asserts.
ericsnowcurrently May 1, 2023
51d325d
Call static_builtin_state_init() for each interpreter.
ericsnowcurrently May 1, 2023
71e52c2
Only call static_builtin_index_clear() during runtime fini.
ericsnowcurrently May 1, 2023
f41a74d
Finalize static builtin types for each interpreter.
ericsnowcurrently May 1, 2023
68331c4
Fix non-debug builds.
ericsnowcurrently May 2, 2023
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
Add clear_static_type_objects().
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed May 1, 2023
commit 4355c00d04f58438029acf5ea6a2a4418cab0e19
18 changes: 13 additions & 5 deletions Objects/typeobject.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4486,18 +4486,26 @@ clear_static_tp_subclasses(PyTypeObject *type)
clear_subclasses(type);
}

static void
clear_static_type_objects(PyInterpreterState *interp, PyTypeObject *type)
{
if (_Py_IsMainInterpreter(interp)) {
Py_CLEAR(type->tp_dict);
Py_CLEAR(type->tp_bases);
Py_CLEAR(type->tp_mro);
Py_CLEAR(type->tp_cache);
}
clear_static_tp_subclasses(type);
}

void
_PyStaticType_Dealloc(PyInterpreterState *interp, PyTypeObject *type)
{
assert(!(type->tp_flags & Py_TPFLAGS_HEAPTYPE));

type_dealloc_common(type);

Py_CLEAR(type->tp_dict);
Py_CLEAR(type->tp_bases);
Py_CLEAR(type->tp_mro);
Py_CLEAR(type->tp_cache);
clear_static_tp_subclasses(type);
clear_static_type_objects(interp, type);

// PyObject_ClearWeakRefs() raises an exception if Py_REFCNT() != 0
if (Py_REFCNT(type) == 0) {
Expand Down