-
Notifications
You must be signed in to change notification settings - Fork 734
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
[GHA] Uplift Linux GPU RT version to 23.17.26241.22 #10087
Conversation
@aelovikov-intel, FYI. |
I somewhat doubt it's the CI because the testing in #10054 finished successfully. Is it possible the drivers are bad/incompatible? |
From the logs:
|
Re-run doesn't have that issue. |
Okay, but we should think about preventive cure for |
FYI: I found a bug report for that issue - #8462. |
Post-commit failure:
For these automated Linux GPU driver updates we could run broader scope of testing on pre-commit step automatically. |
Fix: #10132 |
I think that could be as easy as making llvm/.github/workflows/sycl_precommit.yml Line 69 in 1bb5648
lin_intel task and prefer separate l0_gen9 , ocl_gen9 and ocl_x64 ).
Condition could probably be based on llvm/.github/workflows/sycl_detect_changes.yml Lines 50 to 59 in 1bb5648
|
… driver update (#10132) Fixes post-commit: #10087 (comment)
@dm-vodopyanov, @aelovikov-intel, could you clarify what testing is missing in this PR pre-commit? Don't we run ESIMD tests on GPU in all pre-commits? |
https://github.com/intel/llvm/blob/sycl/sycl/test-e2e/README.md?plain=1#L34-L44 We don't test for |
This text doesn't say if there is any difference in |
So just having additional devices in the system changes the status of the test? Do you understand how broken this system is? How are developers supposed to debug issues like this? |
No. |
)" This reverts commit 46046bd.
Scheduled drivers uplift