Skip to content

gh-120161: Clean Up the Managed Static Types State #120663

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

Closed
ericsnowcurrently wants to merge 8 commits into from
Closed

gh-120161: Clean Up the Managed Static Types State #120663

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
e3057ce
Clean up managed_static_type_state_init() a little.
ericsnowcurrently Jun 4, 2024
049934b
Track managed static types globally, with a shared index pool.
ericsnowcurrently Jun 4, 2024
1503e29
Unify the two per-interpreter structs.
ericsnowcurrently Jun 4, 2024
c820fdc
Add some asserts.
ericsnowcurrently Jun 13, 2024
de98b0f
wip - Do not reset a static type's index.
ericsnowcurrently Jun 6, 2024
bbfafd8
Small cleanup.
ericsnowcurrently Jun 17, 2024
2a30616
Drop an ifdef.
ericsnowcurrently Jun 17, 2024
6a20de3
Silence some compiler warnings.
ericsnowcurrently Jun 18, 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
Small cleanup.
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed Jun 17, 2024
commit bbfafd8aaf05b1e2f7d61e06c151450edc6c5b7d
34 changes: 20 additions & 14 deletions Objects/typeobject.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ type_from_ref(PyObject *ref)
}


/* helpers for for static builtin types */
/* helpers for for managed static types */

#ifndef NDEBUG
static inline int
Expand Down Expand Up @@ -389,6 +389,24 @@ managed_static_type_state_clear(PyInterpreterState *interp, PyTypeObject *self,
}
}

static void
managed_static_types_fini(PyInterpreterState *interp)
{
assert(interp->types.managed_static.num_types == 0);
// All the managed static types should have been finalized already.
for (size_t i = 0; i < _Py_MAX_MANAGED_STATIC_TYPES; i++) {
assert(interp->types.managed_static.types[i].type == NULL);
}
if (_Py_IsMainInterpreter(interp)) {
assert(_PyRuntime.types.managed_static.num_types == 0);
// All the managed static types should have been finalized already.
for (size_t i = 0; i < _Py_MAX_MANAGED_STATIC_TYPES; i++) {
assert(_PyRuntime.types.managed_static.types[i].type == NULL);
assert(_PyRuntime.types.managed_static.types[i].interp_count == 0);
}
}
}

// Also see _PyStaticType_InitBuiltin() and _PyStaticType_FiniBuiltin().

/* end static builtin helpers */
Expand Down Expand Up @@ -946,19 +964,7 @@ _PyTypes_Fini(PyInterpreterState *interp)
struct type_cache *cache = &interp->types.type_cache;
type_cache_clear(cache, NULL);

assert(interp->types.managed_static.num_types == 0);
// All the managed static types should have been finalized already.
for (size_t i = 0; i < _Py_MAX_MANAGED_STATIC_TYPES; i++) {
assert(interp->types.managed_static.types[i].type == NULL);
}
if (_Py_IsMainInterpreter(interp)) {
assert(_PyRuntime.types.managed_static.num_types == 0);
// All the managed static types should have been finalized already.
for (size_t i = 0; i < _Py_MAX_MANAGED_STATIC_TYPES; i++) {
assert(_PyRuntime.types.managed_static.types[i].type == NULL);
assert(_PyRuntime.types.managed_static.types[i].interp_count == 0);
}
}
managed_static_types_fini(interp);
}


Expand Down