Skip to content

[3.5] bpo-33329: Fix multiprocessing regression on newer glibcs (GH-6…#12144

Merged
larryhastings merged 1 commit into
python:3.5from
csabella:backport-25038ec-3.5
Mar 4, 2019
Merged

[3.5] bpo-33329: Fix multiprocessing regression on newer glibcs (GH-6…#12144
larryhastings merged 1 commit into
python:3.5from
csabella:backport-25038ec-3.5

Conversation

@csabella

@csabella csabella commented Mar 3, 2019

Copy link
Copy Markdown
Contributor

…575)

Starting with glibc 2.27.9000-xxx, sigaddset() can return EINVAL for some
reserved signal numbers between 1 and NSIG. The range(1, NSIG) idiom
is commonly used to select all signals for blocking with pthread_sigmask.
So we ignore the sigaddset() return value until we expose sigfillset()
to provide a better idiom.
(cherry picked from commit 25038ec)

Co-authored-by: Antoine Pitrou pitrou@free.fr

https://bugs.python.org/issue33329

…onGH-6575)

Starting with glibc 2.27.9000-xxx, sigaddset() can return EINVAL for some
reserved signal numbers between 1 and NSIG.  The `range(1, NSIG)` idiom
is commonly used to select all signals for blocking with `pthread_sigmask`.
So we ignore the sigaddset() return value until we expose sigfillset()
to provide a better idiom.
(cherry picked from commit 25038ec)

Co-authored-by: Antoine Pitrou <pitrou@free.fr>
@bedevere-bot bedevere-bot added the type-bug An unexpected behavior, bug, or error label Mar 3, 2019

@larryhastings larryhastings left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@bedevere-bot

Copy link
Copy Markdown

@larryhastings: Please replace # with GH- in the commit message next time. Thanks!

@larryhastings

Copy link
Copy Markdown
Contributor

Thanks for the backport, Cheryl, and for the review, Miro!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants