[UR][L0v2] Introduce urGraphGetIdExp to provide unique graph identifiers#22418
[UR][L0v2] Introduce urGraphGetIdExp to provide unique graph identifiers#22418mmichel11 wants to merge 3 commits into
Conversation
fff8797 to
8036347
Compare
8036347 to
17e74a5
Compare
…ry corruption in SYCL tests
fc1e68e to
ebcf935
Compare
| // that predate its support leads to memory corruption errors in | ||
| // sycl/test-e2e/Adapters/level_zero/ tests, so manually check the driver | ||
| // version until the issue is resolved. | ||
| if (this->isDriverVersionNewerOrSimilar(1, 15, 38921)) { |
There was a problem hiding this comment.
I first observed these non-graph test failures in: https://github.com/intel/llvm/actions/runs/28179054588 where the driver does not yet support zeGraphGetIdExp. I reproduced this issue locally and confirmed the affected tests pass once I use a driver that supports zeGraphGetIdExt and skip the zeDriverGetExtensionFunctionAddress call for older drivers.
There's an issue to be filed to L0 which seems to be the source of this memory corruption, but this check provides us a workaround for now.
|
The added UR test is skipped in CI as the driver does not yet contain the new L0 graph function. Here are the results from a local BMG run with a new driver build: urGraphGetIdExp L0v2 BMGThe MacOS build and |
bratpiorka
left a comment
There was a problem hiding this comment.
CUDA/HIP/OpenCL/native cpu LGTM
urGraphGetIdExpwhich internally invokeszeGraphGetIdExtto align SYCL native graph IDs with L0