[src/trunk]: src/tests/kernel Add new tests dbregs[12] in t_ptrace_wait{, 3, 4, ...

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

details:   https://anonhg.NetBSD.org/src/rev/410a83b1b0be
branches:  trunk
changeset: 819348:410a83b1b0be
user:      kamil <kamil%NetBSD.org@localhost>
date:      Tue Nov 29 21:58:13 2016 +0000

description:
Add new tests dbregs[12] in t_ptrace_wait{,3,4,6,id,pid}

dbregs1:
    Verify plain PT_GETDBREGS call without further steps

dbregs2:
    Verify PT_GETDBREGS and PT_SETDBREGS calls without changing regs

These tests are to be used to verify CPU Debug Register accessors in the
ptrace(2) interface.

Additionally fix also fpregs2 test to really call PT_SETFPREGS.

Sponsored by <The NetBSD Foundation>

diffstat:

 tests/kernel/t_ptrace_wait.c |  135 +++++++++++++++++++++++++++++++++++++++++-
 1 files changed, 131 insertions(+), 4 deletions(-)

diffs (184 lines):

diff -r 65ab60b15372 -r 410a83b1b0be tests/kernel/t_ptrace_wait.c
--- a/tests/kernel/t_ptrace_wait.c      Tue Nov 29 21:54:11 2016 +0000
+++ b/tests/kernel/t_ptrace_wait.c      Tue Nov 29 21:58:13 2016 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: t_ptrace_wait.c,v 1.34 2016/11/29 21:31:45 kamil Exp $ */
+/*     $NetBSD: t_ptrace_wait.c,v 1.35 2016/11/29 21:58:13 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.34 2016/11/29 21:31:45 kamil Exp $");
+__RCSID("$NetBSD: t_ptrace_wait.c,v 1.35 2016/11/29 21:58:13 kamil Exp $");
 
 #include <sys/param.h>
 #include <sys/types.h>
@@ -202,6 +202,12 @@
 #define HAVE_FPREGS
 #endif
 
+/* Add guards for cpu debug registers */
+#if defined(PT_GETDBREGS)              \
+    && defined(PT_SETDBREGS)
+#define HAVE_DBREGS
+#endif
+
 /*
  * If waitid(2) returns because one or more processes have a state change to
  * report, 0 is returned.  If an error is detected, a value of -1 is returned
@@ -4414,7 +4420,7 @@
 
        validate_status_stopped(status, sigval);
 
-       printf("Call GETREGS for the child process\n");
+       printf("Call GETFPREGS for the child process\n");
        ATF_REQUIRE(ptrace(PT_GETFPREGS, child, &r, 0) != -1);
 
        printf("Before resuming the child process where it left off and "
@@ -4473,7 +4479,119 @@
        ATF_REQUIRE(ptrace(PT_GETFPREGS, child, &r, 0) != -1);
 
        printf("Call SETFPREGS for the child (without changed regs)\n");
-       ATF_REQUIRE(ptrace(PT_GETFPREGS, child, &r, 0) != -1);
+       ATF_REQUIRE(ptrace(PT_SETFPREGS, child, &r, 0) != -1);
+
+       printf("Before resuming the child process where it left off and "
+           "without signal to be sent\n");
+       ATF_REQUIRE(ptrace(PT_CONTINUE, child, (void *)1, 0) != -1);
+
+       printf("Before calling %s() for the child\n", TWAIT_FNAME);
+       TWAIT_REQUIRE_SUCCESS(wpid = TWAIT_GENERIC(child, &status, 0), child);
+
+       validate_status_exited(status, exitval);
+
+       printf("Before calling %s() for the child\n", TWAIT_FNAME);
+       TWAIT_REQUIRE_FAILURE(ECHILD, wpid = TWAIT_GENERIC(child, &status, 0));
+}
+#endif
+
+#if defined(HAVE_DBREGS)
+ATF_TC(dbregs1);
+ATF_TC_HEAD(dbregs1, tc)
+{
+       atf_tc_set_md_var(tc, "descr",
+           "Verify plain PT_GETDBREGS call without further steps");
+}
+
+ATF_TC_BODY(dbregs1, tc)
+{
+       const int exitval = 5;
+       const int sigval = SIGSTOP;
+       pid_t child, wpid;
+#if defined(TWAIT_HAVE_STATUS)
+       int status;
+#endif
+       struct dbreg r;
+
+       printf("Before forking process PID=%d\n", getpid());
+       child = atf_utils_fork();
+       if (child == 0) {
+               printf("Before calling PT_TRACE_ME from child %d\n", getpid());
+               FORKEE_ASSERT(ptrace(PT_TRACE_ME, 0, NULL, 0) != -1);
+
+               printf("Before raising %s from child\n", strsignal(sigval));
+               FORKEE_ASSERT(raise(sigval) == 0);
+
+               printf("Before exiting of the child process\n");
+               _exit(exitval);
+       }
+       printf("Parent process PID=%d, child's PID=%d\n", getpid(), child);
+
+       printf("Before calling %s() for the child\n", TWAIT_FNAME);
+       TWAIT_REQUIRE_SUCCESS(wpid = TWAIT_GENERIC(child, &status, 0), child);
+
+       validate_status_stopped(status, sigval);
+
+       printf("Call GETDBREGS for the child process\n");
+       ATF_REQUIRE(ptrace(PT_GETDBREGS, child, &r, 0) != -1);
+
+       printf("Before resuming the child process where it left off and "
+           "without signal to be sent\n");
+       ATF_REQUIRE(ptrace(PT_CONTINUE, child, (void *)1, 0) != -1);
+
+       printf("Before calling %s() for the child\n", TWAIT_FNAME);
+       TWAIT_REQUIRE_SUCCESS(wpid = TWAIT_GENERIC(child, &status, 0), child);
+
+       validate_status_exited(status, exitval);
+
+       printf("Before calling %s() for the child\n", TWAIT_FNAME);
+       TWAIT_REQUIRE_FAILURE(ECHILD, wpid = TWAIT_GENERIC(child, &status, 0));
+}
+#endif
+
+#if defined(HAVE_DBREGS)
+ATF_TC(dbregs2);
+ATF_TC_HEAD(dbregs2, tc)
+{
+       atf_tc_set_md_var(tc, "descr",
+           "Verify PT_GETDBREGS and PT_SETDBREGS calls without changing "
+           "regs");
+}
+
+ATF_TC_BODY(dbregs2, tc)
+{
+       const int exitval = 5;
+       const int sigval = SIGSTOP;
+       pid_t child, wpid;
+#if defined(TWAIT_HAVE_STATUS)
+       int status;
+#endif
+       struct dbreg r;
+
+       printf("Before forking process PID=%d\n", getpid());
+       child = atf_utils_fork();
+       if (child == 0) {
+               printf("Before calling PT_TRACE_ME from child %d\n", getpid());
+               FORKEE_ASSERT(ptrace(PT_TRACE_ME, 0, NULL, 0) != -1);
+
+               printf("Before raising %s from child\n", strsignal(sigval));
+               FORKEE_ASSERT(raise(sigval) == 0);
+
+               printf("Before exiting of the child process\n");
+               _exit(exitval);
+       }
+       printf("Parent process PID=%d, child's PID=%d\n", getpid(), child);
+
+       printf("Before calling %s() for the child\n", TWAIT_FNAME);
+       TWAIT_REQUIRE_SUCCESS(wpid = TWAIT_GENERIC(child, &status, 0), child);
+
+       validate_status_stopped(status, sigval);
+
+       printf("Call GETDBREGS for the child process\n");
+       ATF_REQUIRE(ptrace(PT_GETDBREGS, child, &r, 0) != -1);
+
+       printf("Call SETDBREGS for the child (without changed regs)\n");
+       ATF_REQUIRE(ptrace(PT_SETDBREGS, child, &r, 0) != -1);
 
        printf("Before resuming the child process where it left off and "
            "without signal to be sent\n");
@@ -4775,6 +4893,12 @@
 #define ATF_TP_ADD_TC_HAVE_FPREGS(a,b)
 #endif
 
+#if defined(HAVE_DBREGS)
+#define ATF_TP_ADD_TC_HAVE_DBREGS(a,b) ATF_TP_ADD_TC(a,b)
+#else
+#define ATF_TP_ADD_TC_HAVE_DBREGS(a,b)
+#endif
+
 #if defined(PT_STEP)
 #define ATF_TP_ADD_TC_PT_STEP(a,b)     ATF_TP_ADD_TC(a,b)
 #else
@@ -4852,6 +4976,9 @@
        ATF_TP_ADD_TC_HAVE_FPREGS(tp, fpregs1);
        ATF_TP_ADD_TC_HAVE_FPREGS(tp, fpregs2);
 
+       ATF_TP_ADD_TC_HAVE_DBREGS(tp, dbregs1);
+       ATF_TP_ADD_TC_HAVE_DBREGS(tp, dbregs2);
+
        ATF_TP_ADD_TC_PT_STEP(tp, step1);
        ATF_TP_ADD_TC_PT_STEP(tp, step2);
        ATF_TP_ADD_TC_PT_STEP(tp, step3);