Skip to content

[SYCL][Bindless] Add external_mem_handle_type::dma_buf #18988

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

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

Conversation

ProGTX
Copy link
Contributor

@ProGTX ProGTX commented Jun 16, 2025

  • Add support for importing dma_buf handles into SYCL
  • Add supports_importing_handle_type that checks whether a device supports importing an external memory handle type. This is because dma_buf is only supported on L0 backend for now.
  • Updated Bindless Images extension document
  • Updated "Example 6" E2E test to use dma_buf handle
  • Updated buffer_usm Vulkan interop E2E test to use dma_buf handle if the both the Vulkan driver and the device backend supports it
  • Added overload of get_ur_handles that only takes a SYCL device

* Add support for importing `dma_buf` handles into SYCL
* Add `supports_importing_handle_type` that checks whether a device
  supports importing an external memory handle type.
  This is because `dma_buf` is only supported on L0 backend for now.
* Updated Bindless Images extension document
* Updated "Example 6" E2E test to use `dma_buf` handle
* Updated `buffer_usm` Vulkan interop E2E test to use `dma_buf` handle
  if the both the Vulkan driver and the device backend supports it
* Added overload of `get_ur_handles` that only takes a SYCL device
@ProGTX ProGTX requested review from a team as code owners June 16, 2025 10:39
@ProGTX ProGTX temporarily deployed to WindowsCILock June 16, 2025 10:39 — with GitHub Actions Inactive
@ProGTX ProGTX temporarily deployed to WindowsCILock June 16, 2025 11:03 — with GitHub Actions Inactive
@ProGTX ProGTX temporarily deployed to WindowsCILock June 16, 2025 11:03 — with GitHub Actions Inactive
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