summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rw-r--r--ObsoleteFiles.inc3
-rw-r--r--bin/pwait/Makefile6
-rw-r--r--bin/pwait/pwait.130
-rw-r--r--bin/pwait/pwait.c66
-rw-r--r--bin/pwait/tests/Makefile5
-rw-r--r--bin/pwait/tests/Makefile.depend11
-rw-r--r--bin/pwait/tests/pwait_test.sh242
-rw-r--r--etc/mtree/BSD.tests.dist2
-rw-r--r--targets/pseudo/tests/Makefile.depend1
-rw-r--r--usr.bin/timeout/tests/Makefile2
-rw-r--r--usr.bin/timeout/tests/timeout_test.sh (renamed from usr.bin/timeout/tests/timeout.sh)34
11 files changed, 373 insertions, 29 deletions
diff --git a/ObsoleteFiles.inc b/ObsoleteFiles.inc
index ea5ffcd..e673120 100644
--- a/ObsoleteFiles.inc
+++ b/ObsoleteFiles.inc
@@ -38,6 +38,9 @@
# xargs -n1 | sort | uniq -d;
# done
+# 20170308: rename some tests
+OLD_FILES+=usr/tests/bin/pwait/pwait
+OLD_FILES+=usr/tests/usr.bin/timeout/timeout
# 20170214: Four files from ggate tests consolidated into one
OLD_FILES+=usr/tests/sys/geom/class/gate/1_test
OLD_FILES+=usr/tests/sys/geom/class/gate/2_test
diff --git a/bin/pwait/Makefile b/bin/pwait/Makefile
index a282c18..9d2c92d 100644
--- a/bin/pwait/Makefile
+++ b/bin/pwait/Makefile
@@ -1,6 +1,12 @@
# $FreeBSD$
+.include <src.opts.mk>
+
PACKAGE=runtime
PROG= pwait
+.if ${MK_TESTS} != "no"
+SUBDIR+= tests
+.endif
+
.include <bsd.prog.mk>
diff --git a/bin/pwait/pwait.1 b/bin/pwait/pwait.1
index c5fbb53..b317000 100644
--- a/bin/pwait/pwait.1
+++ b/bin/pwait/pwait.1
@@ -32,7 +32,7 @@
.\"
.\" $FreeBSD$
.\"
-.Dd November 1, 2009
+.Dd March 7, 2017
.Dt PWAIT 1
.Os
.Sh NAME
@@ -40,6 +40,7 @@
.Nd wait for processes to terminate
.Sh SYNOPSIS
.Nm
+.Op Fl t Ar duration
.Op Fl v
.Ar pid
\&...
@@ -50,13 +51,36 @@ utility will wait until each of the given processes has terminated.
.Pp
The following option is available:
.Bl -tag -width indent
+.It Fl t Ar duration
+If any process is still running after
+.Ar duration ,
+.Nm
+will exit.
+The
+.Ar duration
+value can be integer or decimal numbers.
+Values without unit symbols are interpreted as seconds.
+.Pp
+Supported unit symbols are:
+.Bl -tag -width indent -compact
+.It s
+seconds
+.It m
+minutes
+.It h
+hours
+.El
.It Fl v
Print the exit status when each process terminates.
.El
-.Sh DIAGNOSTICS
+.Sh EXIT STATUS
The
.Nm
-utility returns 0 on success, and >0 if an error occurs.
+utility exits 0 on success, and >0 if an error occurs.
+.Pp
+If the
+.Fl t
+flag is specified and a timeout occurs, the exit status will be 124.
.Pp
Invalid pids elicit a warning message but are otherwise ignored.
.Sh SEE ALSO
diff --git a/bin/pwait/pwait.c b/bin/pwait/pwait.c
index 6851fad..76989c9 100644
--- a/bin/pwait/pwait.c
+++ b/bin/pwait/pwait.c
@@ -53,7 +53,7 @@ static void
usage(void)
{
- fprintf(stderr, "usage: pwait [-v] pid ...\n");
+ fprintf(stderr, "usage: pwait [-t timeout] [-v] pid ...\n");
exit(EX_USAGE);
}
@@ -63,15 +63,46 @@ usage(void)
int
main(int argc, char *argv[])
{
+ struct itimerval itv;
int kq;
struct kevent *e;
- int verbose = 0;
+ int tflag, verbose;
int opt, nleft, n, i, duplicate, status;
long pid;
char *s, *end;
+ double timeout;
- while ((opt = getopt(argc, argv, "v")) != -1) {
+ tflag = verbose = 0;
+ memset(&itv, 0, sizeof(itv));
+ while ((opt = getopt(argc, argv, "t:v")) != -1) {
switch (opt) {
+ case 't':
+ tflag = 1;
+ errno = 0;
+ timeout = strtod(optarg, &end);
+ if (end == optarg || errno == ERANGE ||
+ timeout < 0)
+ errx(EX_DATAERR, "timeout value");
+ switch(*end) {
+ case 0:
+ case 's':
+ break;
+ case 'h':
+ timeout *= 60;
+ /* FALLTHROUGH */
+ case 'm':
+ timeout *= 60;
+ break;
+ default:
+ errx(EX_DATAERR, "timeout unit");
+ }
+ if (timeout > 100000000L)
+ errx(EX_DATAERR, "timeout value");
+ itv.it_value.tv_sec = (time_t)timeout;
+ timeout -= (time_t)timeout;
+ itv.it_value.tv_usec =
+ (suseconds_t)(timeout * 1000000UL);
+ break;
case 'v':
verbose = 1;
break;
@@ -91,7 +122,7 @@ main(int argc, char *argv[])
if (kq == -1)
err(1, "kqueue");
- e = malloc(argc * sizeof(struct kevent));
+ e = malloc((argc + tflag) * sizeof(struct kevent));
if (e == NULL)
err(1, "malloc");
nleft = 0;
@@ -119,12 +150,30 @@ main(int argc, char *argv[])
}
}
+ if (tflag) {
+ /*
+ * Explicitly detect SIGALRM so that an exit status of 124
+ * can be returned rather than 142.
+ */
+ EV_SET(e + nleft, SIGALRM, EVFILT_SIGNAL, EV_ADD, 0, 0, NULL);
+ if (kevent(kq, e + nleft, 1, NULL, 0, NULL) == -1)
+ err(EX_OSERR, "kevent");
+ /* Ignore SIGALRM to not interrupt kevent(2). */
+ signal(SIGALRM, SIG_IGN);
+ if (setitimer(ITIMER_REAL, &itv, NULL) == -1)
+ err(EX_OSERR, "setitimer");
+ }
while (nleft > 0) {
- n = kevent(kq, NULL, 0, e, nleft, NULL);
+ n = kevent(kq, NULL, 0, e, nleft + tflag, NULL);
if (n == -1)
err(1, "kevent");
- if (verbose)
- for (i = 0; i < n; i++) {
+ for (i = 0; i < n; i++) {
+ if (e[i].filter == EVFILT_SIGNAL) {
+ if (verbose)
+ printf("timeout\n");
+ return (124);
+ }
+ if (verbose) {
status = e[i].data;
if (WIFEXITED(status))
printf("%ld: exited with status %d.\n",
@@ -138,7 +187,8 @@ main(int argc, char *argv[])
printf("%ld: terminated.\n",
(long)e[i].ident);
}
- nleft -= n;
+ --nleft;
+ }
}
exit(EX_OK);
diff --git a/bin/pwait/tests/Makefile b/bin/pwait/tests/Makefile
new file mode 100644
index 0000000..db05b1f
--- /dev/null
+++ b/bin/pwait/tests/Makefile
@@ -0,0 +1,5 @@
+# $FreeBSD$
+
+ATF_TESTS_SH= pwait_test
+
+.include <bsd.test.mk>
diff --git a/bin/pwait/tests/Makefile.depend b/bin/pwait/tests/Makefile.depend
new file mode 100644
index 0000000..f80275d
--- /dev/null
+++ b/bin/pwait/tests/Makefile.depend
@@ -0,0 +1,11 @@
+# $FreeBSD$
+# Autogenerated - do NOT edit!
+
+DIRDEPS = \
+
+
+.include <dirdeps.mk>
+
+.if ${DEP_RELDIR} == ${_DEP_RELDIR}
+# local dependencies - needed for -jN in clean tree
+.endif
diff --git a/bin/pwait/tests/pwait_test.sh b/bin/pwait/tests/pwait_test.sh
new file mode 100644
index 0000000..0e22c94
--- /dev/null
+++ b/bin/pwait/tests/pwait_test.sh
@@ -0,0 +1,242 @@
+# $FreeBSD$
+
+atf_test_case basic
+basic_head()
+{
+ atf_set "descr" "Basic tests on pwait(1) utility"
+}
+
+basic_body()
+{
+ sleep 1 &
+ p1=$!
+
+ sleep 5 &
+ p5=$!
+
+ sleep 10 &
+ p10=$!
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:0 \
+ timeout --preserve-status 15 pwait $p1 $p5 $p10
+
+ atf_check \
+ -o empty \
+ -e inline:"kill: $p1: No such process\n" \
+ -s exit:1 \
+ kill -0 $p1
+
+ atf_check \
+ -o empty \
+ -e inline:"kill: $p5: No such process\n" \
+ -s exit:1 \
+ kill -0 $p5
+
+ atf_check \
+ -o empty \
+ -e inline:"kill: $p10: No such process\n" \
+ -s exit:1 \
+ kill -0 $p10
+
+}
+
+basic_cleanup()
+{
+ kill $p1 $p5 $p10 >/dev/null 2>&1
+ wait $p1 $p5 $p10 >/dev/null 2>&1
+}
+
+atf_test_case time_unit
+time_unit_head()
+{
+ atf_set "descr" "Test parsing the timeout unit and value"
+}
+
+time_unit_body()
+{
+ init=1
+
+ atf_check \
+ -o empty \
+ -e inline:"pwait: timeout unit\n" \
+ -s exit:65 \
+ timeout --preserve-status 2 pwait -t 1d $init
+
+ atf_check \
+ -o empty \
+ -e inline:"pwait: timeout unit\n" \
+ -s exit:65 \
+ timeout --preserve-status 2 pwait -t 1d $init
+
+ atf_check \
+ -o empty \
+ -e inline:"pwait: timeout value\n" \
+ -s exit:65 \
+ timeout --preserve-status 2 pwait -t -1 $init
+
+ atf_check \
+ -o empty \
+ -e inline:"pwait: timeout value\n" \
+ -s exit:65 \
+ timeout --preserve-status 2 pwait -t 100000001 $init
+
+ # These long duration cases are expected to timeout from the
+ # timeout utility rather than pwait -t.
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:143 \
+ timeout --preserve-status 2 pwait -t 100000000 $init
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:143 \
+ timeout --preserve-status 2 pwait -t 1h $init
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:143 \
+ timeout --preserve-status 2 pwait -t 1.5h $init
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:143 \
+ timeout --preserve-status 2 pwait -t 1m $init
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:143 \
+ timeout --preserve-status 2 pwait -t 1.5m $init
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:143 \
+ timeout --preserve-status 2 pwait -t 0 $init
+
+ # The rest are fast enough that pwait -t is expected to trigger
+ # the timeout.
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:124 \
+ timeout --preserve-status 2 pwait -t 1s $init
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:124 \
+ timeout --preserve-status 2 pwait -t 1.5s $init
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:124 \
+ timeout --preserve-status 2 pwait -t 1 $init
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:124 \
+ timeout --preserve-status 2 pwait -t 1.5 $init
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:124 \
+ timeout --preserve-status 2 pwait -t 0.5 $init
+}
+
+atf_test_case timeout_trigger_timeout
+timeout_trigger_timeout_head()
+{
+ atf_set "descr" "Test that exceeding the timeout is detected"
+}
+
+timeout_trigger_timeout_body()
+{
+ sleep 10 &
+ p10=$!
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:124 \
+ timeout --preserve-status 6.5 pwait -t 5 $p10
+}
+
+timeout_trigger_timeout_cleanup()
+{
+ kill $p10 >/dev/null 2>&1
+ wait $p10 >/dev/null 2>&1
+}
+
+atf_test_case timeout_no_timeout
+timeout_no_timeout_head()
+{
+ atf_set "descr" "Test that not exceeding the timeout continues to wait"
+}
+
+timeout_no_timeout_body()
+{
+ sleep 10 &
+ p10=$!
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:0 \
+ timeout --preserve-status 11.5 pwait -t 12 $p10
+}
+
+timeout_no_timeout_cleanup()
+{
+ kill $p10 >/dev/null 2>&1
+ wait $p10 >/dev/null 2>&1
+}
+
+atf_test_case timeout_many
+timeout_many_head()
+{
+ atf_set "descr" "Test timeout on many processes"
+}
+
+timeout_many_body()
+{
+ sleep 1 &
+ p1=$!
+
+ sleep 5 &
+ p5=$!
+
+ sleep 10 &
+ p10=$!
+
+ atf_check \
+ -o empty \
+ -e empty \
+ -s exit:124 \
+ timeout --preserve-status 7.5 pwait -t 6 $p1 $p5 $p10
+}
+
+timeout_many_cleanup()
+{
+ kill $p1 $p5 $p10 >/dev/null 2>&1
+ wait $p1 $p5 $p10 >/dev/null 2>&1
+}
+
+atf_init_test_cases()
+{
+ atf_add_test_case basic
+ atf_add_test_case time_unit
+ atf_add_test_case timeout_trigger_timeout
+ atf_add_test_case timeout_no_timeout
+ atf_add_test_case timeout_many
+}
diff --git a/etc/mtree/BSD.tests.dist b/etc/mtree/BSD.tests.dist
index 9751c97..f7c2bc9 100644
--- a/etc/mtree/BSD.tests.dist
+++ b/etc/mtree/BSD.tests.dist
@@ -626,6 +626,8 @@
..
printf
..
+ pwait
+ ..
sdiff
..
sed
diff --git a/targets/pseudo/tests/Makefile.depend b/targets/pseudo/tests/Makefile.depend
index c8d1f83..f2b5f87 100644
--- a/targets/pseudo/tests/Makefile.depend
+++ b/targets/pseudo/tests/Makefile.depend
@@ -307,6 +307,7 @@ DIRDEPS= \
usr.bin/mkimg/tests \
usr.bin/ncal/tests \
usr.bin/printf/tests \
+ usr.bin/pwait/tests \
usr.bin/sdiff/tests \
usr.bin/sed/tests \
usr.bin/sed/tests/regress.multitest.out \
diff --git a/usr.bin/timeout/tests/Makefile b/usr.bin/timeout/tests/Makefile
index bced155..691a27d 100644
--- a/usr.bin/timeout/tests/Makefile
+++ b/usr.bin/timeout/tests/Makefile
@@ -1,5 +1,5 @@
# $FreeBSD$
-ATF_TESTS_SH= timeout
+ATF_TESTS_SH= timeout_test
.include <bsd.test.mk>
diff --git a/usr.bin/timeout/tests/timeout.sh b/usr.bin/timeout/tests/timeout_test.sh
index 6ccde32..ffddfca 100644
--- a/usr.bin/timeout/tests/timeout.sh
+++ b/usr.bin/timeout/tests/timeout_test.sh
@@ -12,7 +12,7 @@ nominal_body()
-o empty \
-e empty \
-s exit:0 \
- -x timeout 5 true
+ timeout 5 true
}
atf_test_case time_unit
@@ -27,25 +27,25 @@ time_unit_body()
-o empty \
-e empty \
-s exit:0 \
- -x timeout 1d true
+ timeout 1d true
atf_check \
-o empty \
-e empty \
-s exit:0 \
- -x timeout 1h true
+ timeout 1h true
atf_check \
-o empty \
-e empty \
-s exit:0 \
- -x timeout 1m true
+ timeout 1m true
atf_check \
-o empty \
-e empty \
-s exit:0 \
- -x timeout 1s true
+ timeout 1s true
}
atf_test_case no_timeout
@@ -60,7 +60,7 @@ no_timeout_body()
-o empty \
-e empty \
-s exit:0 \
- -x timeout 0 true
+ timeout 0 true
}
atf_test_case exit_numbers
@@ -81,20 +81,20 @@ exit_numbers_body()
-o empty \
-e empty \
-s exit:124 \
- -x timeout .1 sleep 1
+ timeout .1 sleep 1
# With preserv status exit should be 128 + TERM aka 143
atf_check \
-o empty \
-e empty \
-s exit:143 \
- -x timeout --preserve-status .1 sleep 10
+ timeout --preserve-status .1 sleep 10
atf_check \
-o empty \
-e empty \
-s exit:124 \
- -x timeout -s1 -k1 .1 sleep 10
+ timeout -s1 -k1 .1 sleep 10
atf_check \
-o empty \
@@ -129,31 +129,31 @@ invalid_timeout_body()
-o empty \
-e inline:"timeout: invalid duration\n" \
-s exit:125 \
- -x timeout invalid sleep 0
+ timeout invalid sleep 0
atf_check \
-o empty \
-e inline:"timeout: invalid duration\n" \
-s exit:125 \
- -x timeout --kill-after=invalid 1 sleep 0
+ timeout --kill-after=invalid 1 sleep 0
atf_check \
-o empty \
-e inline:"timeout: invalid duration\n" \
-s exit:125 \
- -x timeout 42D sleep 0
+ timeout 42D sleep 0
atf_check \
-o empty \
-e inline:"timeout: invalid duration\n" \
-s exit:125 \
- -x timeout 999999999999999999999999999999999999999999999999999999999999d sleep 0
+ timeout 999999999999999999999999999999999999999999999999999999999999d sleep 0
atf_check \
-o empty \
-e inline:"timeout: invalid duration\n" \
-s exit:125 \
- -x timeout 2.34e+5d sleep 0
+ timeout 2.34e+5d sleep 0
}
atf_test_case invalid_signal
@@ -168,7 +168,7 @@ invalid_signal_body()
-o empty \
-e inline:"timeout: invalid signal\n" \
-s exit:125 \
- -x timeout --signal=invalid 1 sleep 0
+ timeout --signal=invalid 1 sleep 0
}
atf_test_case invalid_command
@@ -183,7 +183,7 @@ invalid_command_body()
-o empty \
-e inline:"timeout: exec(.): Permission denied\n" \
-s exit:126 \
- -x timeout 10 .
+ timeout 10 .
}
atf_test_case no_such_command
@@ -198,7 +198,7 @@ no_such_command_body()
-o empty \
-e inline:"timeout: exec(enoexists): No such file or directory\n" \
-s exit:127 \
- -x timeout 10 enoexists
+ timeout 10 enoexists
}
atf_init_test_cases()
OpenPOWER on IntegriCloud