X Tutup
The Wayback Machine - https://web.archive.org/web/20210609211514/https://github.com/python/devguide/pull/562
Skip to content

/ devguide

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.

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

Jump to bottom

Add section about the design of CPython's garbage collector #562

Merged
merged 28 commits into from Jan 21, 2020
Merged

Add section about the design of CPython's garbage collector #562

merged 28 commits into from Jan 21, 2020

Conversation

@pablogsal
Copy link
Member

@pablogsal pablogsal commented Jan 20, 2020 •
edited

The design of CPython's garbage collector is normally not very well documented anywhere in detail or the documentation is dispersed across multiple places and it on top of that it has plenty of arcane implementation details that difficulties its understanding by just reading the code.

This section is an expanded version of my talk at PyconUS: https://www.youtube.com/watch?v=CLW5Lyc1FN8&t=1428s
@pablogsal pablogsal requested review from brettcannon and willingc Jan 20, 2020
@pablogsal pablogsal self-assigned this Jan 20, 2020
@pablogsal pablogsal force-pushed the pablogsal:gc branch 3 times, most recently from a3b94ff to 42d4a50 Jan 20, 2020
@pablogsal
Copy link
Member Author

@pablogsal pablogsal commented Jan 20, 2020 •
edited

Preview available here: https://deploy-preview-562--cpython-devguide.netlify.com/garbage_collector.html
@pablogsal pablogsal closed this Jan 20, 2020
@pablogsal pablogsal reopened this Jan 20, 2020
@pablogsal pablogsal requested a review from JulienPalard Jan 20, 2020
@pablogsal pablogsal force-pushed the pablogsal:gc branch from 42d4a50 to dd9e668 Jan 20, 2020
@pablogsal
Add section about the design of CPython's garbage collector
10ee4fd
@pablogsal pablogsal force-pushed the pablogsal:gc branch 6 times, most recently from d8105aa to e59e382 Jan 20, 2020
@pablogsal
Fix several typos
8aac485
@pablogsal pablogsal force-pushed the pablogsal:gc branch from e59e382 to 8aac485 Jan 20, 2020
@tim-one
tim-one reviewed Jan 20, 2020
View changes
garbage_collector.rst Outdated Show resolved Hide resolved
@pablogsal @tim-one
Update garbage_collector.rst
7776e14 
Co-Authored-By: Tim Peters <tim.peters@gmail.com>
@tim-one
tim-one reviewed Jan 20, 2020
View changes
garbage_collector.rst Show resolved Hide resolved
pablogsal and others added 2 commits Jan 20, 2020
@pablogsal
Fix more typos
c72ce11
@pablogsal @tim-one
Update garbage_collector.rst
410487b 
Co-Authored-By: Tim Peters <tim.peters@gmail.com>
@pablogsal
Copy link
Member Author

@pablogsal pablogsal commented Jan 20, 2020

@tim-one I hope you like this addition to the dev guide :)
@tim-one
tim-one reviewed Jan 20, 2020
View changes
garbage_collector.rst Show resolved Hide resolved
@pablogsal @tim-one
Update garbage_collector.rst
b18da79 
Co-Authored-By: Tim Peters <tim.peters@gmail.com>
@brettcannon
brettcannon approved these changes Jan 20, 2020
View changes
garbage_collector.rst Outdated Show resolved Hide resolved
garbage_collector.rst Outdated Show resolved Hide resolved
garbage_collector.rst Outdated Show resolved Hide resolved
garbage_collector.rst Outdated Show resolved Hide resolved
garbage_collector.rst Outdated Show resolved Hide resolved
garbage_collector.rst Outdated Show resolved Hide resolved
garbage_collector.rst Outdated Show resolved Hide resolved
garbage_collector.rst Outdated Show resolved Hide resolved
garbage_collector.rst Outdated Show resolved Hide resolved
garbage_collector.rst Outdated Show resolved Hide resolved
pablogsal and others added 4 commits Jan 21, 2020
@pablogsal
Rework sentence about _gc_prev
8fc0547
@pablogsal @tim-one
Update garbage_collector.rst
e956478 
Co-Authored-By: Tim Peters <tim.peters@gmail.com>
@pablogsal
Fix indentation
47190cd
@pablogsal
Fix quotes
722a99c
@tim-one
tim-one reviewed Jan 21, 2020
View changes
garbage_collector.rst Outdated Show resolved Hide resolved
@pablogsal
Fix typo and indentation
b40b030
@tim-one
tim-one reviewed Jan 21, 2020
View changes
garbage_collector.rst Outdated Show resolved Hide resolved
@tim-one
tim-one reviewed Jan 21, 2020
View changes
garbage_collector.rst Outdated Show resolved Hide resolved
@tim-one
tim-one reviewed Jan 21, 2020
View changes
garbage_collector.rst Outdated Show resolved Hide resolved
@pablogsal
Update garbage_collector.rst
111b2bb
@tim-one
tim-one reviewed Jan 21, 2020
View changes
garbage_collector.rst Outdated Show resolved Hide resolved
@pablogsal
Update garbage_collector.rst
2ec5002
@pablogsal pablogsal force-pushed the pablogsal:gc branch from 5391b86 to 0f7d391 Jan 21, 2020
@pablogsal
Add author section
7c8e02a
@pablogsal pablogsal force-pushed the pablogsal:gc branch from 0f7d391 to 7c8e02a Jan 21, 2020
@pablogsal
Add a warning section regarding tagged pointers
e51d354
@pablogsal pablogsal force-pushed the pablogsal:gc branch from 6c02402 to e51d354 Jan 21, 2020
@pablogsal
Add reference to the memory layout
411037f
@pablogsal pablogsal force-pushed the pablogsal:gc branch from d2db8ad to 411037f Jan 21, 2020
@pablogsal
Fix link to the generation section
4081607
@tim-one
tim-one reviewed Jan 21, 2020
View changes
garbage_collector.rst Outdated Show resolved Hide resolved
@tim-one
tim-one reviewed Jan 21, 2020
View changes
garbage_collector.rst Outdated Show resolved Hide resolved
@encukou
encukou reviewed Jan 21, 2020
View changes
garbage_collector.rst Outdated Show resolved Hide resolved
pablogsal and others added 3 commits Jan 21, 2020
@pablogsal @tim-one
Apply suggestions from code review
03178a0 
Co-Authored-By: Tim Peters <tim.peters@gmail.com>
@pablogsal
Fix more typos
6d01e46
@pablogsal
Address Petr feedback and fix more typos
4321593
@pablogsal
Copy link
Member Author

@pablogsal pablogsal commented Jan 21, 2020 •
edited

This is getting very big already so am going to go ahead and land it and future improvements/fixes can be made as separate PRs.

Thanks a lot to everyone involved for your useful comments, reviews and suggestions 🎉
@pablogsal pablogsal merged commit d43d6e9 into python:master Jan 21, 2020
6 checks passed
6 checks passed
@netlify
Header rules No header rules processed
Details
@netlify
Pages changed 3 new files uploaded
Details
@netlify
Redirect rules No redirect rules processed
Details
@netlify
Mixed content No mixed content detected
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@netlify
deploy/netlify Deploy preview ready!
Details
@pablogsal pablogsal deleted the pablogsal:gc branch Jan 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@encukou encukou

@tim-one tim-one

@terryjreedy terryjreedy

@brettcannon brettcannon

@willingc willingc

@JulienPalard JulienPalard

Assignees

@pablogsal pablogsal

Labels
CLA signed
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
@pablogsal @brettcannon @encukou @tim-one @terryjreedy @the-knights-who-say-ni
X Tutup