[src/trunk]: src/tests/lib/libc/sys should terminate and does with ktrace, bu...

[christos <christos%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/668b94df9cfb
branches:  trunk
changeset: 834635:668b94df9cfb
user:      christos <christos%NetBSD.org@localhost>
date:      Tue Aug 21 11:03:27 2018 +0000

description:
should terminate and does with ktrace, but it does not normally.

diffstat:

 tests/lib/libc/sys/t_sendrecv.c |  48 ++++++++++++++++++++++++++++------------
 1 files changed, 33 insertions(+), 15 deletions(-)

diffs (125 lines):

diff -r 642d04e0e8d9 -r 668b94df9cfb tests/lib/libc/sys/t_sendrecv.c
--- a/tests/lib/libc/sys/t_sendrecv.c   Tue Aug 21 10:41:00 2018 +0000
+++ b/tests/lib/libc/sys/t_sendrecv.c   Tue Aug 21 11:03:27 2018 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: t_sendrecv.c,v 1.1 2018/08/21 10:41:00 christos Exp $  */
+/*     $NetBSD: t_sendrecv.c,v 1.2 2018/08/21 11:03:27 christos Exp $  */
 
 /*-
  * Copyright (c) 2018 The NetBSD Foundation, Inc.
@@ -15,13 +15,6 @@
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
- * 3. All advertising materials mentioning features or use of this software
- *    must display the following acknowledgement:
- *        This product includes software developed by the NetBSD
- *        Foundation, Inc. and its contributors.
- * 4. Neither the name of The NetBSD Foundation nor the names of its
- *    contributors may be used to endorse or promote products derived
- *    from this software without specific prior written permission.
  *
  * THIS SOFTWARE IS PROVIDED BY THE NETBSD FOUNDATION, INC. AND CONTRIBUTORS
  * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
@@ -36,7 +29,7 @@
  * POSSIBILITY OF SUCH DAMAGE.
  */
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: t_sendrecv.c,v 1.1 2018/08/21 10:41:00 christos Exp $");
+__RCSID("$NetBSD: t_sendrecv.c,v 1.2 2018/08/21 11:03:27 christos Exp $");
 
 #include <atf-c.h>
 #include <sys/types.h>
@@ -49,6 +42,7 @@
 #include <stdlib.h>
 #include <unistd.h>
 #include <sched.h>
+#include <signal.h>
 
 ATF_TC(sendrecv_basic);
 ATF_TC_HEAD(sendrecv_basic, tc)
@@ -56,13 +50,22 @@
        atf_tc_set_md_var(tc, "descr", "A basic test of sendrecv(2)");
 }
 
-#define COUNT 1000
+#define COUNT 100
 
 union packet {
        uint8_t buf[1316];
        uintmax_t seq;
 };
 
+static volatile sig_atomic_t rdied;
+
+static void
+handle_sigchld(__unused int pid)
+{
+
+       rdied = 1;
+}
+
 static void
 sender(int fd)
 {
@@ -73,9 +76,11 @@
                for (; (n = send(fd, &p, sizeof(p), 0)) == sizeof(p);
                    p.seq++)
                        continue;
-//             printf(">>%zd %d %ju\n", n, errno, p.seq);
+               printf(">>%zd %d %ju\n", n, errno, p.seq);
                ATF_REQUIRE_MSG(errno == ENOBUFS, "send %s", strerror(errno));
+               sched_yield();
        }
+       printf("sender done\n");
 }
 
 static void
@@ -85,12 +90,16 @@
        ssize_t n;
        uintmax_t seq = 0;
 
-       for (size_t i = 0; i < COUNT; i++) {
+       do {
+               if (rdied)
+                       return;
                while ((n = recv(fd, &p, sizeof(p), 0), sizeof(p))
                    == sizeof(p))
                {
-//                     if (p.seq != seq)
-//                             printf("%ju != %ju\n", p.seq, seq);
+                       if (rdied)
+                               return;
+                       if (p.seq != seq)
+                               printf("%ju != %ju\n", p.seq, seq);
                        seq = p.seq + 1;
                }
                printf("<<%zd %d %ju\n", n, errno, seq);
@@ -98,17 +107,26 @@
                        return;
                ATF_REQUIRE_EQ(n, -1);
                ATF_REQUIRE_MSG(errno == ENOBUFS, "recv %s", strerror(errno));
-       }
+       } while (p.seq < COUNT);
 }
 
 ATF_TC_BODY(sendrecv_basic, tc)
 {
        int fd[2], error;
+       struct sigaction sa;
 
        error = socketpair(AF_UNIX, SOCK_DGRAM, 0, fd);
 //     error = pipe(fd);
        ATF_REQUIRE_MSG(error != -1, "socketpair failed (%s)", strerror(errno));
 
+       memset(&sa, 0, sizeof(sa));
+       sa.sa_flags = SA_RESTART;
+       sa.sa_handler = &handle_sigchld;
+       sigemptyset(&sa.sa_mask);
+       error = sigaction(SIGCHLD, &sa, 0);
+       ATF_REQUIRE_MSG(error != -1, "sigaction failed (%s)",
+           strerror(errno));
+
        switch (fork()) {
        case -1:
                ATF_REQUIRE_MSG(errno == 0,