Make sure we remove pid file in monitor before manager exits.

[thirdparty/mdadm.git] / monitor.c

diff --git a/monitor.c b/monitor.c
index 5bb2559602d1f4cf2b241a2fe78df489dceb6623..46d5e0a00380672285a2667c140e8324cc545721 100644 (file)
--- a/monitor.c
+++ b/monitor.c
@@ -1,7 +1,7 @@
 
 #include "mdadm.h"
 #include "mdmon.h"
-
+#include <sys/syscall.h>
 #include <sys/select.h>
 #include <signal.h>
 
@@ -138,7 +138,9 @@ int read_dev_state(int fd)
 
 static void signal_manager(void)
 {
-       kill(getpid(), SIGUSR1);
+       /* tgkill(getpid(), mon_tid, SIGUSR1); */
+       int pid = getpid();
+       syscall(SYS_tgkill, pid, mgr_tid, SIGUSR1);
 }
 
 /* Monitor a set of active md arrays - all of which share the
@@ -240,7 +242,7 @@ static int read_and_act(struct active_array *a)
                 * readonly ???
                 */
                get_resync_start(a);
-               printf("Found a readonly array at %llu\n", a->resync_start);
+//             printf("Found a readonly array at %llu\n", a->resync_start);
                if (a->resync_start == ~0ULL)
                        a->next_state = read_auto; /* array is clean */
                else {
@@ -367,16 +369,6 @@ static void reconcile_failed(struct active_array *aa, struct mdinfo *failed)
        }
 }
 
-static int handle_pipe(struct md_generic_cmd *cmd, struct active_array *aa)
-{
-       switch (cmd->action) {
-       case md_action_ping_monitor:
-               return 0;
-       }
-
-       return -1;
-}
-
 #ifdef DEBUG
 static void dprint_wake_reasons(fd_set *fds)
 {
@@ -407,8 +399,7 @@ static void dprint_wake_reasons(fd_set *fds)
 }
 #endif
 
-static int wait_and_act(struct supertype *container, int pfd,
-                       int monfd, int nowait)
+static int wait_and_act(struct supertype *container, int nowait)
 {
        fd_set rfds;
        int maxfd = 0;
@@ -419,7 +410,6 @@ static int wait_and_act(struct supertype *container, int pfd,
 
        FD_ZERO(&rfds);
 
-       add_fd(&rfds, &maxfd, pfd);
        for (ap = aap ; *ap ;) {
                a = *ap;
                /* once an array has been deactivated we want to
@@ -455,30 +445,24 @@ static int wait_and_act(struct supertype *container, int pfd,
                if (fd >= 0 || errno != EBUSY) {
                        /* OK, we are safe to leave */
                        dprintf("no arrays to monitor... exiting\n");
+                       remove_pidfile(container->devname);
                        exit_now = 1;
                        signal_manager();
-                       remove_pidfile(container->devname);
                        exit(0);
                }
        }
 
        if (!nowait) {
-               rv = select(maxfd+1, &rfds, NULL, NULL, NULL);
-
+               sigset_t set;
+               sigprocmask(SIG_UNBLOCK, NULL, &set);
+               sigdelset(&set, SIGUSR1);
+               rv = pselect(maxfd+1, &rfds, NULL, NULL, NULL, &set);
+               if (rv == -1 && errno == EINTR)
+                       rv = 0;
                #ifdef DEBUG
                dprint_wake_reasons(&rfds);
                #endif
 
-               if (rv <= 0)
-                       return rv;
-
-               if (FD_ISSET(pfd, &rfds)) {
-                       int err = -1;
-
-                       if (read(pfd, &err, 1) > 0)
-                               err = handle_pipe(active_cmd, *aap);
-                       write(monfd, &err, 1);
-               }
        }
 
        if (update_queue) {
@@ -527,8 +511,7 @@ void do_monitor(struct supertype *container)
        int rv;
        int first = 1;
        do {
-               rv = wait_and_act(container, container->mgr_pipe[0],
-                                 container->mon_pipe[1], first);
+               rv = wait_and_act(container, first);
                first = 0;
        } while (rv >= 0);
 }