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

[L0] Update L0 Init checking to print details in error log #2449

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

nrspruit
Copy link
Contributor

  • Changed the L0 Init failures to goto the error log vs debug and always return an error to the error log if no init is possible.
  • Enabled for calling zeParseError to generate the string of L0 error codes in all UR L0 code.

@nrspruit nrspruit requested review from a team as code owners December 11, 2024 19:27
@github-actions github-actions bot added conformance Conformance test suite issues. level-zero L0 adapter specific issues labels Dec 11, 2024
@nrspruit nrspruit added the v0.11.x Include in the v0.11.x release label Dec 11, 2024
@nrspruit nrspruit force-pushed the improve_l0_init_checking branch 3 times, most recently from 13cb355 to a66e000 Compare December 12, 2024 17:04
nrspruit added a commit to nrspruit/llvm that referenced this pull request Dec 12, 2024
-pre-commit PR for oneapi-src/unified-runtime#2449

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
- Changed the L0 Init failures to goto the error log vs debug and always
  return an error to the error log if no init is possible.
- Enabled for calling zeParseError to generate the string of L0 error
  codes in all UR L0 code.

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
@nrspruit nrspruit force-pushed the improve_l0_init_checking branch from a66e000 to 64ae812 Compare December 13, 2024 20:13
nrspruit added a commit to nrspruit/llvm that referenced this pull request Dec 13, 2024
-pre-commit PR for oneapi-src/unified-runtime#2449

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
@nrspruit nrspruit added ready to merge Added to PR's which are ready to merge and removed conformance Conformance test suite issues. labels Dec 13, 2024
@kbenzie kbenzie merged commit 94716fc into oneapi-src:main Dec 16, 2024
73 checks passed
kbenzie added a commit that referenced this pull request Dec 16, 2024
[L0] Update L0 Init checking to print details in error log
sarnex pushed a commit to intel/llvm that referenced this pull request Dec 16, 2024
Pulls in the following bug fixes:

* oneapi-src/unified-runtime#2447
* oneapi-src/unified-runtime#2449
* oneapi-src/unified-runtime#2452
* oneapi-src/unified-runtime#2452

---------

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
Co-authored-by: Kenneth Benzie (Benie) <k.benzie@codeplay.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
level-zero L0 adapter specific issues ready to merge Added to PR's which are ready to merge v0.11.x Include in the v0.11.x release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants