-
Notifications
You must be signed in to change notification settings - Fork 787
[GHA] Uplift Linux GPU RT version to 23.09.25812.14 #9196
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
Conversation
On L0 and OpenCL GPU we have a test unexpectedly passing. That's fine. On both L0 and OpenCL GPU, the Printf/char.cpp test is failing. Does anyone know anything about that? And the L0 queue priority/profling tests are failing. Is this known? Or need to be addressed?
|
I am trying to address them in #9166
|
@smaslov-intel, do you have any updates on this? I see that #9166 is merged. |
@bader : yes, it is merged and I expected the following GPU driver update to be successful. however I see this in a yesterday's attempt: https://github.com/intel/llvm/actions/runs/4987433999/jobs/8929192727 Any idea? |
The job fails to push a branch with the driver version update change because the branch already exists. Please, merge the fix to the https://github.com/intel/llvm/tree/ci/update_gpu_driver-linux-23.09.25812.14 branch. |
Failed Tests (1): Unexpectedly Passed Tests (1): |
I can reproduce fail locally it is a segfault in L0 RT. Following up with @jandres742
@v-klochkov : can you see if this is a reasonable new pass? |
Yes, we were waiting for GPU driver update to enable that test. |
Great, thanks for confirming. |
@smaslov-intel, do you have any updates on this issue? In case it might be helpful: this issue appeared with 22.49.* driver (see #8156 (comment)). |
this has been fixed in IGC. We need to wait for fix to be promoted to compute-runtime GPU repo to use it. |
I'd like to update the driver ASAP as old version gating some important changes in DPC++ compiler like switching to opaque pointers. The risk to update to 23.09.* version is that we disable |
thanks @bader. +1 on my side. |
@intel/llvm-reviewers-runtime, @KseniyaTikhomirova, ping. |
@bader : did you create an issue to re-enable the test when the fixed driver is available? |
Scheduled drivers uplift