fstab-generator: place initrd /sysroot mounts in initrd-fs.target

also do not overwrite /sysroot*.mount units already generated from fstab
This commit is contained in:
Harald Hoyer 2013-03-04 19:04:32 +01:00
parent 7d89ce303f
commit 8330847e94

View file

@ -200,8 +200,8 @@ static bool mount_in_initrd(struct mntent *me) {
} }
static int add_mount(const char *what, const char *where, const char *type, const char *opts, static int add_mount(const char *what, const char *where, const char *type, const char *opts,
int passno, bool noauto, bool nofail, bool automount, bool isbind, bool isnetwork, int passno, bool noauto, bool nofail, bool automount, bool isbind,
const char *source) { bool remote_fs_target, bool initrd_fs_target, const char *source) {
char _cleanup_free_ char _cleanup_free_
*name = NULL, *unit = NULL, *lnk = NULL, *device = NULL, *name = NULL, *unit = NULL, *lnk = NULL, *device = NULL,
*automount_name = NULL, *automount_unit = NULL; *automount_name = NULL, *automount_unit = NULL;
@ -227,9 +227,12 @@ static int add_mount(const char *what, const char *where, const char *type, cons
mount_point_ignore(where)) mount_point_ignore(where))
return 0; return 0;
if (isnetwork) { if (remote_fs_target) {
post = SPECIAL_REMOTE_FS_TARGET; post = SPECIAL_REMOTE_FS_TARGET;
pre = SPECIAL_REMOTE_FS_PRE_TARGET; pre = SPECIAL_REMOTE_FS_PRE_TARGET;
} else if (initrd_fs_target) {
post = SPECIAL_INITRD_FS_TARGET;
pre = SPECIAL_INITRD_FS_PRE_TARGET;
} else { } else {
post = SPECIAL_LOCAL_FS_TARGET; post = SPECIAL_LOCAL_FS_TARGET;
pre = SPECIAL_LOCAL_FS_PRE_TARGET; pre = SPECIAL_LOCAL_FS_PRE_TARGET;
@ -407,6 +410,7 @@ static int parse_fstab(const char *prefix, bool initrd) {
continue; continue;
what = fstab_node_to_udev_node(me->mnt_fsname); what = fstab_node_to_udev_node(me->mnt_fsname);
where = strjoin(prefix, me->mnt_dir, NULL); where = strjoin(prefix, me->mnt_dir, NULL);
if (!what || !where) { if (!what || !where) {
r = log_oom(); r = log_oom();
@ -416,6 +420,21 @@ static int parse_fstab(const char *prefix, bool initrd) {
if (is_path(where)) if (is_path(where))
path_kill_slashes(where); path_kill_slashes(where);
if (initrd) {
char _cleanup_free_ *mu = NULL, *name = NULL;
/* Skip generation, if unit already exists */
name = unit_name_from_path(where, ".mount");
if (!name)
return log_oom();
mu = strjoin(arg_dest, "/", name, NULL);
if (!mu)
return log_oom();
k = access(mu, R_OK);
if (k == 0)
continue;
}
log_debug("Found entry what=%s where=%s type=%s", what, where, me->mnt_type); log_debug("Found entry what=%s where=%s type=%s", what, where, me->mnt_type);
if (streq(me->mnt_type, "swap")) if (streq(me->mnt_type, "swap"))
@ -432,8 +451,8 @@ static int parse_fstab(const char *prefix, bool initrd) {
isnetwork = mount_is_network(me); isnetwork = mount_is_network(me);
k = add_mount(what, where, me->mnt_type, me->mnt_opts, k = add_mount(what, where, me->mnt_type, me->mnt_opts,
me->mnt_passno, noauto, nofail, automount, me->mnt_passno, noauto, nofail, automount,
isbind, isnetwork, fstab_path); isbind, isnetwork, initrd, fstab_path);
} }
if (k < 0) if (k < 0)
@ -447,10 +466,19 @@ finish:
static int parse_new_root_from_proc_cmdline(void) { static int parse_new_root_from_proc_cmdline(void) {
char *w, *state; char *w, *state;
_cleanup_free_ char *what = NULL, *type = NULL, *opts = NULL, *line = NULL; _cleanup_free_ char *what = NULL, *type = NULL, *opts = NULL, *line = NULL, *mu = NULL;
int r; int r;
size_t l; size_t l;
/* Skip generation, if sysroot.mount already exists */
mu = strjoin(arg_dest, "/", "sysroot.mount", NULL);
if (!mu)
return log_oom();
r = access(mu, R_OK);
if (r == 0)
return 0;
r = read_one_line_file("/proc/cmdline", &line); r = read_one_line_file("/proc/cmdline", &line);
if (r < 0) { if (r < 0) {
log_error("Failed to read /proc/cmdline, ignoring: %s", strerror(-r)); log_error("Failed to read /proc/cmdline, ignoring: %s", strerror(-r));
@ -506,7 +534,7 @@ static int parse_new_root_from_proc_cmdline(void) {
log_debug("Found entry what=%s where=/sysroot type=%s", what, type); log_debug("Found entry what=%s where=/sysroot type=%s", what, type);
r = add_mount(what, "/sysroot", type, opts, 0, false, false, false, r = add_mount(what, "/sysroot", type, opts, 0, false, false, false,
false, false, "/proc/cmdline"); false, false, true, "/proc/cmdline");
if (r < 0) if (r < 0)
return r; return r;
@ -566,7 +594,7 @@ static int parse_proc_cmdline(void) {
} }
int main(int argc, char *argv[]) { int main(int argc, char *argv[]) {
int r = 0, k, l = 0; int r = 0, k = 0, l = 0;
if (argc > 1 && argc != 4) { if (argc > 1 && argc != 4) {
log_error("This program takes three or no arguments."); log_error("This program takes three or no arguments.");
@ -585,16 +613,14 @@ int main(int argc, char *argv[]) {
if (parse_proc_cmdline() < 0) if (parse_proc_cmdline() < 0)
return EXIT_FAILURE; return EXIT_FAILURE;
if (in_initrd()) if (arg_enabled)
r = parse_new_root_from_proc_cmdline(); r = parse_fstab("", false);
if (!arg_enabled) if (in_initrd()) {
return (r < 0) ? EXIT_FAILURE : EXIT_SUCCESS; if (arg_enabled)
k = parse_fstab("/sysroot", true);
k = parse_fstab("", false); l = parse_new_root_from_proc_cmdline();
}
if (in_initrd())
l = parse_fstab("/sysroot", true);
return (r < 0) || (k < 0) || (l < 0) ? EXIT_FAILURE : EXIT_SUCCESS; return (r < 0) || (k < 0) || (l < 0) ? EXIT_FAILURE : EXIT_SUCCESS;
} }