Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/tests/lib/libc/sys Add a fifo_inout test case that validates...



details:   https://anonhg.NetBSD.org/src/rev/0ee6b4c4ae38
branches:  trunk
changeset: 988195:0ee6b4c4ae38
user:      thorpej <thorpej%NetBSD.org@localhost>
date:      Sat Oct 02 15:50:06 2021 +0000

description:
Add a fifo_inout test case that validates the expected POLLIN / POLLOUT
behavior for FIFOs:
- A FIFO is readable so long as at least 1 byte is available.
- A FIFO is writable so long as at least PIPE_BUF (obtained with _PC_PIPE_BUF)
  space is avaiable.

This will be cloned for a forthcoming kevent test case.

diffstat:

 tests/lib/libc/sys/t_poll.c |  102 +++++++++++++++++++++++++++++++++++++++++++-
 1 files changed, 101 insertions(+), 1 deletions(-)

diffs (130 lines):

diff -r 6e1b916851f1 -r 0ee6b4c4ae38 tests/lib/libc/sys/t_poll.c
--- a/tests/lib/libc/sys/t_poll.c       Sat Oct 02 14:41:36 2021 +0000
+++ b/tests/lib/libc/sys/t_poll.c       Sat Oct 02 15:50:06 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: t_poll.c,v 1.6 2021/10/02 14:41:36 thorpej Exp $       */
+/*     $NetBSD: t_poll.c,v 1.7 2021/10/02 15:50:06 thorpej Exp $       */
 
 /*-
  * Copyright (c) 2011 The NetBSD Foundation, Inc.
@@ -39,6 +39,7 @@
 #include <paths.h>
 #include <poll.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <signal.h>
 #include <unistd.h>
 
@@ -252,6 +253,104 @@
        }
 }
 
+ATF_TC_WITH_CLEANUP(fifo_inout);
+ATF_TC_HEAD(fifo_inout, tc)
+{
+       atf_tc_set_md_var(tc, "descr",
+           "Check POLLIN/POLLOUT behavior with fifos");
+}
+
+ATF_TC_BODY(fifo_inout, tc)
+{
+       struct pollfd pfd[2];
+       char *buf;
+       int rfd, wfd;
+       long pipe_buf;
+
+       fifo_support();
+
+       ATF_REQUIRE(mkfifo(fifo_path, 0600) == 0);
+       ATF_REQUIRE((rfd = open(fifo_path, O_RDONLY | O_NONBLOCK)) >= 0);
+       ATF_REQUIRE((wfd = open(fifo_path, O_WRONLY | O_NONBLOCK)) >= 0);
+
+       /* Get the maximum atomic pipe write size. */
+       pipe_buf = fpathconf(wfd, _PC_PIPE_BUF);
+       ATF_REQUIRE(pipe_buf > 1);
+
+       buf = malloc(pipe_buf);
+       ATF_REQUIRE(buf != NULL);
+
+       memset(&pfd, 0, sizeof(pfd));
+       pfd[0].fd = rfd;
+       pfd[0].events = POLLIN | POLLRDNORM;
+       pfd[1].fd = wfd;
+       pfd[1].events = POLLOUT | POLLWRNORM;
+
+       /* We expect the FIFO to be writable but not readable. */
+       ATF_REQUIRE(poll(pfd, 2, 0) == 1);
+       ATF_REQUIRE(pfd[0].revents == 0);
+       ATF_REQUIRE(pfd[1].revents == (POLLOUT | POLLWRNORM));
+
+       /* Write a single byte of data into the FIFO. */
+       ATF_REQUIRE(write(wfd, buf, 1) == 1);
+
+       /* We expect the FIFO to be readable and writable. */
+       ATF_REQUIRE(poll(pfd, 2, 0) == 2);
+       ATF_REQUIRE(pfd[0].revents == (POLLIN | POLLRDNORM));
+       ATF_REQUIRE(pfd[1].revents == (POLLOUT | POLLWRNORM));
+
+       /* Read that single byte back out. */
+       ATF_REQUIRE(read(rfd, buf, 1) == 1);
+
+       /*
+        * Write data into the FIFO until it is full, which is
+        * defined as insufficient buffer space to hold a the
+        * maximum atomic pipe write size.
+        */
+       while (write(wfd, buf, pipe_buf) != -1) {
+               continue;
+       }
+       ATF_REQUIRE(errno == EAGAIN);
+
+       /* We expect the FIFO to be readble but not writable. */
+       ATF_REQUIRE(poll(pfd, 2, 0) == 1);
+       ATF_REQUIRE(pfd[0].revents == (POLLIN | POLLRDNORM));
+       ATF_REQUIRE(pfd[1].revents == 0);
+
+       /* Read a single byte of data from the FIFO. */
+       ATF_REQUIRE(read(rfd, buf, 1) == 1);
+
+       /*
+        * Because we have read only a single byte out, there will
+        * be insufficient space for a pipe_buf-sized message, so
+        * the FIFO should still not be writable.
+        */
+       ATF_REQUIRE(poll(pfd, 2, 0) == 1);
+       ATF_REQUIRE(pfd[0].revents == (POLLIN | POLLRDNORM));
+       ATF_REQUIRE(pfd[1].revents == 0);
+
+       /*
+        * Now read all of the data out of the FIFO and ensure that
+        * we get back to the initial state.
+        */
+       while (read(rfd, buf, pipe_buf) != -1) {
+               continue;
+       }
+       ATF_REQUIRE(errno == EAGAIN);
+
+       ATF_REQUIRE(poll(pfd, 2, 0) == 1);
+       ATF_REQUIRE(pfd[0].revents == 0);
+       ATF_REQUIRE(pfd[1].revents == (POLLOUT | POLLWRNORM));
+
+       (void)close(wfd);
+       (void)close(rfd);
+}
+
+ATF_TC_CLEANUP(fifo_inout, tc)
+{
+       (void)unlink(fifo_path);
+}
+
 ATF_TC_WITH_CLEANUP(fifo_hup1);
 ATF_TC_HEAD(fifo_hup1, tc)
 {
@@ -354,6 +453,7 @@
        ATF_TP_ADD_TC(tp, basic);
        ATF_TP_ADD_TC(tp, err);
 
+       ATF_TP_ADD_TC(tp, fifo_inout);
        ATF_TP_ADD_TC(tp, fifo_hup1);
        ATF_TP_ADD_TC(tp, fifo_hup2);
 



Home | Main Index | Thread Index | Old Index