-
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
[SYCL] Fix deferred buf destruction regression caused by host device removal #14396
Conversation
…ce removal Signed-off-by: Tikhomirova, Kseniya <kseniya.tikhomirova@intel.com>
I'm not familiar with neither this area nor the previous PR... Adding @sergey-semenov |
fixes #14397 |
Signed-off-by: Tikhomirova, Kseniya <kseniya.tikhomirova@intel.com>
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.
test changes LGTM
@KseniyaTikhomirova What is your confidence that this change fixes the cause of the hang? If we don't have very high confidence and re-enable the tests, it could break the CI again.
Thanks.
I was able to reproduce hang on local WIN Gen9 system and with this fix tests passed |
Ok, sounds legit to me, thanks. |
No description provided.