Skip to content

[SYCL] Switch Level Zero plugin to specification v1.0 #2408

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

Closed
wants to merge 7 commits into from

Conversation

againull
Copy link
Contributor

@againull againull commented Sep 1, 2020

No description provided.

smaslov-intel and others added 7 commits September 1, 2020 14:08
- Added caching of L0 command lists per pi device
- Command list created with an associated fence to track
  when the command list has successfully executed by the command queue
- Added SYCL_PI_LEVEL0_MAX_COMMAND_LIST_CACHE environment variable
  to set the maximum cache size (default: 20000)
- Fixed issue where command list reuse between queues failed to properly
add a new fence to the map.

Signed-off-by: Spruit, Neil R <[email protected]>
Signed-off-by: Byoungro So <[email protected]>
Add temporary support to enable online linking with the Level Zero
driver if the SYCL_ENABLE_LEVEL_ZERO_LINK environment variable is
set.

At the time this code was written, the Level Zero driver online
linking APIs exist, but they don't work.  We think the DPC++ runtime
support is ready, though, and we can enable it via this environment
variable.  This will allow testing with a new driver if one is
available before the next DPC++ release.
Copy link
Contributor

@smaslov-intel smaslov-intel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but it should be merged with the uplift of L0 version in CI.

@againull
Copy link
Contributor Author

againull commented Sep 4, 2020

This branch was merged in scope of #2409

@againull againull closed this Sep 4, 2020
@againull againull deleted the level_zero_v1 branch December 3, 2022 00:03
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.

6 participants