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.

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

WIP: Potential race condition when updating PC hits #454

Open
anishnaik opened this issue Aug 22, 2024 · 0 comments
Open

WIP: Potential race condition when updating PC hits #454

anishnaik opened this issue Aug 22, 2024 · 0 comments

Comments

@anishnaik
Copy link
Collaborator

Currently, we do not use any atomic read/writes when updating the number of hits for a given PC. It is possible that this is an issue if two workers try to update the number of hits at the same time.

I do not think it is a problem because the updateCoveredAt function is executed on memory that is controlled per-worker. However, it is beneficial to investigate this further.

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

No branches or pull requests

1 participant