d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use kqueue(2) instead of select(2).

Browse Source 
This helps to ensure we will not lose SIGINT sent by parent to child.

Reviewed by:	sbruno, ngie
Sponsored by:	DARPA, AFRL
Sponsored by:	HEIF5
Differential Revision:	https://reviews.freebsd.org/D7892
This commit is contained in:
br 2016-09-21 11:59:52 +00:00
parent df422cbea3
commit a26096b692
1 changed files with 18 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
lib/libutil/tests/pidfile_test.c
View File

@ -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_open)
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_open)
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