Skip to content

[SYCL] Fix possible failure when enqueing only a single host-task #1937

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

Merged
merged 5 commits into from
Jun 26, 2020

Conversation

s-kanaev
Copy link
Contributor

@s-kanaev s-kanaev commented Jun 19, 2020

The issue results in either segfault due to some nullptr dereference or in a deadlock when the buffer gets destroyed and the last access to it was through host task.

@s-kanaev s-kanaev force-pushed the private/s-kanaev/fix-possible-deadlock branch from a650cac to 8b9f200 Compare June 19, 2020 17:05
@s-kanaev s-kanaev marked this pull request as ready for review June 19, 2020 17:13
@s-kanaev s-kanaev requested a review from a team as a code owner June 19, 2020 17:13
@s-kanaev s-kanaev requested a review from v-klochkov June 19, 2020 17:13
@s-kanaev s-kanaev force-pushed the private/s-kanaev/fix-possible-deadlock branch from 8b9f200 to 237f900 Compare June 19, 2020 17:26
@s-kanaev s-kanaev force-pushed the private/s-kanaev/fix-possible-deadlock branch from 237f900 to 463336e Compare June 19, 2020 17:27
@s-kanaev s-kanaev requested a review from v-klochkov June 19, 2020 17:33
@v-klochkov v-klochkov requested a review from romanovvlad June 19, 2020 17:52
@s-kanaev s-kanaev requested a review from sergey-semenov June 22, 2020 18:07
@s-kanaev s-kanaev requested a review from sergey-semenov June 25, 2020 11:15
sergey-semenov
sergey-semenov previously approved these changes Jun 25, 2020
Copy link
Contributor

@sergey-semenov sergey-semenov left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Sergey Kanaev <[email protected]>

Co-authored-by: Sergey Semenov <[email protected]>
Signed-off-by: Sergey Kanaev <[email protected]>
@s-kanaev s-kanaev requested a review from sergey-semenov June 26, 2020 13:15
@bader bader merged commit e911de7 into intel:sycl Jun 26, 2020
@s-kanaev s-kanaev deleted the private/s-kanaev/fix-possible-deadlock branch September 2, 2020 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants