◐ Shell
reader mode source ↗
Skip to content

bpo-33613: multiprocessing semtracker uses sigmask#8500

Closed
vstinner wants to merge 1 commit into
python:masterfrom
vstinner:sem_tracker_block
Closed

bpo-33613: multiprocessing semtracker uses sigmask#8500
vstinner wants to merge 1 commit into
python:masterfrom
vstinner:sem_tracker_block

Conversation

@vstinner

@vstinner vstinner commented Jul 27, 2018

Copy link
Copy Markdown
Member

The multiprocessing semaphore tracker now uses
signal.pthread_sigmask() to block signals before spawning the child
to queue the signals before we modify the signal handlers to ignore
some signals.

https://bugs.python.org/issue33613

The multiprocessing semaphore tracker now uses
signal.pthread_sigmask() to block signals before spawning the child
to queue the signals before we modify the signal handlers to ignore
some signals.

@pablogsal pablogsal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

LGTM I can confirm that this works and solves the problem that #7850 is solving. Should I push the fix for the test in this PR and close #7850 or should we apply this one in #7850?

@vstinner

Copy link
Copy Markdown
Member Author

I prefer your PR since it also fix tests.

@vstinner vstinner closed this Jul 27, 2018
@vstinner vstinner deleted the sem_tracker_block branch July 27, 2018 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants