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

Make timer class multi-threading friendly. #20

Open
v0dro opened this issue Nov 28, 2020 · 0 comments
Open

Make timer class multi-threading friendly. #20

v0dro opened this issue Nov 28, 2020 · 0 comments
Labels
#medium medium-priority task

Comments

@v0dro
Copy link
Contributor

v0dro commented Nov 28, 2020

The timer class currently performs non-atomic operations, which leads to issues when using multi-threading within the library, for example with openMP tasks. Making the operations atomic or somehow building per-thread timing constructs should mitigate this problem.

Here's one of the errors I get:

blr-lu-exp3d: /home/acb10922qh/gitrepos/useful-tsubame-benchmarks/hmatrix-benchmarks/hicma-yokotalab/src/util/timer.cpp:73: void hicma::timing::Timer::start(): Assertion `!running' failed.
@delta-leader delta-leader added the #medium medium-priority task label Nov 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#medium medium-priority task
Projects
None yet
Development

No branches or pull requests

2 participants