Thanks to visit codestin.com
Credit goes to github.com

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
Set the proper relationship between static builtin types and the subi… 
…nterpreters.
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed May 1, 2023
commit eeeff5277ea98fb60700b91387b154508acb22b0
12 changes: 9 additions & 3 deletions Objects/typeobject.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4508,9 +4508,11 @@ _PyStaticType_Dealloc(PyInterpreterState *interp, PyTypeObject *type)

clear_static_type_objects(interp, type);

type->tp_flags &= ~Py_TPFLAGS_READY;
type->tp_flags &= ~Py_TPFLAGS_VALID_VERSION_TAG;
type->tp_version_tag = 0;
if (_Py_IsMainInterpreter(interp)) {
type->tp_flags &= ~Py_TPFLAGS_READY;
type->tp_flags &= ~Py_TPFLAGS_VALID_VERSION_TAG;
type->tp_version_tag = 0;
}

_PyStaticType_ClearWeakRefs(interp, type);
static_builtin_state_clear(interp, type);
Expand Down Expand Up @@ -7037,9 +7039,13 @@ _PyStaticType_InitBuiltin(PyInterpreterState *interp, PyTypeObject *self)
if (self->tp_flags & Py_TPFLAGS_READY) {
assert(self->tp_flags & _Py_TPFLAGS_STATIC_BUILTIN);
assert(_PyType_CheckConsistency(self));
/* Per-interpreter tp_subclasses is done lazily.
Otherwise we would initialize it here. */
return 0;
}

assert(_Py_IsMainInterpreter(interp));

self->tp_flags |= _Py_TPFLAGS_STATIC_BUILTIN;
self->tp_flags |= Py_TPFLAGS_IMMUTABLETYPE;

Expand Down