Skip to content

gh-117953: Always Run Extension Init Func in Main Interpreter First #118157

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ericsnowcurrently merged 18 commits into from
May 7, 2024
Merged

gh-117953: Always Run Extension Init Func in Main Interpreter First #118157

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
a1a3b4c
Load extension modules under the main interpreter first.
ericsnowcurrently Feb 8, 2024
9d630f1
Tweak a comment.
ericsnowcurrently Apr 16, 2024
2d7ee11
Do not switch if already on the main interpreter.
ericsnowcurrently Apr 16, 2024
f9e132f
maybe_switch_to_main_interpreter() -> switch_to_main_interpreter()
ericsnowcurrently Apr 17, 2024
b09c445
Update the global cache while still under the main interpreter.
ericsnowcurrently May 6, 2024
8ca8033
Add a NEWS entry.
ericsnowcurrently May 6, 2024
122761c
Fix the main_tstate checks.
ericsnowcurrently May 6, 2024
bb3402a
Switch back if there is an exception.
ericsnowcurrently May 6, 2024
164a9eb
Cache the module only on success.
ericsnowcurrently May 6, 2024
0c74b25
Add a missing assert.
ericsnowcurrently May 6, 2024
7b9a29d
Clean up the temprary thread state.
ericsnowcurrently May 6, 2024
7965d57
Do not do an extra _Py_IsMainInterpreter() check.
ericsnowcurrently May 6, 2024
2abce96
Drop the superfluous SinglephaseInitTests.tearDownClass().
ericsnowcurrently May 6, 2024
0b16f3f
Work around a refleak.
ericsnowcurrently May 7, 2024
f7d0ff9
Skip some tests when the GIL is disabled.
ericsnowcurrently May 7, 2024
f3b015f
Merge branch 'main' into load-ext-in-main-interp-2
ericsnowcurrently May 7, 2024
048342f
Fix the tests.
ericsnowcurrently May 7, 2024
6d5311e
Disable all interpreters tests under Py_GIL_DISABLED.
ericsnowcurrently May 7, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add a NEWS entry.
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed May 6, 2024
commit 8ca803341bf343f08157f6edb38e118b762d06b6
8 changes: 8 additions & 0 deletions Misc/NEWS.d/next/Core and Builtins/2024-05-06-10-57-54.gh-issue-117953.DqCzIs.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
When a builtin or extension module is imported for the first time, while a
subinterpreter is active, the module's init function is now run by the main
interpreter first before import continues in the subinterpreter.
Consequently, single-phase init modules now fail in an isolated
subinterpreter without the init function running under that interpreter,
whereas before it would run under the subinterpreter *before* failing,
potentially leaving behind global state and callbacks and otherwise leaving
the module in an inconsistent state.