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
Copy link
Copy Markdown
Member

@vstinner vstinner commented Jul 27, 2018

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.
Copy link
Copy Markdown
Member

@pablogsal pablogsal left a comment

Choose a reason for hiding this comment

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

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