-
Notifications
You must be signed in to change notification settings - Fork 218
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.
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
Make nonblocking synchronization robust to errors. #1369
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1369 +/- ##
==========================================
+ Coverage 77.98% 78.01% +0.03%
==========================================
Files 121 121
Lines 8929 8951 +22
==========================================
+ Hits 6963 6983 +20
- Misses 1966 1968 +2
Continue to review full report at Codecov.
|
a472a0d
to
3693ded
Compare
3693ded
to
bacd69a
Compare
@maleadt just tried out this branch, as suggested, on the application that was hanging, and all seem to run fine now with Julia 1.7.1 🙂 |
Make nonblocking synchronization robust to errors.
Make nonblocking synchronization robust to errors.
Our nonblocking synchronization relied on CUDA notifying an async condition, but that may never happen if the stream encounters an error. Protect against this by using a timer that periodically queries the stream in a regular way.
Fixes #1366, may reveal something in #1350.
cc @vchuravy @tkf