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.

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

PR #21213: [GPU] Fix mutex locking of a cuDNN handle. #21303

Merged
merged 1 commit into from
Jan 11, 2025

Conversation

copybara-service[bot]
Copy link

PR #21213: [GPU] Fix mutex locking of a cuDNN handle.

Imported from GitHub PR #21213

The CudnnHandle object containing a mutex has to stay alive while cudnnHandle_t it guards is in use. This brings the use in sync with the other uses in this file. There is no evidence that this caused failures so far, rather prefetching potential problems, therefore no test added.
Copybara import of the project:

--
0472972 by Ilia Sergachev [email protected]:

[GPU] Fix mutex locking of a cuDNN handle.

The CudnnHandle object containing a mutex has to stay alive while
cudnnHandle_t it guards is in use. This brings the use in sync with the
other uses in this file. There is no evidence that this caused failures
so far, rather prefetching potential problems, therefore no test added.

Merging this change closes #21213

FUTURE_COPYBARA_INTEGRATE_REVIEW=#21213 from openxla:fix_cudnn_locking 0472972

Imported from GitHub PR #21213

The CudnnHandle object containing a mutex has to stay alive while cudnnHandle_t it guards is in use. This brings the use in sync with the other uses in this file. There is no evidence that this caused failures so far, rather prefetching potential problems, therefore no test added.
Copybara import of the project:

--
0472972 by Ilia Sergachev <[email protected]>:

[GPU] Fix mutex locking of a cuDNN handle.

The CudnnHandle object containing a mutex has to stay alive while
cudnnHandle_t it guards is in use. This brings the use in sync with the
other uses in this file. There is no evidence that this caused failures
so far, rather prefetching potential problems, therefore no test added.

Merging this change closes #21213

COPYBARA_INTEGRATE_REVIEW=#21213 from openxla:fix_cudnn_locking 0472972
PiperOrigin-RevId: 714275180
@copybara-service copybara-service bot merged commit f03d1f6 into main Jan 11, 2025
@copybara-service copybara-service bot deleted the test_713940694 branch January 11, 2025 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant