python / python/cpython

Heap types implemented in C must have Py_TPFLAGS_HAVE_GC and implement tp_traverse

Ouverte
#116,946 27 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

extension-modules type-bug
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

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. 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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.