Categorize engine and profile options
[fio.git] / engines / net.c
index d6821a4092e7898aa0fea956a6069931fde3a8b2..3a352de70e492fef403ecb8315287514a7cd1199 100644 (file)
@@ -7,9 +7,11 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <unistd.h>
+#include <signal.h>
 #include <errno.h>
 #include <assert.h>
 #include <netinet/in.h>
+#include <netinet/tcp.h>
 #include <arpa/inet.h>
 #include <netdb.h>
 #include <sys/poll.h>
 
 struct netio_data {
        int listenfd;
-       int send_to_net;
        int use_splice;
-       int type;
        int pipes[2];
-       char host[64];
        struct sockaddr_in addr;
        struct sockaddr_un addr_un;
 };
 
+struct netio_options {
+       struct thread_data *td;
+       unsigned int port;
+       unsigned int proto;
+       unsigned int listen;
+       unsigned int pingpong;
+       unsigned int nodelay;
+};
+
 struct udp_close_msg {
        uint32_t magic;
        uint32_t cmd;
@@ -38,13 +46,89 @@ struct udp_close_msg {
 
 enum {
        FIO_LINK_CLOSE = 0x89,
-       FIO_LINK_CLOSE_MAGIC = 0x6c696e6b,
+       FIO_LINK_OPEN_CLOSE_MAGIC = 0x6c696e6b,
+       FIO_LINK_OPEN = 0x98,
 
        FIO_TYPE_TCP    = 1,
        FIO_TYPE_UDP    = 2,
        FIO_TYPE_UNIX   = 3,
 };
 
+static int str_hostname_cb(void *data, const char *input);
+static struct fio_option options[] = {
+       {
+               .name   = "hostname",
+               .type   = FIO_OPT_STR_STORE,
+               .cb     = str_hostname_cb,
+               .help   = "Hostname for net IO engine",
+               .category = FIO_OPT_C_ENGINE,
+               .group  = FIO_OPT_G_NETIO,
+       },
+       {
+               .name   = "port",
+               .type   = FIO_OPT_INT,
+               .off1   = offsetof(struct netio_options, port),
+               .minval = 1,
+               .maxval = 65535,
+               .help   = "Port to use for TCP or UDP net connections",
+               .category = FIO_OPT_C_ENGINE,
+               .group  = FIO_OPT_G_NETIO,
+       },
+       {
+               .name   = "protocol",
+               .alias  = "proto",
+               .type   = FIO_OPT_STR,
+               .off1   = offsetof(struct netio_options, proto),
+               .help   = "Network protocol to use",
+               .def    = "tcp",
+               .posval = {
+                         { .ival = "tcp",
+                           .oval = FIO_TYPE_TCP,
+                           .help = "Transmission Control Protocol",
+                         },
+                         { .ival = "udp",
+                           .oval = FIO_TYPE_UDP,
+                           .help = "User Datagram Protocol",
+                         },
+                         { .ival = "unix",
+                           .oval = FIO_TYPE_UNIX,
+                           .help = "UNIX domain socket",
+                         },
+               },
+               .category = FIO_OPT_C_ENGINE,
+               .group  = FIO_OPT_G_NETIO,
+       },
+#ifdef CONFIG_TCP_NODELAY
+       {
+               .name   = "nodelay",
+               .type   = FIO_OPT_BOOL,
+               .off1   = offsetof(struct netio_options, nodelay),
+               .help   = "Use TCP_NODELAY on TCP connections",
+               .category = FIO_OPT_C_ENGINE,
+               .group  = FIO_OPT_G_NETIO,
+       },
+#endif
+       {
+               .name   = "listen",
+               .type   = FIO_OPT_STR_SET,
+               .off1   = offsetof(struct netio_options, listen),
+               .help   = "Listen for incoming TCP connections",
+               .category = FIO_OPT_C_ENGINE,
+               .group  = FIO_OPT_G_NETIO,
+       },
+       {
+               .name   = "pingpong",
+               .type   = FIO_OPT_STR_SET,
+               .off1   = offsetof(struct netio_options, pingpong),
+               .help   = "Ping-pong IO requests",
+               .category = FIO_OPT_C_ENGINE,
+               .group  = FIO_OPT_G_NETIO,
+       },
+       {
+               .name   = NULL,
+       },
+};
+
 /*
  * Return -1 for error and 'nr events' for a positive number
  * of events
@@ -78,21 +162,24 @@ static int poll_wait(struct thread_data *td, int fd, short events)
 
 static int fio_netio_prep(struct thread_data *td, struct io_u *io_u)
 {
-       struct netio_data *nd = td->io_ops->data;
+       struct netio_options *o = td->eo;
 
        /*
         * Make sure we don't see spurious reads to a receiver, and vice versa
         */
-       if ((nd->send_to_net && io_u->ddir == DDIR_READ) ||
-           (!nd->send_to_net && io_u->ddir == DDIR_WRITE)) {
+       if (o->proto == FIO_TYPE_TCP)
+               return 0;
+
+       if ((o->listen && io_u->ddir == DDIR_WRITE) ||
+           (!o->listen && io_u->ddir == DDIR_READ)) {
                td_verror(td, EINVAL, "bad direction");
                return 1;
        }
-               
+
        return 0;
 }
 
-#ifdef FIO_HAVE_SPLICE
+#ifdef CONFIG_LINUX_SPLICE
 static int splice_io_u(int fdin, int fdout, unsigned int len)
 {
        int bytes = 0;
@@ -230,10 +317,11 @@ static int fio_netio_splice_out(struct thread_data *td, struct io_u *io_u)
 static int fio_netio_send(struct thread_data *td, struct io_u *io_u)
 {
        struct netio_data *nd = td->io_ops->data;
-       int ret, flags = OS_MSG_DONTWAIT;
+       struct netio_options *o = td->eo;
+       int ret, flags = 0;
 
        do {
-               if (nd->type == FIO_TYPE_UDP) {
+               if (o->proto == FIO_TYPE_UDP) {
                        struct sockaddr *to = (struct sockaddr *) &nd->addr;
 
                        ret = sendto(io_u->file->fd, io_u->xfer_buf,
@@ -244,8 +332,8 @@ static int fio_netio_send(struct thread_data *td, struct io_u *io_u)
                         * if we are going to write more, set MSG_MORE
                         */
 #ifdef MSG_MORE
-                       if (td->this_io_bytes[DDIR_WRITE] + io_u->xfer_buflen <
-                           td->o.size)
+                       if ((td->this_io_bytes[DDIR_WRITE] + io_u->xfer_buflen <
+                           td->o.size) && !o->pingpong)
                                flags |= MSG_MORE;
 #endif
                        ret = send(io_u->file->fd, io_u->xfer_buf,
@@ -257,8 +345,6 @@ static int fio_netio_send(struct thread_data *td, struct io_u *io_u)
                ret = poll_wait(td, io_u->file->fd, POLLOUT);
                if (ret <= 0)
                        break;
-
-               flags &= ~OS_MSG_DONTWAIT;
        } while (1);
 
        return ret;
@@ -272,7 +358,7 @@ static int is_udp_close(struct io_u *io_u, int len)
                return 0;
 
        msg = io_u->xfer_buf;
-       if (ntohl(msg->magic) != FIO_LINK_CLOSE_MAGIC)
+       if (ntohl(msg->magic) != FIO_LINK_OPEN_CLOSE_MAGIC)
                return 0;
        if (ntohl(msg->cmd) != FIO_LINK_CLOSE)
                return 0;
@@ -283,11 +369,12 @@ static int is_udp_close(struct io_u *io_u, int len)
 static int fio_netio_recv(struct thread_data *td, struct io_u *io_u)
 {
        struct netio_data *nd = td->io_ops->data;
-       int ret, flags = OS_MSG_DONTWAIT;
+       struct netio_options *o = td->eo;
+       int ret, flags = 0;
 
        do {
-               if (nd->type == FIO_TYPE_UDP) {
-                       fio_socklen_t len = sizeof(nd->addr);
+               if (o->proto == FIO_TYPE_UDP) {
+                       socklen_t len = sizeof(nd->addr);
                        struct sockaddr *from = (struct sockaddr *) &nd->addr;
 
                        ret = recvfrom(io_u->file->fd, io_u->xfer_buf,
@@ -302,33 +389,34 @@ static int fio_netio_recv(struct thread_data *td, struct io_u *io_u)
                }
                if (ret > 0)
                        break;
+               else if (!ret && (flags & MSG_WAITALL))
+                       break;
 
                ret = poll_wait(td, io_u->file->fd, POLLIN);
                if (ret <= 0)
                        break;
-               flags &= ~OS_MSG_DONTWAIT;
                flags |= MSG_WAITALL;
        } while (1);
 
        return ret;
 }
 
-static int fio_netio_queue(struct thread_data *td, struct io_u *io_u)
+static int __fio_netio_queue(struct thread_data *td, struct io_u *io_u,
+                            enum fio_ddir ddir)
 {
        struct netio_data *nd = td->io_ops->data;
+       struct netio_options *o = td->eo;
        int ret;
 
-       fio_ro_check(td, io_u);
-
-       if (io_u->ddir == DDIR_WRITE) {
-               if (!nd->use_splice || nd->type == FIO_TYPE_UDP ||
-                   nd->type == FIO_TYPE_UNIX) 
+       if (ddir == DDIR_WRITE) {
+               if (!nd->use_splice || o->proto == FIO_TYPE_UDP ||
+                   o->proto == FIO_TYPE_UNIX)
                        ret = fio_netio_send(td, io_u);
                else
                        ret = fio_netio_splice_out(td, io_u);
-       } else if (io_u->ddir == DDIR_READ) {
-               if (!nd->use_splice || nd->type == FIO_TYPE_UDP ||
-                   nd->type == FIO_TYPE_UDP)
+       } else if (ddir == DDIR_READ) {
+               if (!nd->use_splice || o->proto == FIO_TYPE_UDP ||
+                   o->proto == FIO_TYPE_UNIX)
                        ret = fio_netio_recv(td, io_u);
                else
                        ret = fio_netio_splice_in(td, io_u);
@@ -343,7 +431,7 @@ static int fio_netio_queue(struct thread_data *td, struct io_u *io_u)
                } else {
                        int err = errno;
 
-                       if (io_u->ddir == DDIR_WRITE && err == EMSGSIZE)
+                       if (ddir == DDIR_WRITE && err == EMSGSIZE)
                                return FIO_Q_BUSY;
 
                        io_u->error = err;
@@ -356,22 +444,45 @@ static int fio_netio_queue(struct thread_data *td, struct io_u *io_u)
        return FIO_Q_COMPLETED;
 }
 
+static int fio_netio_queue(struct thread_data *td, struct io_u *io_u)
+{
+       struct netio_options *o = td->eo;
+       int ret;
+
+       fio_ro_check(td, io_u);
+
+       ret = __fio_netio_queue(td, io_u, io_u->ddir);
+       if (!o->pingpong || ret != FIO_Q_COMPLETED)
+               return ret;
+
+       /*
+        * For ping-pong mode, receive or send reply as needed
+        */
+       if (td_read(td) && io_u->ddir == DDIR_READ)
+               ret = __fio_netio_queue(td, io_u, DDIR_WRITE);
+       else if (td_write(td) && io_u->ddir == DDIR_WRITE)
+               ret = __fio_netio_queue(td, io_u, DDIR_READ);
+
+       return ret;
+}
+
 static int fio_netio_connect(struct thread_data *td, struct fio_file *f)
 {
        struct netio_data *nd = td->io_ops->data;
+       struct netio_options *o = td->eo;
        int type, domain;
 
-       if (nd->type == FIO_TYPE_TCP) {
+       if (o->proto == FIO_TYPE_TCP) {
                domain = AF_INET;
                type = SOCK_STREAM;
-       } else if (nd->type == FIO_TYPE_UDP) {
+       } else if (o->proto == FIO_TYPE_UDP) {
                domain = AF_INET;
                type = SOCK_DGRAM;
-       } else if (nd->type == FIO_TYPE_UNIX) {
+       } else if (o->proto == FIO_TYPE_UNIX) {
                domain = AF_UNIX;
                type = SOCK_STREAM;
        } else {
-               log_err("fio: bad network type %d\n", nd->type);
+               log_err("fio: bad network type %d\n", o->proto);
                f->fd = -1;
                return 1;
        }
@@ -382,10 +493,21 @@ static int fio_netio_connect(struct thread_data *td, struct fio_file *f)
                return 1;
        }
 
-       if (nd->type == FIO_TYPE_UDP)
+#ifdef CONFIG_TCP_NODELAY
+       if (o->nodelay && o->proto == FIO_TYPE_TCP) {
+               int optval = 1;
+
+               if (setsockopt(f->fd, IPPROTO_TCP, TCP_NODELAY, (void *) &optval, sizeof(int)) < 0) {
+                       log_err("fio: cannot set TCP_NODELAY option on socket (%s), disable with 'nodelay=0'\n", strerror(errno));
+                       return 1;
+               }
+       }
+#endif
+
+       if (o->proto == FIO_TYPE_UDP)
                return 0;
-       else if (nd->type == FIO_TYPE_TCP) {
-               fio_socklen_t len = sizeof(nd->addr);
+       else if (o->proto == FIO_TYPE_TCP) {
+               socklen_t len = sizeof(nd->addr);
 
                if (connect(f->fd, (struct sockaddr *) &nd->addr, len) < 0) {
                        td_verror(td, errno, "connect");
@@ -394,7 +516,7 @@ static int fio_netio_connect(struct thread_data *td, struct fio_file *f)
                }
        } else {
                struct sockaddr_un *addr = &nd->addr_un;
-               fio_socklen_t len;
+               socklen_t len;
 
                len = sizeof(addr->sun_family) + strlen(addr->sun_path) + 1;
 
@@ -411,39 +533,46 @@ static int fio_netio_connect(struct thread_data *td, struct fio_file *f)
 static int fio_netio_accept(struct thread_data *td, struct fio_file *f)
 {
        struct netio_data *nd = td->io_ops->data;
-       fio_socklen_t socklen = sizeof(nd->addr);
+       struct netio_options *o = td->eo;
+       socklen_t socklen = sizeof(nd->addr);
+       int state;
 
-       if (nd->type == FIO_TYPE_UDP) {
+       if (o->proto == FIO_TYPE_UDP) {
                f->fd = nd->listenfd;
                return 0;
        }
 
+       state = td->runstate;
+       td_set_runstate(td, TD_SETTING_UP);
+
        log_info("fio: waiting for connection\n");
 
        if (poll_wait(td, nd->listenfd, POLLIN) < 0)
-               return 1;
+               goto err;
 
        f->fd = accept(nd->listenfd, (struct sockaddr *) &nd->addr, &socklen);
        if (f->fd < 0) {
                td_verror(td, errno, "accept");
-               return 1;
+               goto err;
        }
 
-       return 0;
-}
+#ifdef CONFIG_TCP_NODELAY
+       if (o->nodelay && o->proto == FIO_TYPE_TCP) {
+               int optval = 1;
 
-static int fio_netio_open_file(struct thread_data *td, struct fio_file *f)
-{
-       int ret;
-
-       if (td_read(td))
-               ret = fio_netio_accept(td, f);
-       else
-               ret = fio_netio_connect(td, f);
+               if (setsockopt(f->fd, IPPROTO_TCP, TCP_NODELAY, (void *) &optval, sizeof(int)) < 0) {
+                       log_err("fio: cannot set TCP_NODELAY option on socket (%s), disable with 'nodelay=0'\n", strerror(errno));
+                       return 1;
+               }
+       }
+#endif
 
-       if (ret)
-               f->fd = -1;
-       return ret;
+       reset_all_stats(td);
+       td_set_runstate(td, state);
+       return 0;
+err:
+       td_set_runstate(td, state);
+       return 1;
 }
 
 static void fio_netio_udp_close(struct thread_data *td, struct fio_file *f)
@@ -453,10 +582,10 @@ static void fio_netio_udp_close(struct thread_data *td, struct fio_file *f)
        struct sockaddr *to = (struct sockaddr *) &nd->addr;
        int ret;
 
-       msg.magic = htonl(FIO_LINK_CLOSE_MAGIC);
+       msg.magic = htonl(FIO_LINK_OPEN_CLOSE_MAGIC);
        msg.cmd = htonl(FIO_LINK_CLOSE);
 
-       ret = sendto(f->fd, &msg, sizeof(msg), MSG_WAITALL, to,
+       ret = sendto(f->fd, (void *) &msg, sizeof(msg), MSG_WAITALL, to,
                        sizeof(nd->addr));
        if (ret < 0)
                td_verror(td, errno, "sendto udp link close");
@@ -464,23 +593,110 @@ static void fio_netio_udp_close(struct thread_data *td, struct fio_file *f)
 
 static int fio_netio_close_file(struct thread_data *td, struct fio_file *f)
 {
-       struct netio_data *nd = td->io_ops->data;
+       struct netio_options *o = td->eo;
 
        /*
         * If this is an UDP connection, notify the receiver that we are
         * closing down the link
         */
-       if (nd->type == FIO_TYPE_UDP)
+       if (o->proto == FIO_TYPE_UDP)
                fio_netio_udp_close(td, f);
 
        return generic_close_file(td, f);
 }
 
+static int fio_netio_udp_recv_open(struct thread_data *td, struct fio_file *f)
+{
+       struct netio_data *nd = td->io_ops->data;
+       struct udp_close_msg msg;
+       struct sockaddr *to = (struct sockaddr *) &nd->addr;
+       socklen_t len = sizeof(nd->addr);
+       int ret;
+
+       ret = recvfrom(f->fd, (void *) &msg, sizeof(msg), MSG_WAITALL, to, &len);
+       if (ret < 0) {
+               td_verror(td, errno, "sendto udp link open");
+               return ret;
+       }
+
+       if (ntohl(msg.magic) != FIO_LINK_OPEN_CLOSE_MAGIC ||
+           ntohl(msg.cmd) != FIO_LINK_OPEN) {
+               log_err("fio: bad udp open magic %x/%x\n", ntohl(msg.magic),
+                                                               ntohl(msg.cmd));
+               return -1;
+       }
+
+       return 0;
+}
+
+static int fio_netio_udp_send_open(struct thread_data *td, struct fio_file *f)
+{
+       struct netio_data *nd = td->io_ops->data;
+       struct udp_close_msg msg;
+       struct sockaddr *to = (struct sockaddr *) &nd->addr;
+       int ret;
+
+       msg.magic = htonl(FIO_LINK_OPEN_CLOSE_MAGIC);
+       msg.cmd = htonl(FIO_LINK_OPEN);
+
+       ret = sendto(f->fd, (void *) &msg, sizeof(msg), MSG_WAITALL, to,
+                       sizeof(nd->addr));
+       if (ret < 0) {
+               td_verror(td, errno, "sendto udp link open");
+               return ret;
+       }
+
+       return 0;
+}
+
+static int fio_netio_open_file(struct thread_data *td, struct fio_file *f)
+{
+       int ret;
+       struct netio_options *o = td->eo;
+
+       if (o->listen)
+               ret = fio_netio_accept(td, f);
+       else
+               ret = fio_netio_connect(td, f);
+
+       if (ret) {
+               f->fd = -1;
+               return ret;
+       }
+
+       if (o->proto == FIO_TYPE_UDP) {
+               if (td_write(td))
+                       ret = fio_netio_udp_send_open(td, f);
+               else {
+                       int state;
+
+                       state = td->runstate;
+                       td_set_runstate(td, TD_SETTING_UP);
+                       ret = fio_netio_udp_recv_open(td, f);
+                       td_set_runstate(td, state);
+               }
+       }
+
+       if (ret)
+               fio_netio_close_file(td, f);
+
+       return ret;
+}
+
 static int fio_netio_setup_connect_inet(struct thread_data *td,
                                        const char *host, unsigned short port)
 {
        struct netio_data *nd = td->io_ops->data;
 
+       if (!host) {
+               log_err("fio: connect with no host to connect to.\n");
+               if (td_read(td))
+                       log_err("fio: did you forget to set 'listen'?\n");
+
+               td_verror(td, EINVAL, "no hostname= set");
+               return 1;
+       }
+
        nd->addr.sin_family = AF_INET;
        nd->addr.sin_port = htons(port);
 
@@ -510,15 +726,14 @@ static int fio_netio_setup_connect_unix(struct thread_data *td,
        return 0;
 }
 
-static int fio_netio_setup_connect(struct thread_data *td, const char *host,
-                                  unsigned short port)
+static int fio_netio_setup_connect(struct thread_data *td)
 {
-       struct netio_data *nd = td->io_ops->data;
+       struct netio_options *o = td->eo;
 
-       if (nd->type == FIO_TYPE_UDP || nd->type == FIO_TYPE_TCP)
-               return fio_netio_setup_connect_inet(td, host, port);
+       if (o->proto == FIO_TYPE_UDP || o->proto == FIO_TYPE_TCP)
+               return fio_netio_setup_connect_inet(td, td->o.filename,o->port);
        else
-               return fio_netio_setup_connect_unix(td, host);
+               return fio_netio_setup_connect_unix(td, td->o.filename);
 }
 
 static int fio_netio_setup_listen_unix(struct thread_data *td, const char *path)
@@ -557,9 +772,10 @@ static int fio_netio_setup_listen_unix(struct thread_data *td, const char *path)
 static int fio_netio_setup_listen_inet(struct thread_data *td, short port)
 {
        struct netio_data *nd = td->io_ops->data;
+       struct netio_options *o = td->eo;
        int fd, opt, type;
 
-       if (nd->type == FIO_TYPE_TCP)
+       if (o->proto == FIO_TYPE_TCP)
                type = SOCK_STREAM;
        else
                type = SOCK_DGRAM;
@@ -571,12 +787,12 @@ static int fio_netio_setup_listen_inet(struct thread_data *td, short port)
        }
 
        opt = 1;
-       if (setsockopt(fd, SOL_SOCKET, SO_REUSEADDR, &opt, sizeof(opt)) < 0) {
+       if (setsockopt(fd, SOL_SOCKET, SO_REUSEADDR, (void *) &opt, sizeof(opt)) < 0) {
                td_verror(td, errno, "setsockopt");
                return 1;
        }
 #ifdef SO_REUSEPORT
-       if (setsockopt(fd, SOL_SOCKET, SO_REUSEPORT, &opt, sizeof(opt)) < 0) {
+       if (setsockopt(fd, SOL_SOCKET, SO_REUSEPORT, (void *) &opt, sizeof(opt)) < 0) {
                td_verror(td, errno, "setsockopt");
                return 1;
        }
@@ -595,20 +811,20 @@ static int fio_netio_setup_listen_inet(struct thread_data *td, short port)
        return 0;
 }
 
-static int fio_netio_setup_listen(struct thread_data *td, const char *path,
-                                 short port)
+static int fio_netio_setup_listen(struct thread_data *td)
 {
        struct netio_data *nd = td->io_ops->data;
+       struct netio_options *o = td->eo;
        int ret;
 
-       if (nd->type == FIO_TYPE_UDP || nd->type == FIO_TYPE_TCP)
-               ret = fio_netio_setup_listen_inet(td, port);
+       if (o->proto == FIO_TYPE_UDP || o->proto == FIO_TYPE_TCP)
+               ret = fio_netio_setup_listen_inet(td, o->port);
        else
-               ret = fio_netio_setup_listen_unix(td, path);
+               ret = fio_netio_setup_listen_unix(td, td->o.filename);
 
        if (ret)
                return ret;
-       if (nd->type == FIO_TYPE_UDP)
+       if (o->proto == FIO_TYPE_UDP)
                return 0;
 
        if (listen(nd->listenfd, 10) < 0) {
@@ -622,72 +838,55 @@ static int fio_netio_setup_listen(struct thread_data *td, const char *path,
 
 static int fio_netio_init(struct thread_data *td)
 {
-       struct netio_data *nd = td->io_ops->data;
-       unsigned int port;
-       char host[64], buf[128];
-       char *sep, *portp, *modep;
+       struct netio_options *o = td->eo;
        int ret;
 
-       if (td_rw(td)) {
-               log_err("fio: network connections must be read OR write\n");
-               return 1;
-       }
+#ifdef WIN32
+       WSADATA wsd;
+       WSAStartup(MAKEWORD(2,2), &wsd);
+#endif
+
        if (td_random(td)) {
                log_err("fio: network IO can't be random\n");
                return 1;
        }
 
-       strcpy(buf, td->o.filename);
-
-       sep = strchr(buf, ',');
-       if (!sep)
-               goto bad_host;
+       if (o->proto == FIO_TYPE_UNIX && o->port) {
+               log_err("fio: network IO port not valid with unix socket\n");
+               return 1;
+       } else if (o->proto != FIO_TYPE_UNIX && !o->port) {
+               log_err("fio: network IO requires port for tcp or udp\n");
+               return 1;
+       }
 
-       *sep = '\0';
-       sep++;
-       strcpy(host, buf);
-       if (!strlen(host))
-               goto bad_host;
+       if (o->proto != FIO_TYPE_TCP) {
+               if (o->listen) {
+                       log_err("fio: listen only valid for TCP proto IO\n");
+                       return 1;
+               }
+               if (td_rw(td)) {
+                       log_err("fio: datagram network connections must be"
+                                  " read OR write\n");
+                       return 1;
+               }
+               if (o->proto == FIO_TYPE_UNIX && !td->o.filename) {
+                       log_err("fio: UNIX sockets need host/filename\n");
+                       return 1;
+               }
+               o->listen = td_read(td);
+       }
 
-       modep = NULL;
-       portp = sep;
-       sep = strchr(portp, ',');
-       if (sep) {
-               *sep = '\0';
-               modep = sep + 1;
+       if (o->proto != FIO_TYPE_UNIX && o->listen && td->o.filename) {
+               log_err("fio: hostname not valid for inbound network IO\n");
+               return 1;
        }
 
-       if (!strncmp("tcp", modep, strlen(modep)) ||
-           !strncmp("TCP", modep, strlen(modep)))
-               nd->type = FIO_TYPE_TCP;
-       else if (!strncmp("udp", modep, strlen(modep)) ||
-                !strncmp("UDP", modep, strlen(modep)))
-               nd->type = FIO_TYPE_UDP;
-       else if (!strncmp("unix", modep, strlen(modep)) ||
-                !strncmp("UNIX", modep, strlen(modep)))
-               nd->type = FIO_TYPE_UNIX;
+       if (o->listen)
+               ret = fio_netio_setup_listen(td);
        else
-               goto bad_host;
-
-       if (nd->type != FIO_TYPE_UNIX) {
-               port = strtol(portp, NULL, 10);
-               if (!port || port > 65535)
-                       goto bad_host;
-       } else
-               port = 0;
-
-       if (td_read(td)) {
-               nd->send_to_net = 0;
-               ret = fio_netio_setup_listen(td, host, port);
-       } else {
-               nd->send_to_net = 1;
-               ret = fio_netio_setup_connect(td, host, port);
-       }
+               ret = fio_netio_setup_connect(td);
 
        return ret;
-bad_host:
-       log_err("fio: bad network host/port/protocol: %s\n", td->o.filename);
-       return 1;
 }
 
 static void fio_netio_cleanup(struct thread_data *td)
@@ -710,6 +909,11 @@ static int fio_netio_setup(struct thread_data *td)
 {
        struct netio_data *nd;
 
+       if (!td->files_index) {
+               add_file(td, td->o.filename ?: "net");
+               td->o.nr_files = td->o.nr_files ?: 1;
+       }
+
        if (!td->io_ops->data) {
                nd = malloc(sizeof(*nd));;
 
@@ -722,7 +926,12 @@ static int fio_netio_setup(struct thread_data *td)
        return 0;
 }
 
-#ifdef FIO_HAVE_SPLICE
+static void fio_netio_terminate(struct thread_data *td)
+{
+       kill(td->pid, SIGUSR2);
+}
+
+#ifdef CONFIG_LINUX_SPLICE
 static int fio_netio_setup_splice(struct thread_data *td)
 {
        struct netio_data *nd;
@@ -742,38 +951,54 @@ static int fio_netio_setup_splice(struct thread_data *td)
 }
 
 static struct ioengine_ops ioengine_splice = {
-       .name           = "netsplice",
-       .version        = FIO_IOOPS_VERSION,
-       .prep           = fio_netio_prep,
-       .queue          = fio_netio_queue,
-       .setup          = fio_netio_setup_splice,
-       .init           = fio_netio_init,
-       .cleanup        = fio_netio_cleanup,
-       .open_file      = fio_netio_open_file,
-       .close_file     = generic_close_file,
-       .flags          = FIO_SYNCIO | FIO_DISKLESSIO | FIO_UNIDIR |
-                         FIO_SIGTERM | FIO_PIPEIO,
+       .name                   = "netsplice",
+       .version                = FIO_IOOPS_VERSION,
+       .prep                   = fio_netio_prep,
+       .queue                  = fio_netio_queue,
+       .setup                  = fio_netio_setup_splice,
+       .init                   = fio_netio_init,
+       .cleanup                = fio_netio_cleanup,
+       .open_file              = fio_netio_open_file,
+       .close_file             = fio_netio_close_file,
+       .terminate              = fio_netio_terminate,
+       .options                = options,
+       .option_struct_size     = sizeof(struct netio_options),
+       .flags                  = FIO_SYNCIO | FIO_DISKLESSIO | FIO_UNIDIR |
+                                 FIO_PIPEIO,
 };
 #endif
 
 static struct ioengine_ops ioengine_rw = {
-       .name           = "net",
-       .version        = FIO_IOOPS_VERSION,
-       .prep           = fio_netio_prep,
-       .queue          = fio_netio_queue,
-       .setup          = fio_netio_setup,
-       .init           = fio_netio_init,
-       .cleanup        = fio_netio_cleanup,
-       .open_file      = fio_netio_open_file,
-       .close_file     = fio_netio_close_file,
-       .flags          = FIO_SYNCIO | FIO_DISKLESSIO | FIO_UNIDIR |
-                         FIO_SIGTERM | FIO_PIPEIO,
+       .name                   = "net",
+       .version                = FIO_IOOPS_VERSION,
+       .prep                   = fio_netio_prep,
+       .queue                  = fio_netio_queue,
+       .setup                  = fio_netio_setup,
+       .init                   = fio_netio_init,
+       .cleanup                = fio_netio_cleanup,
+       .open_file              = fio_netio_open_file,
+       .close_file             = fio_netio_close_file,
+       .terminate              = fio_netio_terminate,
+       .options                = options,
+       .option_struct_size     = sizeof(struct netio_options),
+       .flags                  = FIO_SYNCIO | FIO_DISKLESSIO | FIO_UNIDIR |
+                                 FIO_PIPEIO | FIO_BIT_BASED,
 };
 
+static int str_hostname_cb(void *data, const char *input)
+{
+       struct netio_options *o = data;
+
+       if (o->td->o.filename)
+               free(o->td->o.filename);
+       o->td->o.filename = strdup(input);
+       return 0;
+}
+
 static void fio_init fio_netio_register(void)
 {
        register_ioengine(&ioengine_rw);
-#ifdef FIO_HAVE_SPLICE
+#ifdef CONFIG_LINUX_SPLICE
        register_ioengine(&ioengine_splice);
 #endif
 }
@@ -781,7 +1006,7 @@ static void fio_init fio_netio_register(void)
 static void fio_exit fio_netio_unregister(void)
 {
        unregister_ioengine(&ioengine_rw);
-#ifdef FIO_HAVE_SPLICE
+#ifdef CONFIG_LINUX_SPLICE
        unregister_ioengine(&ioengine_splice);
 #endif
 }