On Thu, 13.06.13 08:46, Ross Lagerwall (rosslagerw...@gmail.com) wrote:

> -                r = unit_add_dependency_by_name(u, UNIT_CONFLICTS, 
> SPECIAL_UMOUNT_TARGET, NULL, true);
> -                if (r < 0)
> -                        goto fail;
> +                if (!path_equal(where, "/") && !path_equal(where, "/usr") &&
> +                    !mount_test_option(options, "x-initrd.mount")) {
> +                        r = unit_add_dependency_by_name(u, UNIT_CONFLICTS, 
> SPECIAL_UMOUNT_TARGET, NULL, true);
> +                        if (r < 0)
> +                                goto fail;
> +                }

Hmm "x-initrd.mount" will never show up in the "options" variable, since
that's from the kernel, and the kernel doesn't know that option, that's
purely a userspace option.

For now I have commited a simpler patch which only checks for /, which
also mimics what fstab-generator currently does.

(In the long run we really should figure out which deps fstab-genrator
generates and which are implicitly added in by mount.c. Currently,
there's no clear structure in there, and quite a bit of duplication.)

Lennart

-- 
Lennart Poettering - Red Hat, Inc.
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to