Revert: bpo-33608: Factor out a private, per-interpreter _Py_AddPendingCall(). (GH-11617)

[vstinner]

• Revert "bpo-33608: Simplify ceval's DISPATCH by hoisting eval_breaker ahead of time. (bpo-33608: Simplify DISPATCH by hoisting eval_breaker ahead of time. #12062)"
• Revert "bpo-33608: Use _Py_AddPendingCall() in _PyCrossInterpreterData_Release(). (bpo-33608: Use _Py_AddPendingCall() in _PyCrossInterpreterData_Release(). #12024)"
• Revert "bpo-33608: Factor out a private, per-interpreter _Py_AddPendingCall(). (bpo-33608: Factor out a private, per-interpreter _Py_AddPendingCall(). #11617)"

https://bugs.python.org/issue33608

[vstinner]

The world hates me: I wrote this PR to fix https://bugs.python.org/issue36114 race condition, but test__xxsubinterpreters failed on AppVeyor :-(

https://ci.appveyor.com/project/python/cpython/builds/22802115

test_bad_id (test.test__xxsubinterpreters.ChannelIDTests) ... Fatal Python error: Py_EndInterpreter: thread still has a frame
Current thread 0x00001338 (most recent call first):
  File "C:\projects\cpython\lib\test\test__xxsubinterpreters.py", line 297 in tearDown
  File "C:\projects\cpython\lib\unittest\case.py", line 645 in run
  File "C:\projects\cpython\lib\unittest\case.py", line 702 in __call__
  File "C:\projects\cpython\lib\unittest\suite.py", line 122 in run
  File "C:\projects\cpython\lib\unittest\suite.py", line 84 in __call__
  File "C:\projects\cpython\lib\unittest\suite.py", line 122 in run
  File "C:\projects\cpython\lib\unittest\suite.py", line 84 in __call__
  File "C:\projects\cpython\lib\unittest\suite.py", line 122 in run
  File "C:\projects\cpython\lib\unittest\suite.py", line 84 in __call__
  File "C:\projects\cpython\lib\unittest\runner.py", line 176 in run
  File "C:\projects\cpython\lib\test\support\__init__.py", line 1968 in _run_suite
  File "C:\projects\cpython\lib\test\support\__init__.py", line 2064 in run_unittest
  File "C:\projects\cpython\lib\test\libregrtest\runtest.py", line 178 in test_runner
  File "C:\projects\cpython\lib\test\libregrtest\runtest.py", line 182 in runtest_inner
  File "C:\projects\cpython\lib\test\libregrtest\runtest.py", line 137 in runtest
  File "C:\projects\cpython\lib\test\libregrtest\main.py", line 305 in rerun_failed_tests
  File "C:\projects\cpython\lib\test\libregrtest\main.py", line 623 in _main
  File "C:\projects\cpython\lib\test\libregrtest\main.py", line 586 in main
  File "C:\projects\cpython\lib\test\libregrtest\main.py", line 640 in main
  File "C:\projects\cpython\lib\test\__main__.py", line 2 in <module>
  File "C:\projects\cpython\lib\runpy.py", line 85 in _run_code
  File "C:\projects\cpython\lib\runpy.py", line 192 in _run_module_as_main
C:\projects\cpython>set lastexitcode=-1073740791 

[vstinner]

My intent is to revert the commit ef4ac96, but sadly I had to also revert 3 commits written by Eric Snow which are based on this commit :-( I would prefer to revert a single commit, but I had to revert 4 commits :-(

[vstinner]

This PR reverts the commit ef4ac96 because it introduced multiple regressions:

• test_multiprocessing_spawn started to produce coredumps on FreeBSD: https://bugs.python.org/issue36114
• test_multiprocessing_spawn started to fail randomly on Windows: https://bugs.python.org/issue36116
• test_io started to fail randomly on Windows: https://bugs.python.org/issue36177

[vstinner]

I tested manually this PR and I confirm that it fix https://bugs.python.org/issue36114

I stressed my FreeBSD 12 VM with test_multiprocessing_spawn:

• Without this change, I can reliably reproduce https://bugs.python.org/issue36114 crash
• With this change, WithThreadsTestManagerRestart.test_rapid_restart() doesn't crash anymore.