Skip to content

Missing implementation libc::EPOLLONESHOT on the android #548

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
Closed
gdf8gdn8 opened this issue Mar 11, 2017 · 3 comments · Fixed by #619
Closed

Missing implementation libc::EPOLLONESHOT on the android #548

gdf8gdn8 opened this issue Mar 11, 2017 · 3 comments · Fixed by #619

Comments

@gdf8gdn8
Copy link
Contributor

epoll EPOLLONESHOT is implemented since linux kernel 2.6.2, but on android platform EPOLLONESHOT is missing in rust libc.

Should be an bug in rust libc implementation.
Because EPOLLONESHOT is available on android platform.
see https://doc.rust-lang.org/libc/arm-linux-androideabi/libc/index.html?search=EPOLLONESHOT
@gdf8gdn8 gdf8gdn8 mentioned this issue Mar 11, 2017
Closed
@alexcrichton
Copy link
Member

Ah yeah if bindings aren't present that just means that they haven't been added yet, feel free to submit a PR!

@gdf8gdn8
Copy link
Contributor Author

gdf8gdn8 commented Jun 8, 2017

Which android versions are supported by rust libc?
Currently since android ndk 21 (android 5) EPOLLONESHOT is available.

@alexcrichton
Copy link
Member

We typically use the latest when it's written down, and this can be updated at any time

@gdf8gdn8 gdf8gdn8 mentioned this issue Jun 15, 2017
Merged
bors added a commit that referenced this issue Jun 15, 2017
@bors
Auto merge of #619 - gdf8gdn8:fixed-missing-epolloneshot-android, r=a…
edbc1f1 
…lexcrichton

Fixed missing EPOLLONESHOT for android

resolve #548
@bors bors closed this as completed in #619 Jun 15, 2017
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

Successfully merging a pull request may close this issue.

Fixed missing EPOLLONESHOT for android
2 participants
@alexcrichton @gdf8gdn8