[src/trunk]: src/tests/lib/libc/sys ATF: Reenable attach2 in t_ptrace_wait*

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

details:   https://anonhg.NetBSD.org/src/rev/86250926a97a
branches:  trunk
changeset: 321244:86250926a97a
user:      kamil <kamil%NetBSD.org@localhost>
date:      Wed Mar 07 16:40:06 2018 +0000

description:
ATF: Reenable attach2 in t_ptrace_wait*

The primary race specific to this test has been fixed in previous commit
(wrong WNOHANG).

This test is still racy and breaks like once every 30,000 execution.
This is down like from once from every 100th execution in the past.

The remaning race is not specific to attach2 and I can reproduce it with
at least attach1. It still looks like being specific to NetBSD and it's
not reproducible on Linux and FreeBSD. Perhaps a bug with pipe(2)/write(2)/
read(2) or close to these features.

Sponsored by <The NetBSD Foundation>

diffstat:

 tests/lib/libc/sys/t_ptrace_wait.c |  10 ++--------
 1 files changed, 2 insertions(+), 8 deletions(-)

diffs (38 lines):

diff -r bb7cadc7a724 -r 86250926a97a tests/lib/libc/sys/t_ptrace_wait.c
--- a/tests/lib/libc/sys/t_ptrace_wait.c        Wed Mar 07 16:06:57 2018 +0000
+++ b/tests/lib/libc/sys/t_ptrace_wait.c        Wed Mar 07 16:40:06 2018 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: t_ptrace_wait.c,v 1.24 2018/03/07 10:53:55 kamil Exp $ */
+/*     $NetBSD: t_ptrace_wait.c,v 1.25 2018/03/07 16:40:06 kamil Exp $ */
 
 /*-
  * Copyright (c) 2016 The NetBSD Foundation, Inc.
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: t_ptrace_wait.c,v 1.24 2018/03/07 10:53:55 kamil Exp $");
+__RCSID("$NetBSD: t_ptrace_wait.c,v 1.25 2018/03/07 16:40:06 kamil Exp $");
 
 #include <sys/param.h>
 #include <sys/types.h>
@@ -424,9 +424,6 @@
        int status;
 #endif
 
-       // Feature pending for refactoring
-       atf_tc_expect_fail("test is racy");
-
        DPRINTF("Spawn tracee\n");
        SYSCALL_REQUIRE(msg_open(&parent_tracee) == 0);
        tracee = atf_utils_fork();
@@ -507,9 +504,6 @@
 
        msg_close(&parent_tracer);
        msg_close(&parent_tracee);
-
-       // Test is racy
-       ATF_REQUIRE(0 && "In order to get reliable failure, abort");
 }
 #endif