Skip to content

gh-99113: A Per-Interpreter GIL! #104209

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

Closed
ericsnowcurrently wants to merge 17 commits into from
Closed

gh-99113: A Per-Interpreter GIL! #104209

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
2aad06a
Add a comment.
ericsnowcurrently Sep 12, 2022
e55fbdd
_gil_runtime_state -> _gil_state.
ericsnowcurrently Sep 12, 2022
711b19d
Pass PyInterpreterState to _PyEval_ThreadInitialized().
ericsnowcurrently Sep 12, 2022
dba4460
runtime->main -> runtime->interpreters.main.
ericsnowcurrently Sep 12, 2022
5342672
Pass PyInterpreterState to _PyEval_InitState().
ericsnowcurrently Sep 12, 2022
f2361d7
Label some struct fields.
ericsnowcurrently Sep 12, 2022
adc09a0
Add _get_gil().
ericsnowcurrently Sep 12, 2022
7014bff
Pass the runtime to _PyEval_InitRuntimeState().
ericsnowcurrently Sep 12, 2022
2026034
Add some comments.
ericsnowcurrently Sep 12, 2022
3ce9695
Add _GET_OWN_GIL().
ericsnowcurrently Sep 12, 2022
57ca2fa
Pass PyInterpreterState to _PyEval_FiniState().
ericsnowcurrently Sep 12, 2022
936c86d
Call _gil_initialize() in _PyEval_InitState().
ericsnowcurrently Sep 12, 2022
66ca3e4
Drop _PyEval_InitRuntimeState().
ericsnowcurrently Sep 12, 2022
6d09320
Move the GIL to PyInterpreterState (but still share).
ericsnowcurrently Sep 12, 2022
5075992
Keep sharing the GIL.
ericsnowcurrently Sep 12, 2022
f39c883
Check for GIL ownership.
ericsnowcurrently Sep 12, 2022
ae2eb3c
Stop sharing the GIL if configured as "isolated".
ericsnowcurrently Sep 12, 2022
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
runtime->main -> runtime->interpreters.main.
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed Sep 12, 2022
commit dba446005c301df7c5482f53de79a298c09acb71
2 changes: 1 addition & 1 deletion Python/ceval_gil.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -465,7 +465,7 @@ int
_PyEval_ThreadsInitialized(PyInterpreterState *interp)
{
if (interp == NULL) {
interp = &_PyRuntime.main;
interp = _PyRuntime.interpreters.main;
if (interp == NULL) {
return 0;
}
Expand Down
3 changes: 2 additions & 1 deletion Python/pystate.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1679,7 +1679,8 @@ PyGILState_Ensure(void)

/* Ensure that _PyEval_InitThreads() has been called by Py_Initialize() */
// XXX Use the appropriate interpreter.
assert(runtime->main && _PyEval_ThreadsInitialized(runtime->main));
assert(runtime->interpreters.main &&
_PyEval_ThreadsInitialized(runtime->interpreters.main));
/* Ensure that _PyGILState_Init() has been called by Py_Initialize() */
assert(gilstate->autoInterpreterState);

Expand Down