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.

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

[SYCL][CUDA]Massive runfail for CUDA SYCL CTS on unrelated commit #8556

Open
dm-vodopyanov opened this issue Mar 7, 2023 · 5 comments
Open

[SYCL][CUDA]Massive runfail for CUDA SYCL CTS on unrelated commit #8556

dm-vodopyanov opened this issue Mar 7, 2023 · 5 comments
Labels
bug Something isn't working CTS Impacts Khronos SYCL CTS cuda CUDA back-end

Comments

@dm-vodopyanov
Copy link
Contributor

Post-commit failed on unrelated patch for ESIMD d2bc9d1 (@v-klochkov please confirm that this is unrelated) - CUDA SYCL CTS run failed. This could be due to changes in https://github.com/KhronosGroup/SYCL-CTS.

Logs: https://github.com/intel/llvm/actions/runs/4348365190/jobs/7597934088
@dm-vodopyanov dm-vodopyanov added bug Something isn't working cuda CUDA back-end labels Mar 7, 2023
@dm-vodopyanov dm-vodopyanov mentioned this issue Mar 7, 2023
Merged
@dm-vodopyanov
Copy link
Contributor Author

Looks like culprit is KhronosGroup/SYCL-CTS@df2259b

@dm-vodopyanov
Copy link
Contributor Author

@vladimirkhashev FYI.

@bader
Copy link
Contributor

bader commented Mar 8, 2023

Looks like culprit is KhronosGroup/SYCL-CTS@df2259b

@dm-vodopyanov, I suggest disabling new test to fix post-commit.
Please, keep this issue open to address the failures and re-enable the test.

@dm-vodopyanov dm-vodopyanov added the CTS Impacts Khronos SYCL CTS label Mar 8, 2023
@dm-vodopyanov
Copy link
Contributor Author

dm-vodopyanov commented Mar 8, 2023 •
edited
Loading

Created #8575 to disable the tests

@JackAKirk
Copy link
Contributor

JackAKirk commented Sep 25, 2024 •
edited
Loading

full atomic_ref functionality requires building with sm_70 specified using nvptx target flag. We only support the full ptx 6.0 memory model (since #12516) and this requires sm_70 and we have confirmed that the sycl CTS has pull pass rate for atomic tests for this case. Earlier versions have an unspecified memory model that we have not fully implemented.

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 CTS Impacts Khronos SYCL CTS cuda CUDA back-end
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bader @dm-vodopyanov @JackAKirk