From 449ddb2d23a63ca4c8cd70d13a070fba87c1fb30 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Fri, 20 Aug 2010 03:26:15 +0200 Subject: remount: add tool that applies /etc/fstab mount options to all api mounts --- src/hashmap.c | 9 ++++ src/hashmap.h | 1 + src/mount-setup.c | 4 +- src/remount-api-vfs.c | 147 ++++++++++++++++++++++++++++++++++++++++++++++++++ src/set.c | 7 +-- 5 files changed, 160 insertions(+), 8 deletions(-) create mode 100644 src/remount-api-vfs.c (limited to 'src') diff --git a/src/hashmap.c b/src/hashmap.c index 6b6e909ba..51f00131f 100644 --- a/src/hashmap.c +++ b/src/hashmap.c @@ -173,6 +173,15 @@ void hashmap_free(Hashmap*h) { free(h); } +void hashmap_free_free(Hashmap *h) { + void *p; + + while ((p = hashmap_steal_first(h))) + free(p); + + hashmap_free(h); +} + void hashmap_clear(Hashmap *h) { if (!h) return; diff --git a/src/hashmap.h b/src/hashmap.h index 9ab66c51b..c48d6b31f 100644 --- a/src/hashmap.h +++ b/src/hashmap.h @@ -47,6 +47,7 @@ int trivial_compare_func(const void *a, const void *b); Hashmap *hashmap_new(hash_func_t hash_func, compare_func_t compare_func); void hashmap_free(Hashmap *h); +void hashmap_free_free(Hashmap *h); Hashmap *hashmap_copy(Hashmap *h); int hashmap_ensure_allocated(Hashmap **h, hash_func_t hash_func, compare_func_t compare_func); diff --git a/src/mount-setup.c b/src/mount-setup.c index ecd9e181c..8b4d8c7f4 100644 --- a/src/mount-setup.c +++ b/src/mount-setup.c @@ -68,11 +68,11 @@ bool mount_point_is_api(const char *path) { * should be ignored */ for (i = 0; i < ELEMENTSOF(mount_table); i ++) - if (path_startswith(path, mount_table[i].where)) + if (path_equal(path, mount_table[i].where)) return true; for (i = 0; i < ELEMENTSOF(ignore_paths); i++) - if (path_startswith(path, ignore_paths[i])) + if (path_equal(path, ignore_paths[i])) return true; return path_startswith(path, "/cgroup/"); diff --git a/src/remount-api-vfs.c b/src/remount-api-vfs.c new file mode 100644 index 000000000..d51a584f2 --- /dev/null +++ b/src/remount-api-vfs.c @@ -0,0 +1,147 @@ +/*-*- Mode: C; c-basic-offset: 8; indent-tabs-mode: nil -*-*/ + +/*** + This file is part of systemd. + + Copyright 2010 Lennart Poettering + + systemd is free software; you can redistribute it and/or modify it + under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 2 of the License, or + (at your option) any later version. + + systemd is distributed in the hope that 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 systemd; If not, see . +***/ + +#include +#include +#include +#include +#include +#include +#include + +#include "log.h" +#include "util.h" +#include "set.h" +#include "mount-setup.h" + +/* Goes through /etc/fstab and remounts all API file systems, applying + * options that are in /etc/fstab that systemd might not have + * respected */ + +int main(int argc, char *argv[]) { + int ret = 1; + FILE *f = NULL; + struct mntent* me; + Hashmap *pids = NULL; + + if (argc > 1) { + log_error("This program takes no argument."); + return 1; + } + + log_set_target(LOG_TARGET_SYSLOG_OR_KMSG); + log_parse_environment(); + log_open(); + + if (!(f = setmntent("/etc/fstab", "r"))) { + log_error("Failed to open /etc/fstab: %m"); + goto finish; + } + + if (!(pids = hashmap_new(trivial_hash_func, trivial_compare_func))) { + log_error("Failed to allocate set"); + goto finish; + } + + ret = 0; + + while ((me = getmntent(f))) { + pid_t pid; + int k; + char *s; + + if (!mount_point_is_api(me->mnt_dir)) + continue; + + log_debug("Remounting %s", me->mnt_dir); + + if ((pid = fork()) < 0) { + log_error("Failed to fork: %m"); + ret = 1; + continue; + } + + if (pid == 0) { + const char *arguments[5]; + /* Child */ + + arguments[0] = "/bin/mount"; + arguments[1] = me->mnt_dir; + arguments[2] = "-o"; + arguments[3] = "remount"; + arguments[4] = NULL; + + execv("/bin/mount", (char **) arguments); + + log_error("Failed to execute /bin/mount: %m"); + _exit(1); + } + + /* Parent */ + + s = strdup(me->mnt_dir); + + if ((k = hashmap_put(pids, UINT_TO_PTR(pid), s)) < 0) { + log_error("Failed to add PID to set: %s", strerror(-k)); + ret = 1; + continue; + } + } + + while (!hashmap_isempty(pids)) { + siginfo_t si; + char *s; + + zero(si); + if (waitid(P_ALL, 0, &si, WEXITED) < 0) { + + if (errno == EINTR) + continue; + + log_error("waitid() failed: %m"); + ret = 1; + break; + } + + if ((s = hashmap_remove(pids, UINT_TO_PTR(si.si_pid)))) { + if (!is_clean_exit(si.si_code, si.si_status)) { + if (si.si_code == CLD_EXITED) + log_error("/bin/mount for %s exited with exit status %i.", s, si.si_status); + else + log_error("/bin/mount for %s terminated by signal %s.", s, signal_to_string(si.si_status)); + + ret = 1; + } + + free(s); + } + } + +finish: + + if (pids) + hashmap_free_free(pids); + + if (f) + endmntent(f); + + return ret; +} diff --git a/src/set.c b/src/set.c index 331d37e3d..097b9d3aa 100644 --- a/src/set.c +++ b/src/set.c @@ -38,12 +38,7 @@ void set_free(Set* s) { } void set_free_free(Set *s) { - void *p; - - while ((p = set_steal_first(s))) - free(p); - - set_free(s); + hashmap_free_free(MAKE_HASHMAP(s)); } int set_ensure_allocated(Set **s, hash_func_t hash_func, compare_func_t compare_func) { -- cgit v1.2.3