Miscellaneous emscripten fixes #510
Conversation
Emscripten's stack size was recently decreased to 64 kB from 5 MB, (emscripten-core/emscripten#18191). Stack overflow appears to be the cause of frequent crashes of Tracy in my browser, especially at start-up. This increase is modest, but seems to be enough to resolve the issue.
In my browser, I noticed that zooming seemed jumpy and unreliable. Turns out that my fingers were moving horizontally on my trackpad, and the x-scroll was causing the unexpected behavior.
I never noticed this causing any user-visible deadlocks, but I saw warnings in the Javascript console that the thread pool was getting exhausted. Increasing it to this number seems to resolve the issue.
|
If the pthread pool size change may be unnecessary, why is it made? |
|
|
The thread pool is definitely being exhausted, but I don't know enough about Tracy's usage of threads to know when it depends on being able to execute all its threads concurrently. Happy to remove the change if you'd prefer.
Fair enough - I've submitted ocornut/imgui#6096 |
I have seen some weird problems related to threads, but these were mainly on Chrome-based browsers. Firefox seems to handle this much better. There are already some ifdefs to minimize thread usage with emscripten, but maybe this is not enough? https://github.com/wolfpld/tracy/blob/e4bd88c/server/TracyWorker.cpp#L1271-L1277 |
|
Good point. I'll try to investigate this weekend and see I can track down where the threads are being started |
Series of emscripten-specific fixes:
STACK_SIZEto 128 kB from 64 kB (was causing crashes, especially on startup)PTHREAD_POOL_SIZEfrom 4 to 8 (may be unnecessary)