Module Name: src Committed By: kamil Date: Tue Nov 29 21:58:14 UTC 2016
Modified Files: src/tests/kernel: t_ptrace_wait.c Log Message: 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> To generate a diff of this commit: cvs rdiff -u -r1.34 -r1.35 src/tests/kernel/t_ptrace_wait.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/tests/kernel/t_ptrace_wait.c diff -u src/tests/kernel/t_ptrace_wait.c:1.34 src/tests/kernel/t_ptrace_wait.c:1.35 --- src/tests/kernel/t_ptrace_wait.c:1.34 Tue Nov 29 21:31:45 2016 +++ src/tests/kernel/t_ptrace_wait.c Tue Nov 29 21:58:13 2016 @@ -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 @@ do { \ #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 @@ ATF_TC_BODY(fpregs1, tc) 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_TC_BODY(fpregs2, tc) 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 @@ ATF_TC_BODY(step4, tc) #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_TCS(tp) 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);