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.

Sign up for GitHub

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

Jump to bottom

Add a systhread pool of workers for eio_posix #607

Closed
Tracked by #456
talex5 opened this issue Aug 17, 2023 · 1 comment
Closed
Tracked by #456

Add a systhread pool of workers for eio_posix #607

talex5 opened this issue Aug 17, 2023 · 1 comment

Comments

@talex5
Copy link
Collaborator

talex5 commented Aug 17, 2023 •
edited
Loading

Eio_posix performs file operations (e.g. stat) in their own sys-threads, creating a new thread for each operation. This is quite inefficient; we should use a pool of threads like Lwt does.
@talex5 talex5 mentioned this issue Aug 17, 2023
Open
5 tasks
@talex5 talex5 changed the title Add a systhread pool of workers so that e.g. file operations never block the main thread. Add a systhread pool of workers for eio_posix Aug 17, 2023
@SGrondin SGrondin mentioned this issue Aug 18, 2023
Merged
@SGrondin SGrondin mentioned this issue Dec 17, 2023
Closed
@talex5
Copy link
Collaborator Author

talex5 commented Feb 20, 2024

Fixed by #681.

@talex5 talex5 closed this as completed Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@talex5