Skip to content

sycl/test-e2e/Plugin/interop-level-zero-image.cpp #13090

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.

Sign up for GitHub

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

Jump to bottom
Closed
ldrumm opened this issue Mar 21, 2024 · 2 comments
Closed

sycl/test-e2e/Plugin/interop-level-zero-image.cpp #13090

ldrumm opened this issue Mar 21, 2024 · 2 comments
Labels
bug Something isn't working confirmed

Comments

@ldrumm
Copy link
Contributor

ldrumm commented Mar 21, 2024 •
edited
Loading

Post-commit testing has been failing for multiple weeks. The root cause of this is sycl/test-e2e/Plugin/interop-level-zero-image.cpp which has been failing on the ARC GPUs consistently. This has clouded other test failures - why should anyone care about post-commit testing if it's always red?

We should disable this test to lower the noise level; it's too late to revert the patch that introduced it.
@ldrumm ldrumm added the bug Something isn't working label Mar 21, 2024
@ldrumm ldrumm mentioned this issue Mar 21, 2024
Merged
ldrumm added a commit to ldrumm/llvm that referenced this issue Mar 22, 2024
@ldrumm
[test-e2e] Disable broken test
e089800 
Post-commit testing has been failing for multiple weeks. The root cause
of this is sycl/test-e2e/Plugin/interop-level-zero-image.cpp which has
been failing on the ARC GPUs consistently. This has clouded other test
failures - why should anyone care about post-commit testing if it's
already red.

We should disable this test to lower the noise level; it's too late to
revert the patch that introduced it.

See intel#13090
ldrumm added a commit that referenced this issue Mar 22, 2024
@ldrumm
[test-e2e] Disable broken test (#13091)
6246e20 
Post-commit testing has been failing for multiple weeks. The root cause
of this is sycl/test-e2e/Plugin/interop-level-zero-image.cpp which has
been failing on the ARC GPUs consistently. This has clouded other test
failures - why should anyone care about post-commit testing if it's
already red.

We should disable this test to lower the noise level; it's too late to
revert the patch that introduced it.

See #13090
@ldrumm
Copy link
Contributor Author

ldrumm commented Mar 22, 2024 •
edited
Loading

disable this test: #13110

@ldrumm ldrumm mentioned this issue Mar 22, 2024
Merged
ldrumm added a commit that referenced this issue Mar 22, 2024
@ldrumm
[test-e2e] Disable broken test part 2 (#13110)
6bef29a 
6246e20 wasn't enough, as the test still fails on Arc GPUs. Mark
this as entirely unsupported until someone can properly fix the root
cause as per #13090
wenju-he added a commit to wenju-he/unified-runtime that referenced this issue Apr 9, 2024
@wenju-he
[L0][Image] Set ZeImageDesc member of _ur_image in release build for …
0497923 
…legacy image

This fix a regression intel/llvm#13090 caused
by 8499b57 which adds ZeImageDesc member to _ur_image regardless of NDEBUG.
@wenju-he wenju-he mentioned this issue Apr 9, 2024
Merged
@nrspruit
Copy link
Contributor

Fixed in #13338

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working confirmed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ldrumm @nrspruit @KornevNikita