Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Investigate if atomic:fence needs to be added in broadcast and try_receive #47

Open
filipdulic opened this issue May 10, 2020 · 0 comments
Open

Investigate if atomic:fence needs to be added in broadcast and try_receive #47

filipdulic opened this issue May 10, 2020 · 0 comments
Labels
investigation Something needs to be investigated.

Comments

@filipdulic
Copy link
Owner

filipdulic commented May 10, 2020 •
edited
Loading

Ordering is very important for certain operations in broadcast, and try_recv. Investigate weather the std::sync::atomic::fence needs to be added to these functions where appropriate.

  • between store and wi.inc in broadcast.
  • between val load and loop in try_recv.
@filipdulic filipdulic changed the title Add atomic:fence in broadcast and try_receive Investigate if atomic:fence needs to be added in broadcast and try_receive May 10, 2020
@filipdulic filipdulic added the investigation Something needs to be investigated. label May 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
investigation Something needs to be investigated.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@filipdulic