Skip to content

[UR][L0] fix external semaphore with updated headers and report device info support #17286

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

Merged
merged 1 commit into from
Mar 5, 2025

Conversation

nrspruit
Copy link
Contributor

@nrspruit nrspruit commented Mar 3, 2025

No description provided.

@nrspruit nrspruit requested review from a team as code owners March 3, 2025 23:04
@DBDuncan
Copy link
Contributor

DBDuncan commented Mar 4, 2025

Thanks for the PR! Do you know what is the current status of L0 driver semaphore interop support? I understand that support in windows is more mature then linux? Should timeline semaphores be expected to work?

I am mostly done with a PR to add support for timeline semaphore interop with Vulkan. The Cuda side is working. On the L0 side I have made the changes it should need but have had trouble with the drivers on linux. Seen L0 return the semaphore interop extension as supported but when using the semaphore interop functions it returns unsupported.

@nrspruit
Copy link
Contributor Author

nrspruit commented Mar 4, 2025

Thanks for the PR! Do you know what is the current status of L0 driver semaphore interop support? I understand that support in windows is more mature then linux? Should timeline semaphores be expected to work?

I am mostly done with a PR to add support for timeline semaphore interop with Vulkan. The Cuda side is working. On the L0 side I have made the changes it should need but have had trouble with the drivers on linux. Seen L0 return the semaphore interop extension as supported but when using the semaphore interop functions it returns unsupported.

Timeline semaphores work has not started yet for L0, we will need to track that separately for a future release.

The driver that supports external semaphore api calls in L0 is:
Intel® Graphics Driver 32.0.101.6632 link: https://www.intel.com/content/www/us/en/download/785597/848516/intel-arc-iris-xe-graphics-windows.html

I can see if I can update the CI driver version in a future update since that could affect more than this change.

@nrspruit
Copy link
Contributor Author

nrspruit commented Mar 4, 2025

All tests are passing related to this change, the only failure is unrelated and a known issue here: #17275

@nrspruit nrspruit force-pushed the offical_external_semaphore branch from 30f9009 to 497c834 Compare March 4, 2025 17:22
@nrspruit
Copy link
Contributor Author

nrspruit commented Mar 4, 2025

@intel/unified-runtime-reviewers please review and if possible approve so this can be merged to update this interface. Thank you for your time!

@againull againull merged commit 60ffdc3 into intel:sycl Mar 5, 2025
30 checks passed
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.

4 participants