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

ThreadSanitizer: data race amrex::InitRandom w/ OMP #4279

Open
ax3l opened this issue Jan 2, 2025 · 3 comments
Open

ThreadSanitizer: data race amrex::InitRandom w/ OMP #4279

ax3l opened this issue Jan 2, 2025 · 3 comments
Assignees
@WeiqunZhang
Labels
bug

Comments

@ax3l
Copy link
Member

ax3l commented Jan 2, 2025

With OpenMP, we see from the Clang's ThreadSanitizer that there is a data race flagged in amrex::InitRandom:
ECP-WarpX/WarpX#5492 (comment)
@ax3l ax3l added the bug label Jan 2, 2025
@ax3l ax3l mentioned this issue Jan 2, 2025
Open
@ax3l
Copy link
Member Author

ax3l commented Jan 2, 2025

I uploaded debug logs here: ECP-WarpX/WarpX#5492 (comment)

Looks like it complains about this line:
https://github.com/AMReX-Codes/amrex/blob/25.01/Src/Base/AMReX_Random.cpp#L105

@ax3l ax3l mentioned this issue Jan 2, 2025
Closed
5 tasks
@WeiqunZhang
Copy link
Member

I don't think it's a bug in our code.

@ax3l
Copy link
Member Author

ax3l commented Jan 6, 2025

Attn @EZoni @lucafedeli88

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@WeiqunZhang WeiqunZhang

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Silence Thread Sanitizer in InitRandom ax3l/amrex
2 participants
@ax3l @WeiqunZhang