From b8e5776d38728d6f81bc250eca2c573f0b618772 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Fri, 15 Nov 2019 14:00:12 +0100 Subject: [PATCH 1/2] mount: extend list of extrinsic mounts a bit --- src/core/mount.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/core/mount.c b/src/core/mount.c index 6e42176c09..1d1d2683da 100644 --- a/src/core/mount.c +++ b/src/core/mount.c @@ -409,9 +409,13 @@ static bool mount_is_extrinsic(Mount *m) { if (!MANAGER_IS_SYSTEM(UNIT(m)->manager)) /* We only automatically manage mounts if we are in system mode */ return true; + if (UNIT(m)->perpetual) /* All perpetual units never change state */ + return true; + if (PATH_IN_SET(m->where, /* Don't bother with the OS data itself */ - "/", - "/usr")) + "/", /* (strictly speaking redundant: should already be covered by the perpetual flag check above) */ + "/usr", + "/etc")) return true; if (PATH_STARTSWITH_SET(m->where, From bf7eedbf8f8c83d9e775c80275f98f506ec963c6 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Fri, 15 Nov 2019 14:00:54 +0100 Subject: [PATCH 2/2] mount: do not update exec deps on mountinfo changes Fixes: #13978 --- src/core/mount.c | 42 ++++++++++++++++++++++++++++-------------- 1 file changed, 28 insertions(+), 14 deletions(-) diff --git a/src/core/mount.c b/src/core/mount.c index 1d1d2683da..dfed691c43 100644 --- a/src/core/mount.c +++ b/src/core/mount.c @@ -550,6 +550,32 @@ static int mount_verify(Mount *m) { return 0; } +static int mount_add_non_exec_dependencies(Mount *m) { + int r; + assert(m); + + /* Adds in all dependencies directly responsible for ordering the mount, as opposed to dependencies + * resulting from the ExecContext and such. */ + + r = mount_add_device_dependencies(m); + if (r < 0) + return r; + + r = mount_add_mount_dependencies(m); + if (r < 0) + return r; + + r = mount_add_quota_dependencies(m); + if (r < 0) + return r; + + r = mount_add_default_dependencies(m); + if (r < 0) + return r; + + return 0; +} + static int mount_add_extras(Mount *m) { Unit *u = UNIT(m); int r; @@ -577,18 +603,6 @@ static int mount_add_extras(Mount *m) { return r; } - r = mount_add_device_dependencies(m); - if (r < 0) - return r; - - r = mount_add_mount_dependencies(m); - if (r < 0) - return r; - - r = mount_add_quota_dependencies(m); - if (r < 0) - return r; - r = unit_patch_contexts(u); if (r < 0) return r; @@ -601,7 +615,7 @@ static int mount_add_extras(Mount *m) { if (r < 0) return r; - r = mount_add_default_dependencies(m); + r = mount_add_non_exec_dependencies(m); if (r < 0) return r; @@ -1574,7 +1588,7 @@ static int mount_setup_existing_unit( unit_remove_dependencies(u, UNIT_DEPENDENCY_MOUNTINFO_IMPLICIT); - r = mount_add_extras(MOUNT(u)); + r = mount_add_non_exec_dependencies(MOUNT(u)); if (r < 0) return r; }