-
Notifications
You must be signed in to change notification settings - Fork 730
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
Q.fill() improvements fail on gen12 #13787
Comments
cc @steffenlarsen @aelovikov-intel any ideas on this? We have limited access to gen12 so haven't been able to figure it out. We could maybe mark the test unsupported/xfail to unblock the patch until someone else can have a look? |
@konradkusiak97 it's flaky issue, right? |
It might be a flaky issue. Though, I've seen it being reproduced quite reliably on the post-commit CI on gen12 but only there. I don't have access to |
@KornevNikita, do you think we could we mark the test unsupported/xfail as @npmiller suggested? This is now the only thing that's blocking merging improvements to |
If your patch introduces this regression, I guess it's not a good idea to turn off the test, isnt't? Also as I remember it affects the CTS: #13679 Then If the post-commit is green I guess we have no blockers to just merge it and monitor the post-commit. |
Yes, the CTS failure is fixed with this patch: oneapi-src/unified-runtime#1603 |
I tried again reproducing this failure but without any luck. Would you be able to try out the post-commit with my patch: #13788 @KornevNikita and see if it still fails? |
I suggest to just trigger it in the PR. |
@konradkusiak97 hi! As I understand you made a fix / found a workaround, so this issue can be closed once the PR is merged? |
I didn't get to the bottom of what's causing this issue so for now for I'm fine with closing this. |
Describe the bug
When trying to make implementation of
Q.fill()
to usepiextUSMEnqueueFill
in this PR: #13788, the sycl/test-e2e/out_of_order_queue_status.cpp failed on the post-commit CI, on gen12:The failure is very hard to reproduce since it only fails randomly on
Intel(R) Iris(R) Xe Graphics
. I managed to reproduce it a few times only with a fresh build and laptop reboot, running all e2e tests for bothlevel_zero:gpu
andopencl:fpga
which is how the post-commit CI does it.For now, for
level_zero
target, the path topiextUSMEnqueueFill
was reverted due to this failure and workaround was implemented to use previous implementation ofQ.fill()
in #13788.DPC++ build configuration:
Environment
OS: Linux
Target device and vendor: Intel(R) Iris(R) Xe Graphics
DPC++ commit: c173fbf
sycl-ls --verbose:
The text was updated successfully, but these errors were encountered: