Skip to content

[GHA] Uplift Linux IGC Dev RT version to igc-dev-46629d9 #19933

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
Merged
merged 4 commits into from
Sep 2, 2025
Merged

[GHA] Uplift Linux IGC Dev RT version to igc-dev-46629d9 #19933

merged 4 commits into from
Sep 2, 2025

Conversation

bb-sycl
Copy link
Contributor

@bb-sycl bb-sycl commented Sep 1, 2025

Scheduled igc dev drivers uplift

@bb-sycl bb-sycl requested review from a team as code owners September 1, 2025 03:28
@dkhaldi
Copy link
Contributor

dkhaldi commented Sep 2, 2025

These also show as xpassed in daily CI and CMPLRTST-27768 but I am suspecting them to be sporadic because I see them still failing in #19927. We can remove xpass for now and see what happens.
Unexpectedly Passed Tests (2):
SYCL :: Matrix/joint_matrix_bf16_fill_k_cache_arg_dim.cpp
SYCL :: Matrix/joint_matrix_bf16_fill_k_cache_runtime_dim.cpp

@sarnex
Copy link
Contributor

sarnex commented Sep 2, 2025

@dkhaldi Should I remove the xfail or is your PR gonna do it? Thx

@dkhaldi
Copy link
Contributor

dkhaldi commented Sep 2, 2025

@dkhaldi Should I remove the xfail or is your PR gonna do it? Thx

Please remove it here.

@sarnex
Copy link
Contributor

sarnex commented Sep 2, 2025

sure

@sarnex
enable tests
76c827f 
Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex
Copy link
Contributor

sarnex commented Sep 2, 2025

@dkhaldi pushed the new commit, but not sure if i implemented your feedback correctly

dkhaldi
dkhaldi reviewed Sep 2, 2025
View reviewed changes
sycl/test-e2e/Matrix/joint_matrix_bf16_fill_k_cache_arg_dim.cpp Outdated
@@ -15,8 +15,8 @@

// Waiting for the commit in IGC to be pulled into the driver to resolve the
// test.
// XFAIL: run-mode && arch-intel_gpu_pvc
// XFAIL-TRACKER: GSD-10510
// UNSUPPORTED: run-mode && arch-intel_gpu_pvc && !igc-dev
Copy link
Contributor

@dkhaldi dkhaldi Sep 2, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

We should keep them as xfail in this PR for !igc-dev.

Copy link
Contributor

Choose a reason for hiding this comment

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

thanks, hopefully latest commit is correct

@sarnex
xfail
1adc00f 
Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex
Copy link
Contributor

sarnex commented Sep 2, 2025

SYCL :: Matrix/SG32/get_coordinate_ops.cpp

failing. should we disable that?

@dkhaldi
Copy link
Contributor

dkhaldi commented Sep 2, 2025

SYCL :: Matrix/SG32/get_coordinate_ops.cpp

failing. should we disable that?

Yes, the revert is taking more time than expected @ggojska.
In the meantime, please set this to xfail. tracker is GSD-11139

@sarnex
xfail test
135fb4c 
Signed-off-by: Sarnie, Nick <[email protected]>
dkhaldi
dkhaldi approved these changes Sep 2, 2025
View reviewed changes
Copy link
Contributor

@dkhaldi dkhaldi left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@sarnex sarnex merged commit 45cf443 into sycl Sep 2, 2025
27 of 28 checks passed
This was referenced Sep 2, 2025
Merged
Merged
@bader bader deleted the ci/update_gpu_driver-linux-igc-dev-46629d9 branch September 3, 2025 05:00
@dkhaldi dkhaldi mentioned this pull request Sep 3, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sarnex sarnex sarnex approved these changes

@dkhaldi dkhaldi dkhaldi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@bb-sycl @dkhaldi @sarnex @actions-user