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

Loading data with more than 1 space view deadlocks if RAYON_NUM_THREADS=2 #4939

Closed
teh-cmc opened this issue Jan 29, 2024 · 2 comments · Fixed by #4942
Closed

Loading data with more than 1 space view deadlocks if RAYON_NUM_THREADS=2 #4939

teh-cmc opened this issue Jan 29, 2024 · 2 comments · Fixed by #4942
Assignees
Labels
🪳 bug Something isn't working 💣 crash crash, deadlock/freeze, do-no-start 📺 re_viewer affects re_viewer itself
Milestone

Comments

@teh-cmc
Copy link
Member

teh-cmc commented Jan 29, 2024

This is true for both our latest release (0.12.1) and our latest main: RAYON_NUM_THREADS=2 rerun plots.rrd successfully starts a viewer and loads the data, but then deadlocks.

I think this might possibly hide another deadlock in the query cache, but we need to fix this one first to know for sure.

Putting on high priority as this might hide something very very wrong.

24-01-29_15.05.51.patched.mp4
@teh-cmc teh-cmc added 🪳 bug Something isn't working 📺 re_viewer affects re_viewer itself 💣 crash crash, deadlock/freeze, do-no-start labels Jan 29, 2024
@Wumpf
Copy link
Member

Wumpf commented Jan 29, 2024

Tried with both caching enabled and disabled, both from startup or changed at runtime and it doesn't repro for me so far.

But both me and @abey79 hit a deadlock recently during visualizer execution, so it's likely that!

@teh-cmc
Copy link
Member Author

teh-cmc commented Jan 29, 2024

This is caused by external loaders, removing them all from my PATH fixes the issue -- something is blocking and/or waiting for another task on the compute threadpool?

No idea why the number of space views seemed to affect my earlier repro? We'll see what happens when I fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪳 bug Something isn't working 💣 crash crash, deadlock/freeze, do-no-start 📺 re_viewer affects re_viewer itself
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants