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

elastic_aio doesn't work on OS X #16

Closed
grencez opened this issue Feb 2, 2021 · 2 comments
Closed

elastic_aio doesn't work on OS X #16

grencez opened this issue Feb 2, 2021 · 2 comments
Assignees

Comments

@grencez
Copy link
Member

grencez commented Feb 2, 2021

Last week I wrote a poll()-based version of elastic instead of fixing the async version on OS X, but we should use the async version if possible.

Why? AIO seems to (1) be faster than polling, (2) match the Windows IOCP approach, and (3) admit an abstraction for threads to exchange data without copying too much.

@grencez grencez added the bug label Feb 2, 2021
@grencez grencez self-assigned this Feb 14, 2021
@grencez
Copy link
Member Author

grencez commented Feb 14, 2021

aio_read() on stdin fails and sets errno to EAGAIN (resource temporarily unavailable).

The same code works for regular files, so it appears that OS X just doesn't support AIO on pipes. Drat!

@grencez
Copy link
Member Author

grencez commented Feb 14, 2021

3579855 is the result of this work. No fix, but some improvements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant