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
Call _PyIO_FiniTypes() earlier.
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed May 1, 2023
commit 98f3ed7e02ca70cf01e29bc9997d5033ddf0d738
4 changes: 2 additions & 2 deletions Python/pylifecycle.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1663,6 +1663,8 @@ flush_std_files(void)
static void
finalize_interp_types(PyInterpreterState *interp)
{
_PyIO_FiniTypes(interp);

_PyUnicode_FiniTypes(interp);
_PySys_Fini(interp);
_PyExc_Fini(interp);
Expand Down Expand Up @@ -1706,8 +1708,6 @@ finalize_interp_clear(PyThreadState *tstate)
/* Clear interpreter state and all thread states */
_PyInterpreterState_Clear(tstate);

_PyIO_FiniTypes(tstate->interp);

/* Clear all loghooks */
/* Both _PySys_Audit function and users still need PyObject, such as tuple.
Call _PySys_ClearAuditHooks when PyObject available. */
Expand Down