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
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to