FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.12] gh-104372: Drop the GIL around the vfork() call. (GH-104782) by miss-islington · Pull Request #104942 · python/cpython · GitHub

/ cpython Public

[3.12] gh-104372: Drop the GIL around the vfork() call. (GH-104782) - #104942

Merged
gpshead merged 1 commit into
python:3.12from
miss-islington:backport-d086792-3.12
May 25, 2023
Merged

[3.12] gh-104372: Drop the GIL around the vfork() call. (GH-104782)#104942
gpshead merged 1 commit into
python:3.12from
miss-islington:backport-d086792-3.12

Conversation

miss-islington commented May 25, 2023
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

On Linux where the subprocess module can use the vfork syscall for
faster spawning, prevent the parent process from blocking other threads
by dropping the GIL while it waits for the vfork'ed child process exec
outcome. This prevents spawning a binary from a slow filesystem from
blocking the rest of the application.

Fixes GH-104372.
(cherry picked from commit d086792)

Co-authored-by: Gregory P. Smith gps@python.org


📚 Documentation preview 📚: https://cpython-previews--104942.org.readthedocs.build/

On Linux where the `subprocess` module can use the `vfork` syscall for
faster spawning, prevent the parent process from blocking other threads
by dropping the GIL while it waits for the vfork'ed child process `exec`
outcome.  This prevents spawning a binary from a slow filesystem from
blocking the rest of the application.

Fixes pythonGH-104372.
(cherry picked from commit d086792)

Co-authored-by: Gregory P. Smith <gps@python.org>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL