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 #16337: [ROCM] disable setting workspace on ROCM #16369

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

copybara-service[bot]
Copy link

PR #16337: [ROCM] disable setting workspace on ROCM

Imported from GitHub PR #16337

This is a follow-up PR for #16149 which was closed with unmerged commits.

Here I disable setting workspace on ROCM. This feature seems to produce sporadic errors because rocblas_set_workspace uses blocking memory functions (e.g. hipMalloc/hipFree) to manipulate scratch memory (which do not work under stream capture). Disabled until we have a clear idea on how to make it working on ROCM.

@xla-rotation: could you have a look please ?
Copybara import of the project:

--
6f42f69 by Pavel Emeliyanenko [email protected]:

disable workspace

Merging this change closes #16337

FUTURE_COPYBARA_INTEGRATE_REVIEW=#16337 from ROCm:ci_rocblas_disable_workspace 6f42f69

@copybara-service copybara-service bot force-pushed the test_666501268 branch 2 times, most recently from ced13a2 to c9dfd03 Compare August 22, 2024 22:50
Imported from GitHub PR #16337

This is a follow-up PR for #16149 which was closed with unmerged commits.

Here I disable setting workspace on ROCM. This feature seems to produce sporadic errors because rocblas_set_workspace uses blocking memory functions (e.g. hipMalloc/hipFree) to manipulate scratch memory (which do not work under stream capture). **Disabled** until we have a clear idea on how to make it working on ROCM.

@xla-rotation: could you have a look please ?
Copybara import of the project:

--
6f42f69 by Pavel Emeliyanenko <[email protected]>:

disable workspace

Merging this change closes #16337

COPYBARA_INTEGRATE_REVIEW=#16337 from ROCm:ci_rocblas_disable_workspace 6f42f69
PiperOrigin-RevId: 666524181
@copybara-service copybara-service bot merged commit 119d74c into main Aug 22, 2024
@copybara-service copybara-service bot deleted the test_666501268 branch August 22, 2024 23:19
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