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

Enable creation of bindless images backed by host USM #2551

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

przemektmalon
Copy link
Contributor

@przemektmalon przemektmalon commented Jan 13, 2025

Small patch to enable bindless images backed by host USM in the CUDA adapter.

Host and Device USM pointers are usable across the host and device for all versions of CUDA that we support. There is no need to provide the CU_MEMHOSTALLOC_DEVICEMAP flag during allocation, or calling cuMemHostGetDevicePointer to retrieve a device usable address.

Passing a CU_MEMHOSTALLOC_WRITECOMBINED flag to the host USM allocation will enhance performance in certain scenarios, however, an extension allowing this is not yet available.

Related DPC++ PR: intel/llvm#16607

@przemektmalon przemektmalon requested a review from a team as a code owner January 13, 2025 16:18
@github-actions github-actions bot added images UR images cuda CUDA adapter specific issues labels Jan 13, 2025
Small patch to enable bindless images backed by host USM in the CUDA
adapter.

Host and Device USM pointers are usable across the host and device
for all versions of CUDA that we support. There is no need to provide
the `CU_MEMHOSTALLOC_DEVICEMAP` flag during allocation, or calling
`cuMemHostGetDevicePointer` to retrieve a device usable address.

Passing a `CU_MEMHOSTALLOC_WRITECOMBINED` flag to the host USM
allocation will enhance performance in certain scenarios, however, an
extension allowing this is not yet available.
@przemektmalon przemektmalon force-pushed the przemek/bindless-images-host-usm branch from 7005365 to d49c8cd Compare January 20, 2025 10:31
@przemektmalon przemektmalon added the ready to merge Added to PR's which are ready to merge label Jan 20, 2025
@kbenzie kbenzie removed the ready to merge Added to PR's which are ready to merge label Jan 20, 2025
@kbenzie
Copy link
Contributor

kbenzie commented Jan 20, 2025

I've removed ready to merge as intel/llvm#16607 still requires approvals (except from unified-runtime-reviewers) and must have all checks passing. Please add the lable again once these conditions are satisfied.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuda CUDA adapter specific issues images UR images
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants