gh-110752: Reset ceval.eval_breaker to 0 in interpreter_clear (GH-110753)

gaogaotiantian · web-flow · commit 1e3460d9faaf · 2023-10-12T15:10:21.000+01:00
diff --git a/Misc/NEWS.d/next/Core and Builtins/2023-10-12-06-32-25.gh-issue-110752.FYfI0h.rst b/Misc/NEWS.d/next/Core and Builtins/2023-10-12-06-32-25.gh-issue-110752.FYfI0h.rst
@@ -0,0 +1 @@
+Reset ``ceval.eval_breaker`` in :func:`interpreter_clear`
diff --git a/Python/pystate.c b/Python/pystate.c
@@ -890,6 +890,10 @@ interpreter_clear(PyInterpreterState *interp, PyThreadState *tstate)
 
     Py_CLEAR(interp->audit_hooks);
 
+    // At this time, all the threads should be cleared so we don't need
+    // atomic operations for eval_breaker
+    interp->ceval.eval_breaker = 0;
+
     for (int i = 0; i < _PY_MONITORING_UNGROUPED_EVENTS; i++) {
         interp->monitors.tools[i] = 0;
     }

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+Reset ``ceval.eval_breaker` 8C8D ` in :func:`interpreter_clear`
Original file line number	Diff line number	Diff line change
`@@ -890,6 +890,10 @@ interpreter_clear(PyInterpreterState interp, PyThreadState tstate)`
`890`	`890`
`891`	`891`	`Py_CLEAR(interp->audit_hooks);`
`892`	`892`
	`893`	`+ // At this time, all the threads should be cleared so we don't need`
	`894`	`+ // atomic operations for eval_breaker`
	`895`	`+ interp->ceval.eval_breaker = 0;`
	`896`	`+`
`893`	`897`	`for (int i = 0; i < _PY_MONITORING_UNGROUPED_EVENTS; i++) {`
`894`	`898`	`interp->monitors.tools[i] = 0;`
`895`	`899`	`}`