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

Skip to content

GH-126491: GC: Mark objects reachable from roots before doing cycle collection #127110

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 18 commits into from
Dec 2, 2024
Merged

GH-126491: GC: Mark objects reachable from roots before doing cycle collection #127110

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
2b0a2c5
Change the young generation to 'not visited'. WIP
markshannon Nov 19, 2024
754bb68
Tidy up visited tracking and add consistency asserts
markshannon Nov 20, 2024
ac3ad26
Partially apply mark-first-gc patch
markshannon Nov 21, 2024
5c3a5ef
Rename get_heap_size
markshannon Nov 21, 2024
0ac1c07
Set visited on entry frame
markshannon Nov 21, 2024
bb51b19
Apply more diff, but comment out marking step
markshannon Nov 21, 2024
be636c9
Refactor work to do calculation
markshannon Nov 21, 2024
287f381
Turn on marking. Explain issue with tuple untracking
markshannon Nov 21, 2024
7e920d6
Add news
markshannon Nov 21, 2024
0b46494
Add stats for marking
markshannon Nov 21, 2024
698abb3
Do not touch permanent generation
markshannon Nov 21, 2024
6d8a0d4
Move informative failure message in test
markshannon Nov 22, 2024
d9632c6
Increase threshold a bit
markshannon Nov 22, 2024
0702959
Increase number of iterations
markshannon Nov 22, 2024
6063dc8
Return number of iterations to 20k
markshannon Nov 22, 2024
67b1e05
Fix compiler warning
markshannon Nov 22, 2024
eaea41e
Turn off extra debugging
markshannon Nov 22, 2024
79ab26c
Add missing macro
markshannon Nov 22, 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
Rename get_heap_size
  • Loading branch information
@markshannon
markshannon committed Nov 21, 2024
commit 5c3a5effc10718131a9da5b494cabfb7a5fcee29
4 changes: 2 additions & 2 deletions Lib/test/test_gc.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1173,14 +1173,14 @@ def make_ll(depth):
enabled = gc.isenabled()
gc.enable()
olds = []
initial_heap_size = _testinternalcapi.get_heap_size()
initial_heap_size = _testinternalcapi.get_tracked_heap_size()
for i in range(20_000):
newhead = make_ll(20)
count += 20
newhead.surprise = head
olds.append(newhead)
if len(olds) == 20:
new_objects = _testinternalcapi.get_heap_size() - initial_heap_size
new_objects = _testinternalcapi.get_tracked_heap_size() - initial_heap_size
self.assertLess(new_objects, 25_000)
del olds[:]
if not enabled:
Expand Down
4 changes: 2 additions & 2 deletions Modules/_testinternalcapi.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2077,7 +2077,7 @@ has_deferred_refcount(PyObject *self, PyObject *op)
}

static PyObject *
get_heap_size(PyObject *self, PyObject *Py_UNUSED(ignored))
get_tracked_heap_size(PyObject *self, PyObject *Py_UNUSED(ignored))
{
return PyLong_FromInt64(PyInterpreterState_Get()->gc.heap_size);
}
Expand Down Expand Up @@ -2179,7 +2179,7 @@ static PyMethodDef module_functions[] = {
{"get_static_builtin_types", get_static_builtin_types, METH_NOARGS},
{"identify_type_slot_wrappers", identify_type_slot_wrappers, METH_NOARGS},
{"has_deferred_refcount", has_deferred_refcount, METH_O},
{"get_heap_size", get_heap_size, METH_NOARGS},
{"get_tracked_heap_size", get_tracked_heap_size, METH_NOARGS},
{NULL, NULL} /* sentinel */
};

Expand Down