Skip to content

gh-118727: Don't drop the GIL in drop_gil() unless the current thread holds it #118745

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
colesbury merged 10 commits into from
May 23, 2024
Merged

gh-118727: Don't drop the GIL in drop_gil() unless the current thread holds it #118745

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
1d21956
Don't drop the GIL during thread deletion unless the current thread h…
swtaarrs May 7, 2024
b91a1d5
Store holds_gil flag on PyThreadState instead, and use it where appro…
swtaarrs May 8, 2024
402ab90
Merge remote-tracking branch 'upstream/main' into cpython-drop-gil
swtaarrs May 8, 2024
29c853d
Account for tstate == NULL case
swtaarrs May 8, 2024
694926f
Always track holds_gil, related cleanup
swtaarrs May 8, 2024
480c07d
Fix ASAN error, pull drop_gil_impl() back into drop_gil()
swtaarrs May 9, 2024
d28bf13
Rename thread_dying to final_release
swtaarrs May 9, 2024
43f64d7
Merge remote-tracking branch 'upstream/main' into cpython-drop-gil
swtaarrs May 9, 2024
d056a46
Restore drop_gil_impl(), clear GIL drop request when disabling GIL
swtaarrs May 13, 2024
f32a503
Merge remote-tracking branch 'upstream/main' into cpython-drop-gil
swtaarrs May 13, 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
Merge remote-tracking branch 'upstream/main' into cpython-drop-gil
  • Loading branch information
@swtaarrs
swtaarrs committed May 8, 2024
commit 402ab90f9f140a2ba35f4436eba26d1927eca3bf

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.