Skip to content

[GHA] Uplift Linux GPU RT version to 24.45.31740.9 #16312

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

Merged
merged 15 commits into from
Dec 19, 2024
Merged

[GHA] Uplift Linux GPU RT version to 24.45.31740.9 #16312

Changes from 2 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
9c2ac0a
[GHA] Uplift Linux GPU RT version to 24.45.31740.9
actions-user Dec 10, 2024
d81751a
fix igc dep
sarnex Dec 17, 2024
b9cbd1b
use 24.04
sarnex Dec 17, 2024
4f3e923
Merge remote-tracking branch 'upstream/sycl' into ci/update_gpu_drive…
sarnex Dec 17, 2024
7038aff
script
sarnex Dec 17, 2024
659a737
fix script again
sarnex Dec 17, 2024
a5d2742
disable failing tests
sarnex Dec 17, 2024
7a7ded1
missed jobs
sarnex Dec 17, 2024
a7aba9a
fix test
sarnex Dec 18, 2024
2985507
Merge remote-tracking branch 'upstream/sycl' into ci/update_gpu_drive…
sarnex Dec 18, 2024
6ec0ea9
remove cpu xfail
sarnex Dec 19, 2024
e720b1e
Merge remote-tracking branch 'upstream/sycl' into ci/update_gpu_drive…
sarnex Dec 19, 2024
dd8aaff
remove xfail
sarnex Dec 19, 2024
3281e37
xfail
sarnex Dec 19, 2024
1171989
readd xfail
sarnex Dec 19, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion sycl/test-e2e/OnlineCompiler/online_compiler_OpenCL.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
// REQUIRES: opencl, opencl_icd, cm-compiler
// XFAIL: gpu
// XFAIL: gpu || cpu
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@intel/dpcpp-sanitizers-review Ping, needed to disable some tests that started failing in the GPU driver, please take a look, thanks!

@sarnex I'm a bit confused, this is the only change I see since my last review, how does xfailing in CPU relate to problems in the GPU driver?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah youre right this makes no sense, let me remove it try to remember why i added it, thanks

Copy link
Contributor

@sarnex sarnex Dec 19, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok it took me forever but i investigated it, basically the test calls the gpu driver manually even when the target device is cpu , that's why it's failing, so im gonna add the XFAIL back, idk if thats intentional or not but ill let the test owner investigate

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that expected? Should we rewrite the test not to do that?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no idea, theres a GH issue tracking the XFAIL so i'll let the issue owner make a decision

// XFAIL-TRACKER: https://github.com/intel/llvm/issues/16406
// RUN: %{build} -Wno-error=deprecated-declarations -DRUN_KERNELS %opencl_lib -o %t.out
// RUN: %{run} %t.out
Expand Down
Loading