Add support for illumos target

Author: pfmooney

Cargo.toml

@@ -35,7 +35,7 @@ extra-docs = []
 log = "0.4.8"
 
 [target.'cfg(unix)'.dependencies]
-libc = "0.2.62"
+libc = "0.2.69"
 
 [target.'cfg(windows)'.dependencies]
 miow   = "0.3.3"

src/poll.rs

@@ -174,6 +174,7 @@ use std::{fmt, io};
 /// | NetBSD        | [kqueue]  |
 /// | OpenBSD       | [kqueue]  |
 /// | Solaris       | [epoll]   |
+/// | illumos       | [epoll]   |
 /// | Windows       | [IOCP]    |
 /// | iOS           | [kqueue]  |
 /// | macOS         | [kqueue]  |

src/sys/unix/net.rs

@@ -27,6 +27,7 @@ pub(crate) fn new_socket(
         target_os = "android",
         target_os = "dragonfly",
         target_os = "freebsd",
+        target_os = "illumos",
         target_os = "linux",
         target_os = "netbsd",
         target_os = "openbsd"

src/sys/unix/selector/epoll.rs

@@ -21,10 +21,15 @@ pub struct Selector {
 
 impl Selector {
     pub fn new() -> io::Result<Selector> {
-        // According to libuv `EPOLL_CLOEXEC` is not defined on Android API <
-        // 21. But `EPOLL_CLOEXEC` is an alias for `O_CLOEXEC` on all platforms,
-        // so we use that instead.
-        syscall!(epoll_create1(libc::O_CLOEXEC)).map(|ep| Selector {
+        // According to libuv, `EPOLL_CLOEXEC` is not defined on Android API <
+        // 21. But `EPOLL_CLOEXEC` is an alias for `O_CLOEXEC` on that platform,
+        // so we use it instead.
+        #[cfg(target_os = "android")]
+        let flag = libc::O_CLOEXEC;
+        #[cfg(not(target_os = "android"))]
+        let flag = libc::EPOLL_CLOEXEC;
+
+        syscall!(epoll_create1(flag)).map(|ep| Selector {
             #[cfg(debug_assertions)]
             id: NEXT_ID.fetch_add(1, Ordering::Relaxed),
             ep,
@@ -212,6 +217,7 @@ pub mod event {
     }
 }
 
+#[cfg(target_os = "linux")]
 #[test]
 fn assert_close_on_exec_flag() {
     // This assertion need to be true for Selector::new.

src/sys/unix/selector/mod.rs

@@ -1,7 +1,17 @@
-#[cfg(any(target_os = "linux", target_os = "android", target_os = "solaris"))]
+#[cfg(any(
+    target_os = "android",
+    target_os = "illumos",
+    target_os = "linux",
+    target_os = "solaris"
+))]
 mod epoll;
 
-#[cfg(any(target_os = "linux", target_os = "android", target_os = "solaris"))]
+#[cfg(any(
+    target_os = "android",
+    target_os = "illumos",
+    target_os = "linux",
+    target_os = "solaris"
+))]
 pub(crate) use self::epoll::{event, Event, Events, Selector};
 
 #[cfg(any(

src/sys/unix/tcp.rs

@@ -62,7 +62,8 @@ pub fn accept(listener: &net::TcpListener) -> io::Result<(net::TcpStream, Socket
         target_os = "dragonfly",
         target_os = "freebsd",
         target_os = "linux",
-        target_os = "openbsd"
+        target_os = "openbsd",
+        target_os = "illumos"
     ))]
     let stream = {
         syscall!(accept4(

src/sys/unix/waker.rs

@@ -102,7 +102,8 @@ pub use self::kqueue::Waker;
     target_os = "dragonfly",
     target_os = "netbsd",
     target_os = "openbsd",
-    target_os = "solaris"
+    target_os = "solaris",
+    target_os = "illumos"
 ))]
 mod pipe {
     use crate::sys::unix::Selector;
@@ -167,6 +168,7 @@ mod pipe {
     target_os = "dragonfly",
     target_os = "netbsd",
     target_os = "openbsd",
-    target_os = "solaris"
+    target_os = "solaris",
+    target_os = "illumos"
 ))]
 pub use self::pipe::Waker;

tests/tcp_stream.rs

@@ -540,7 +540,7 @@ fn tcp_shutdown_client_read_close_event() {
 #[test]
 #[cfg_attr(windows, ignore = "fails; client write_closed events are not found")]
 #[cfg_attr(
-    any(target_os = "linux", target_os = "android", target_os = "solaris"),
+    any(target_os = "linux", target_os = "android", target_os = "solaris", target_os = "illumos"),
     ignore = "fails; client write_closed events are not found"
 )]
 fn tcp_shutdown_client_write_close_event() {