Skip to content
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

Jump to bottom

[3.5] bpo-30320, bpo-25277: backport test_eintr enhancements from master to 3.5 #1532

Merged
merged 4 commits into from
May 10, 2017
Merged

[3.5] bpo-30320, bpo-25277: backport test_eintr enhancements from master to 3.5 #1532

merged 4 commits into from
May 10, 2017

Conversation

vstinner
Copy link
Member

@vstinner
bpo-30320: test_eintr now uses pthread_sigmask() (#1523)
6922a52 
Rewrite sigwaitinfo() and sigtimedwait() unit tests for EINTR using
pthread_sigmask() to fix a race condition between the child and the
parent process.

Remove the pipe which was used as a weak workaround against the race
condition.

sigtimedwait() is now tested with a child process sending a signal
instead of testing the timeout feature which is more unstable
(especially regarding to clock resolution depending on the platform).
(cherry picked from commit 211a392)
@vstinner
test_eintr: Fix ResourceWarning warnings
1b6a7f2 
(cherry picked from commit c50cccf)
@vstinner
test_eintr: remove unused import
0e6b061
@vstinner
bpo-25277: Add a watchdog to test_eintr
4c29a01 
Set a timeout of 10 minutes in test_eintr using faulthandler.
@vstinner vstinner merged commit dab10f4 into python:3.5 May 10, 2017
@vstinner vstinner deleted the eintr_35 branch June 15, 2017 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vstinner @the-knights-who-say-ni