X-Git-Url: http://git.ipfire.org/?a=blobdiff_plain;f=monitor.c;h=0cafc3ac3418c66e191a8eb72eeda4875d4eac5b;hb=22e305169f02253890da169b90678da48ffa5120;hp=5a95149027da2f28244a5d8ff5b8728f7e39f7ef;hpb=7e1432fb14bb7d362821b8c62dd282e67c536efe;p=thirdparty%2Fmdadm.git diff --git a/monitor.c b/monitor.c index 5a951490..0cafc3ac 100644 --- a/monitor.c +++ b/monitor.c @@ -1,7 +1,26 @@ +/* + * mdmon - monitor external metadata arrays + * + * Copyright (C) 2007-2009 Neil Brown + * Copyright (C) 2007-2009 Intel Corporation + * + * This program is free software; you can redistribute it and/or modify it + * under the terms and conditions of the GNU General Public License, + * version 2, as published by the Free Software Foundation. + * + * This program is distributed in the hope it will be useful, but WITHOUT + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or + * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for + * more details. + * + * You should have received a copy of the GNU General Public License along with + * this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA. + */ #include "mdadm.h" #include "mdmon.h" - +#include #include #include @@ -47,8 +66,7 @@ static int read_attr(char *buf, int len, int fd) return n; } - -static int get_resync_start(struct active_array *a) +int get_resync_start(struct active_array *a) { char buf[30]; int n; @@ -56,36 +74,14 @@ static int get_resync_start(struct active_array *a) n = read_attr(buf, 30, a->resync_start_fd); if (n <= 0) return n; + if (strncmp(buf, "none", 4) == 0) + a->resync_start = ~0ULL; + else + a->resync_start = strtoull(buf, NULL, 10); - a->resync_start = strtoull(buf, NULL, 10); - - return 1; -} - -static int attr_match(const char *attr, const char *str) -{ - /* See if attr, read from a sysfs file, matches - * str. They must either be the same, or attr can - * have a trailing newline or comma - */ - while (*attr && *str && *attr == *str) { - attr++; - str++; - } - - if (*str || (*attr && *attr != ',' && *attr != '\n')) - return 0; return 1; } -static int match_word(const char *word, char **list) -{ - int n; - for (n=0; list[n]; n++) - if (attr_match(word, list[n])) - break; - return n; -} static enum array_state read_state(int fd) { @@ -94,7 +90,7 @@ static enum array_state read_state(int fd) if (n <= 0) return bad_word; - return (enum array_state) match_word(buf, array_states); + return (enum array_state) sysfs_match_word(buf, array_states); } static enum sync_action read_action( int fd) @@ -104,7 +100,7 @@ static enum sync_action read_action( int fd) if (n <= 0) return bad_action; - return (enum sync_action) match_word(buf, sync_actions); + return (enum sync_action) sysfs_match_word(buf, sync_actions); } int read_dev_state(int fd) @@ -119,15 +115,15 @@ int read_dev_state(int fd) cp = buf; while (cp) { - if (attr_match(cp, "faulty")) + if (sysfs_attr_match(cp, "faulty")) rv |= DS_FAULTY; - if (attr_match(cp, "in_sync")) + if (sysfs_attr_match(cp, "in_sync")) rv |= DS_INSYNC; - if (attr_match(cp, "write_mostly")) + if (sysfs_attr_match(cp, "write_mostly")) rv |= DS_WRITE_MOSTLY; - if (attr_match(cp, "spare")) + if (sysfs_attr_match(cp, "spare")) rv |= DS_SPARE; - if (attr_match(cp, "blocked")) + if (sysfs_attr_match(cp, "blocked")) rv |= DS_BLOCKED; cp = strchr(cp, ','); if (cp) @@ -138,7 +134,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 @@ -203,6 +201,7 @@ static int read_and_act(struct active_array *a) int check_degraded = 0; int deactivate = 0; struct mdinfo *mdi; + int dirty = 0; a->next_state = bad_word; a->next_action = bad_action; @@ -218,6 +217,7 @@ static int read_and_act(struct active_array *a) if (a->curr_state <= inactive && a->prev_state > inactive) { /* array has been stopped */ + get_resync_start(a); a->container->ss->set_array_state(a, 1); a->next_state = clear; deactivate = 1; @@ -226,30 +226,45 @@ static int read_and_act(struct active_array *a) get_resync_start(a); a->container->ss->set_array_state(a, 0); a->next_state = active; + dirty = 1; } if (a->curr_state == active_idle) { - /* Set array to 'clean' FIRST, then - * a->ss->mark_clean(a, ~0ULL); - * just ignore for now. + /* Set array to 'clean' FIRST, then mark clean + * in the metadata */ + a->next_state = clean; + dirty = 1; } - + if (a->curr_state == clean) { + get_resync_start(a); + a->container->ss->set_array_state(a, 1); + } + if (a->curr_state == active || + a->curr_state == suspended || + a->curr_state == bad_word) + dirty = 1; if (a->curr_state == readonly) { - /* Well, I'm ready to handle things, so - * read-auto is OK. FIXME what if we really want - * readonly ??? + /* Well, I'm ready to handle things. If readonly + * wasn't requested, transition to read-auto. */ - get_resync_start(a); - 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 { - a->container->ss->set_array_state(a, 0); - a->next_state = active; + char buf[64]; + read_attr(buf, sizeof(buf), a->metadata_fd); + if (strncmp(buf, "external:-", 10) == 0) { + /* explicit request for readonly array. Leave it alone */ + ; + } else { + get_resync_start(a); + if (a->container->ss->set_array_state(a, 2)) + a->next_state = read_auto; /* array is clean */ + else { + a->next_state = active; /* Now active for recovery etc */ + dirty = 1; + } } } - if (a->curr_action == idle && + if (!deactivate && + a->curr_action == idle && a->prev_action == resync) { /* A resync has finished. The endpoint is recorded in * 'sync_start'. We don't update the metadata @@ -257,12 +272,16 @@ static int read_and_act(struct active_array *a) * Just check if we need to fiddle spares. */ get_resync_start(a); - a->container->ss->set_array_state(a, 0); + a->container->ss->set_array_state(a, a->curr_state <= clean); check_degraded = 1; } - if (a->curr_action == idle && + if (!deactivate && + a->curr_action == idle && a->prev_action == recover) { + /* A recovery has finished. Some disks may be in sync now, + * and the array may no longer be degraded + */ for (mdi = a->info.devs ; mdi ; mdi = mdi->next) { a->container->ss->set_disk(a, mdi->disk.raid_disk, mdi->curr_state); @@ -271,40 +290,67 @@ static int read_and_act(struct active_array *a) } } + /* Check for failures and if found: + * 1/ Record the failure in the metadata and unblock the device. + * FIXME update the kernel to stop notifying on failed drives when + * the array is readonly and we have cleared 'blocked' + * 2/ Try to remove the device if the array is writable, or can be + * made writable. + */ for (mdi = a->info.devs ; mdi ; mdi = mdi->next) { if (mdi->curr_state & DS_FAULTY) { a->container->ss->set_disk(a, mdi->disk.raid_disk, mdi->curr_state); check_degraded = 1; - mdi->next_state = DS_REMOVE; + mdi->next_state |= DS_UNBLOCK; + if (a->curr_state == read_auto) { + a->container->ss->set_array_state(a, 0); + a->next_state = active; + } + if (a->curr_state > readonly) + mdi->next_state |= DS_REMOVE; } } a->container->ss->sync_metadata(a->container); + dprintf("%s(%d): state:%s action:%s next(", __func__, a->info.container_member, + array_states[a->curr_state], sync_actions[a->curr_action]); /* Effect state changes in the array */ - if (a->next_state != bad_word) + if (a->next_state != bad_word) { + dprintf(" state:%s", array_states[a->next_state]); write_attr(array_states[a->next_state], a->info.state_fd); - if (a->next_action != bad_action) + } + if (a->next_action != bad_action) { write_attr(sync_actions[a->next_action], a->action_fd); + dprintf(" action:%s", sync_actions[a->next_action]); + } for (mdi = a->info.devs; mdi ; mdi = mdi->next) { - if (mdi->next_state == DS_REMOVE && mdi->state_fd >= 0) { + if (mdi->next_state & DS_UNBLOCK) { + dprintf(" %d:-blocked", mdi->disk.raid_disk); + write_attr("-blocked", mdi->state_fd); + } + + if ((mdi->next_state & DS_REMOVE) && mdi->state_fd >= 0) { int remove_result; - write_attr("-blocked", mdi->state_fd); /* the kernel may not be able to immediately remove the * disk, we can simply wait until the next event to try * again. */ remove_result = write_attr("remove", mdi->state_fd); if (remove_result > 0) { + dprintf(" %d:removed", mdi->disk.raid_disk); close(mdi->state_fd); mdi->state_fd = -1; } } - if (mdi->next_state & DS_INSYNC) + if (mdi->next_state & DS_INSYNC) { write_attr("+in_sync", mdi->state_fd); + dprintf(" %d:+in_sync", mdi->disk.raid_disk); + } } + dprintf(" )\n"); /* move curr_ to prev_ */ a->prev_state = a->curr_state; @@ -325,7 +371,7 @@ static int read_and_act(struct active_array *a) if (deactivate) a->container = NULL; - return 1; + return dirty; } static struct mdinfo * @@ -357,61 +403,39 @@ static void reconcile_failed(struct active_array *aa, struct mdinfo *failed) } } -static int handle_remove_device(struct md_remove_device_cmd *cmd, struct active_array *aa) +#ifdef DEBUG +static void dprint_wake_reasons(fd_set *fds) { - struct active_array *a; - struct mdinfo *victim; + int i; + char proc_path[256]; + char link[256]; + char *basename; int rv; - /* scan all arrays for the given device, if ->state_fd is closed (-1) - * in all cases then mark the disk as removed in the metadata. - * Otherwise reply that it is busy. - */ - - /* pass1 check that it is not in use anywhere */ - /* note: we are safe from re-adds as long as the device exists in the - * container - */ - for (a = aa; a; a = a->next) { - if (!a->container) - continue; - victim = find_device(a, major(cmd->rdev), minor(cmd->rdev)); - if (!victim) - continue; - if (victim->state_fd > 0) - return -EBUSY; - } + fprintf(stderr, "monitor: wake ( "); + for (i = 0; i < FD_SETSIZE; i++) { + if (FD_ISSET(i, fds)) { + sprintf(proc_path, "/proc/%d/fd/%d", + (int) getpid(), i); - /* pass2 schedule and process removal per array */ - for (a = aa; a; a = a->next) { - if (!a->container) - continue; - victim = find_device(a, major(cmd->rdev), minor(cmd->rdev)); - if (!victim) - continue; - victim->curr_state |= DS_REMOVE; - rv = read_and_act(a); - if (rv < 0) - return rv; + rv = readlink(proc_path, link, sizeof(link) - 1); + if (rv < 0) { + fprintf(stderr, "%d:unknown ", i); + continue; + } + link[rv] = '\0'; + basename = strrchr(link, '/'); + fprintf(stderr, "%d:%s ", + i, basename ? ++basename : link); + } } - - return 0; + fprintf(stderr, ")\n"); } +#endif -static int handle_pipe(struct md_generic_cmd *cmd, struct active_array *aa) -{ - switch (cmd->action) { - case md_action_ping_monitor: - return 0; - case md_action_remove_device: - return handle_remove_device((void *) cmd, aa); - } - - return -1; -} +int monitor_loop_cnt; -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,10 +443,10 @@ static int wait_and_act(struct supertype *container, int pfd, struct active_array *a, **ap; int rv; struct mdinfo *mdi; + static unsigned int dirty_arrays = ~0; /* start at some non-zero value */ FD_ZERO(&rfds); - add_fd(&rfds, &maxfd, pfd); for (ap = aap ; *ap ;) { a = *ap; /* once an array has been deactivated we want to @@ -448,35 +472,40 @@ static int wait_and_act(struct supertype *container, int pfd, ap = &(*ap)->next; } - if (manager_ready && *aap == NULL) { - /* No interesting arrays. Lets see about exiting. - * Note that blocking at this point is not a problem - * as there are no active arrays, there is nothing that - * we need to be ready to do. + if (manager_ready && (*aap == NULL || (sigterm && !dirty_arrays))) { + /* No interesting arrays, or we have been told to + * terminate and everything is clean. Lets see about + * exiting. Note that blocking at this point is not a + * problem as there are no active arrays, there is + * nothing that we need to be ready to do. */ - int fd = open(container->device_name, O_RDONLY|O_EXCL); + int fd = open_dev_excl(container->devnum); if (fd >= 0 || errno != EBUSY) { /* OK, we are safe to leave */ + if (sigterm && !dirty_arrays) + dprintf("caught sigterm, all clean... exiting\n"); + else + 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); - - if (rv <= 0) - return rv; - - if (FD_ISSET(pfd, &rfds)) { - int err = -1; + sigset_t set; + sigprocmask(SIG_UNBLOCK, NULL, &set); + sigdelset(&set, SIGUSR1); + monitor_loop_cnt |= 1; + rv = pselect(maxfd+1, NULL, NULL, &rfds, NULL, &set); + monitor_loop_cnt += 1; + if (rv == -1 && errno == EINTR) + rv = 0; + #ifdef DEBUG + dprint_wake_reasons(&rfds); + #endif - if (read(pfd, &err, 1) > 0) - err = handle_pipe(active_cmd, *aap); - write(monfd, &err, 1); - } } if (update_queue) { @@ -491,7 +520,11 @@ static int wait_and_act(struct supertype *container, int pfd, container->ss->sync_metadata(container); } + rv = 0; + dirty_arrays = 0; for (a = *aap; a ; a = a->next) { + int is_dirty; + if (a->replaces && !discard_this) { struct active_array **ap; for (ap = &a->next; *ap && *ap != a->replaces; @@ -504,8 +537,17 @@ static int wait_and_act(struct supertype *container, int pfd, /* FIXME check if device->state_fd need to be cleared?*/ signal_manager(); } - if (a->container) - rv += read_and_act(a); + if (a->container) { + is_dirty = read_and_act(a); + rv |= 1; + dirty_arrays += is_dirty; + /* when terminating stop manipulating the array after it + * is clean, but make sure read_and_act() is given a + * chance to handle 'active_idle' + */ + if (sigterm && !is_dirty) + a->container = NULL; /* stop touching this array */ + } } /* propagate failures across container members */ @@ -525,8 +567,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); }