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

[Windows][CI][L0] Drivers do not support zeCommandListHostSynchronize #12344

Closed
Bensuo opened this issue Jan 10, 2024 · 1 comment
Closed

[Windows][CI][L0] Drivers do not support zeCommandListHostSynchronize #12344

Bensuo opened this issue Jan 10, 2024 · 1 comment
Labels
bug Something isn't working infrastructure Windows

Comments

@Bensuo
Copy link
Contributor

Bensuo commented Jan 10, 2024

zeCommandListHostSynchronize is used by the runtime when immediate command lists are in use to synchronize SYCL kernel operations (such as when calling queue::wait()). However, on the drivers currently in use on CI (L0 runtime version 1.3.26370 reported from sycl-ls) this does not seem to function correctly and results in synchronization errors in E2E tests.

Local testing with similarly aged drivers and testing with ZE_DEBUG shows that this function is returning ZE_RESULT_ERROR_UNSUPPORTED_FEATURE. When updating to the latest windows drivers (1.3.27616) this is no longer the case and the function is supported. It appears that this entry-point was added in L0 1.6.0 and should be supported by this compute-runtime release at the earliest: https://github.com/intel/compute-runtime/releases/tag/23.22.26516.18

This issue was first seen here in this CI run for PR #12279 . This PR adds additional runs for many graph E2E tests which force immediate command list usage, however this issue is not graph specific. That PR includes a non-graph E2E test which was used to confirm and identify this issue here: https://github.com/intel/llvm/blob/9f35877bbe8776fee62bcc6f23df2d81203a4535/sycl/test-e2e/USM/queue_copy.cpp

@Bensuo Bensuo added the bug Something isn't working label Jan 10, 2024
martygrant pushed a commit that referenced this issue May 15, 2024
)

These tests had been disabled as part of
#12344 due to the windows level-zero
drivers being out of date on CI. This PR re-enables the disabled tests
since the driver version has since been updated and the tests are now
passing.
@Bensuo
Copy link
Contributor Author

Bensuo commented May 22, 2024

It seems that the Windows drivers have now been updated to a version which supports this call, and the tests were re-enabled for immediate command lists in #13685.

Resolving this issue now.

@Bensuo Bensuo closed this as completed May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working infrastructure Windows
Projects
None yet
Development

No branches or pull requests

1 participant