-
-
Notifications
You must be signed in to change notification settings - Fork 411
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
Multi-threaded stats #872
Comments
To add a little more detail to this issue. Currently the flecs/src/addons/pipeline/pipeline.c Lines 632 to 641 in b5241cc
This means that the stats only knows about the amount of time the operation took to execute the main thread portion of the work, however, systems may have unbalanced work. Meaning a worker thread could have more to do than the main thread. If this happens it will cause the time interval on the main thread will not be equivalent to the amount of time the system actually took to execute. |
Describe the problem you are trying to solve.
The stats addon and explorer is not MT aware, the values stop making sense with multiple threads.
Describe the solution you'd like
Add support for per-thread stats, thread-specific stats (could indicate how well work is distributed, etc), and proper sums for time spent in systems (wall clock vs cpu time). And then support for it in explorer, of course.
The text was updated successfully, but these errors were encountered: