Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[GPU]fix scatter_elements_update_ref data race #29154

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

[GPU]fix scatter_elements_update_ref data race #29154

wants to merge 4 commits into from
+33 −15

Conversation

zaixing-wang
Copy link
Contributor

@zaixing-wang zaixing-wang requested review from a team as code owners February 25, 2025 10:56
@github-actions github-actions bot added the category: GPU OpenVINO GPU plugin label Feb 25, 2025
sshlyapn
sshlyapn reviewed Feb 26, 2025
View reviewed changes
.../intel_gpu/src/kernel_selector/kernels/scatter_update/scatter_elements_update_kernel_ref.cpp
Comment on lines +128 to +131
if (is_second) {
dispatchData.gws = {1, 1, 1};
dispatchData.lws = {1, 1, 1};
}
Copy link
Contributor

Choose a reason for hiding this comment

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

[random spot]
Can you please describe at which stage/moment data race is exactly happening?

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@sshlyapn sshlyapn sshlyapn left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
category: GPU OpenVINO GPU plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zaixing-wang @sshlyapn