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

Multithreading issues #21

Open
janpaul123 opened this issue Feb 10, 2022 · 0 comments
Open

Multithreading issues #21

janpaul123 opened this issue Feb 10, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@janpaul123
Copy link
Contributor

Spawning threads all the time is a bit of an antipattern (you should use a thread pool) so I don't think this should be part of https://github.com/Zaplib/zaplib/milestone/1, but there are some issues with threading that should be addressed at some point:

  • test_multithread deadlocks in Chrome after a while (not sure about other browsers).
  • In Safari 15.2 test_multithread doesn't work well at all.

There are some deeper issues that might require language changes, and are less urgent, but I'll mention them here anyway. We might want to split this out to a different ticket at some point.

  • Threads leak memory since we never deallocate the TLS/shadow stack (see also this issue).
@janpaul123 janpaul123 added the bug Something isn't working label Feb 10, 2022
@stevekrouse stevekrouse moved this to Upcoming in Current work Feb 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Backlog
Development

No branches or pull requests

1 participant