-
Notifications
You must be signed in to change notification settings - Fork 803
[CI][L0 v2] Add to CI options for L0v2 adapter #20890
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
6ea82f0 to
55b2227
Compare
| binaries_artifact: e2e_bin_preview | ||
| - name: L0 v2 | ||
| runner: '["L0v2_arl"]' | ||
| target_devices: level_zero_v2:arch-intel_gpu_mtl_u |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mtl? shouldn't this be arl?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lscpu: "Intel(R) Core(TM) Ultra 7 265"
sycl-ls: "Architecture: intel_gpu_mtl_u"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So the tool is incorrectly identifying the architecture. That seems like a bug.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| extra_lit_opts: --param test-preview-mode=True | ||
| binaries_artifact: e2e_bin_preview | ||
| - name: Intel / ARL with L0 v2 | ||
| runner: '["L0v2_arl"]' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kind of a nit but is there any way we can make the runner labels match the others? so we can have like
'["Linux", "arl"]'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think XFAIL for this test already got removed in f910e7e
Maybe this PR need a rebase. Thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rebased
|
@intel/llvm-gatekeepers Please consider merge |
No description provided.