python / python/cpython

Remote debugger should be checked only if interpreter paused on STW

Offen
#145,970 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

interpreter-core type-feature
Vorherrschende Sprache
Python
Sterne
77.2k
Forks
35.9k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

Feature or enhancement

Proposal:

Now it is checked every _Py_HandlePending call.

https://github.com/python/cpython/blob/1dfe99ae3bed6cac01732b47bf7fa637abadf51b/Python/ceval_gil.c#L1361-L1368

https://github.com/python/cpython/blob/1dfe99ae3bed6cac01732b47bf7fa637abadf51b/Python/ceval_gil.c#L1431-L1433

Should we move this under if ((breaker & _PY_EVAL_PLEASE_STOP_BIT) != 0)? If yes, I have a PR ready.

cc @pablogsal

Has this already been discussed elsewhere?

No response given

Links to previous discussion of this feature:

No response

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne in Python/ceval_gil.c an den referenzierten Aufrufstellen von _Py_HandlePending und vergleiche die Prüfung des Remote-Debuggers mit der Behandlung von _PY_EVAL_PLEASE_STOP_BIT. Bestätige das beabsichtigte Verhalten für Interpreter, die bei STW pausiert sind, und führe anschließend die relevanten CPython-Interpretertests aus oder ergänze eine Testabdeckung, die zeigt, dass die Prüfung nicht bei jedem Pfad zur Behandlung ausstehender Aufrufe durchgeführt wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
backend
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.