svn commit: r335964 - stable/11/lib/libutil/tests
Ed Maste
emaste at FreeBSD.org
Wed Jul 4 18:01:54 UTC 2018
Author: emaste
Date: Wed Jul 4 18:01:53 2018
New Revision: 335964
URL: https://svnweb.freebsd.org/changeset/base/335964
Log:
MFC r306098 (br): Use kqueue(2) instead of select(2).
This helps to ensure we will not lose SIGINT sent by parent to child.
PR: 212562, 228492
Modified:
stable/11/lib/libutil/tests/pidfile_test.c
Directory Properties:
stable/11/ (props changed)
Modified: stable/11/lib/libutil/tests/pidfile_test.c
==============================================================================
--- stable/11/lib/libutil/tests/pidfile_test.c Wed Jul 4 17:18:35 2018 (r335963)
+++ stable/11/lib/libutil/tests/pidfile_test.c Wed Jul 4 18:01:53 2018 (r335964)
@@ -30,6 +30,7 @@ __FBSDID("$FreeBSD$");
#include <sys/param.h>
#include <sys/wait.h>
+#include <sys/event.h>
#include <fcntl.h>
#include <errno.h>
@@ -43,8 +44,8 @@ __FBSDID("$FreeBSD$");
#include <libutil.h>
/*
- * We need a signal handler so kill(2) will interrupt our child's
- * select(2) instead of killing it.
+ * We need a signal handler so kill(2) will interrupt the child
+ * instead of killing it.
*/
static void
signal_handler(int sig)
@@ -129,7 +130,9 @@ common_test_pidfile_child(const char *fn, int parent_o
struct pidfh *pf = NULL;
pid_t other = 0, pid = 0;
int fd[2], serrno, status;
+ struct kevent event, ke;
char ch;
+ int kq;
unlink(fn);
if (pipe(fd) != 0)
@@ -166,10 +169,20 @@ common_test_pidfile_child(const char *fn, int parent_o
if (pf == NULL)
_exit(1);
if (pidfile_write(pf) != 0)
- _exit(1);
+ _exit(2);
+ kq = kqueue();
+ if (kq == -1)
+ _exit(3);
+ EV_SET(&ke, SIGINT, EVFILT_SIGNAL, EV_ADD, 0, 0, NULL);
+ /* Attach event to the kqueue. */
+ if (kevent(kq, &ke, 1, NULL, 0, NULL) != 0)
+ _exit(4);
+ /* Inform the parent we are ready to receive SIGINT */
if (write(fd[1], "*", 1) != 1)
- _exit(1);
- select(0, 0, 0, 0, 0);
+ _exit(5);
+ /* Wait for SIGINT received */
+ if (kevent(kq, NULL, 0, &event, 1, NULL) != 1)
+ _exit(6);
_exit(0);
}
// parent
More information about the svn-src-stable-11
mailing list