Skip to content

Handle CUDA_ERROR_DEINITIALIZED error in cu2umf_result() #1184

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
Merged
merged 1 commit into from
Mar 12, 2025
Merged

Handle CUDA_ERROR_DEINITIALIZED error in cu2umf_result() #1184

merged 1 commit into from
Mar 12, 2025

Conversation

ldorau
Copy link
Contributor

@ldorau ldorau commented Mar 12, 2025 •
edited
Loading

Description

Handle CUDA_ERROR_DEINITIALIZED error in cu2umf_result().

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly

@ldorau ldorau requested a review from a team as a code owner March 12, 2025 13:04
@ldorau ldorau changed the title Handle CUDA_ERROR_DEINITIALIZED error in cu2umf_result() Handle CUDA_ERROR_DEINITIALIZED error in cu2umf_result() Mar 12, 2025
@ldorau
Copy link
Contributor Author

ldorau commented Mar 12, 2025

@bratpiorka @lukaszstolarczuk it is required for the CUDA fix: intel/llvm#17411

@ldorau
Copy link
Contributor Author

ldorau commented Mar 12, 2025

Do not merge. Log fatal should be removed

@lukaszstolarczuk lukaszstolarczuk marked this pull request as draft March 12, 2025 15:14
@ldorau ldorau force-pushed the Handle_CUDA_ERROR_DEINITIALIZED_error_in_cu2umf_result branch from f28a459 to 4cba219 Compare March 12, 2025 15:42
@ldorau
Copy link
Contributor Author

ldorau commented Mar 12, 2025

Do not merge. Log fatal should be removed

LOG_FATAL() replaced with LOG_ERR().
It can be merged now.

@ldorau ldorau marked this pull request as ready for review March 12, 2025 15:42
@ldorau ldorau merged commit 9abb40d into oneapi-src:main Mar 12, 2025
159 of 160 checks passed
@ldorau ldorau deleted the Handle_CUDA_ERROR_DEINITIALIZED_error_in_cu2umf_result branch March 12, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bratpiorka bratpiorka bratpiorka approved these changes

@lplewa lplewa lplewa approved these changes

@lukaszstolarczuk lukaszstolarczuk lukaszstolarczuk approved these changes

@PatKamin PatKamin PatKamin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ldorau @bratpiorka @lplewa @lukaszstolarczuk @PatKamin