Heap types implemented in C must have Py_TPFLAGS_HAVE_GC and implement tp_traverse
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 77.2k
- Forks
- 35.9k
- Métriques de merge des PR
- Métriques de PR en attente
Description
If a heap type does not have Py_TPFLAGS_HAVE_GC or doesn't not implement tp_traverse, the garbage collector may fail to collect the type and most of its instances at Python exit. The problem is worse when sub-interpreters are involved: if a sub-interpreter exits and does not clear all of its memory, the other interpreters will continue running with these leaks.
See issue gh-87138 for the rationale.
I wrote an article on the lock type which had this bug and caused a memory leak: https://vstinner.github.io/subinterpreter-leaks.html
Linked PRs
- gh-116947
- gh-138290
- gh-138327
- gh-138328
- gh-138331
- gh-138333
- gh-138427
- gh-138428
- gh-138334
- gh-138340
- gh-138341
- gh-138582
- gh-138807
- gh-139073
- gh-139474
- gh-141268
- gh-141448
- gh-152310
- gh-156034
The following PRs were directly closed because the types are immutable and empty:
- gh-138329
- gh-138330
- gh-138332
The following PRs were wrongly merged so we reverted them:
- gh-138266
- gh-138320
- gh-138322
- gh-138288
- gh-138321
- gh-138323
- gh-138289
- gh-138324
- gh-138326
PRs for reverting the above commits:
- gh-138338
- gh-138337
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par issue gh-87138 et l’article lié sur le type de lock, puis dressez l’inventaire des types heap C couverts par les PR liés. C’est terminé lorsque les types heap concernés possèdent Py_TPFLAGS_HAVE_GC et tp_traverse, et que les fuites décrites à la sortie de l’interpréteur et dans les sous-interpréteurs sont corrigées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- c, python
- Domaine
- backend
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 25/100