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

IOHandler should use 'epoll' or similar instead of 'select' #3

Closed
ardeujho opened this issue Feb 7, 2014 · 1 comment
Closed

IOHandler should use 'epoll' or similar instead of 'select' #3

ardeujho opened this issue Feb 7, 2014 · 1 comment

Comments

@ardeujho
Copy link

ardeujho commented Feb 7, 2014

The SEAM VM seems to use the select API call in IOHandler.cc which has limitations compared to newer APIs like epoll. In particular, select is limited to 1024 file handles on some platforms.
ardeujho added a commit to ardeujho/aliceml that referenced this issue Feb 13, 2014
@ardeujho
Initial attempt at using epoll instead of select. Fixes doublec#3
37f45c9 
The approach taken here is inefficient but is a proof of concept
that epoll works. It creates an epoll file descriptor on each
attempt to 'select'. A future improvement would be to use a
single epoll file descriptor and use epoll_ctl to add/modify/remove
watched file descriptors.
ardeujho added a commit to ardeujho/aliceml that referenced this issue Feb 14, 2014
@ardeujho
Initial attempt at using epoll instead of select. Fixes doublec#3
26944c4 
The approach taken here is inefficient but is a proof of concept
that epoll works. It creates an epoll file descriptor on each
attempt to 'select'. A future improvement would be to use a
single epoll file descriptor and use epoll_ctl to add/modify/remove
watched file descriptors.
@doublec doublec mentioned this issue Jul 15, 2014
Open
@doublec
Copy link
Owner

doublec commented Jul 15, 2014

Following up at aliceml/aliceml#5.

@doublec doublec closed this as completed Jul 15, 2014
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
2 participants
@doublec @ardeujho