[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/daf4b45e1541
branches:  trunk
changeset: 831012:daf4b45e1541
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 142a866cf6ea -r daf4b45e1541 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